* [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance i
@ 2012-07-17 0:14 ` Fengguang Wu
0 siblings, 0 replies; 4+ messages in thread
From: Fengguang Wu @ 2012-07-17 0:14 UTC (permalink / raw)
To: Chuck Lever; +Cc: kernel-janitors, Linux-NFS
Hi Chuck,
There are new sparse warnings (likely a true locking bug) show up in
tree: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git nfs-for-bryan
head: 6a1a1e34dc55f17e7bd260809207442dbb7a0296
commit: 6a1a1e34dc55f17e7bd260809207442dbb7a0296 [100/100] SUNRPC: Add rpcauth_list_flavors()
All sparse warnings:
net/sunrpc/auth_gss/gss_mech_switch.c:143:21: sparse: symbol '_gss_mech_get_by_name' was not declared. Should it be static?
net/sunrpc/auth_gss/gss_mech_switch.c:208:21: sparse: symbol '_gss_mech_get_by_pseudoflavor' was not declared. Should it be static?
+ net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit
vim +252 net/sunrpc/auth_gss/gss_mech_switch.c
249 * The returned array is not sorted by any policy. Callers should not
250 * rely on the order of the items in the returned array.
251 */
> 252 int gss_mech_list_pseudoflavors(rpc_authflavor_t *array_ptr, int size)
253 {
254 struct gss_api_mech *pos = NULL;
255 int j, i = 0;
---
0-DAY kernel build testing backend Open Source Technology Centre
Fengguang Wu <wfg@linux.intel.com> Intel Corporation
^ permalink raw reply [flat|nested] 4+ messages in thread
* [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit
@ 2012-07-17 0:14 ` Fengguang Wu
0 siblings, 0 replies; 4+ messages in thread
From: Fengguang Wu @ 2012-07-17 0:14 UTC (permalink / raw)
To: Chuck Lever; +Cc: kernel-janitors, Linux-NFS
Hi Chuck,
There are new sparse warnings (likely a true locking bug) show up in
tree: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git nfs-for-bryan
head: 6a1a1e34dc55f17e7bd260809207442dbb7a0296
commit: 6a1a1e34dc55f17e7bd260809207442dbb7a0296 [100/100] SUNRPC: Add rpcauth_list_flavors()
All sparse warnings:
net/sunrpc/auth_gss/gss_mech_switch.c:143:21: sparse: symbol '_gss_mech_get_by_name' was not declared. Should it be static?
net/sunrpc/auth_gss/gss_mech_switch.c:208:21: sparse: symbol '_gss_mech_get_by_pseudoflavor' was not declared. Should it be static?
+ net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit
vim +252 net/sunrpc/auth_gss/gss_mech_switch.c
249 * The returned array is not sorted by any policy. Callers should not
250 * rely on the order of the items in the returned array.
251 */
> 252 int gss_mech_list_pseudoflavors(rpc_authflavor_t *array_ptr, int size)
253 {
254 struct gss_api_mech *pos = NULL;
255 int j, i = 0;
---
0-DAY kernel build testing backend Open Source Technology Centre
Fengguang Wu <wfg@linux.intel.com> Intel Corporation
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalan
2012-07-17 0:14 ` [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit Fengguang Wu
@ 2012-07-17 0:52 ` Chuck Lever
-1 siblings, 0 replies; 4+ messages in thread
From: Chuck Lever @ 2012-07-17 0:52 UTC (permalink / raw)
To: Fengguang Wu; +Cc: kernel-janitors, Linux-NFS
On Jul 16, 2012, at 8:14 PM, Fengguang Wu wrote:
> Hi Chuck,
>
> There are new sparse warnings (likely a true locking bug) show up in
>
> tree: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git nfs-for-bryan
> head: 6a1a1e34dc55f17e7bd260809207442dbb7a0296
> commit: 6a1a1e34dc55f17e7bd260809207442dbb7a0296 [100/100] SUNRPC: Add rpcauth_list_flavors()
>
> All sparse warnings:
>
> net/sunrpc/auth_gss/gss_mech_switch.c:143:21: sparse: symbol '_gss_mech_get_by_name' was not declared. Should it be static?
> net/sunrpc/auth_gss/gss_mech_switch.c:208:21: sparse: symbol '_gss_mech_get_by_pseudoflavor' was not declared. Should it be static?
> + net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit
gss_mech_list_pseudoflavors() is missing a spin_unlock() before the "return -ENOMEM;".
> vim +252 net/sunrpc/auth_gss/gss_mech_switch.c
> 249 * The returned array is not sorted by any policy. Callers should not
> 250 * rely on the order of the items in the returned array.
> 251 */
>> 252 int gss_mech_list_pseudoflavors(rpc_authflavor_t *array_ptr, int size)
> 253 {
> 254 struct gss_api_mech *pos = NULL;
> 255 int j, i = 0;
--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit
@ 2012-07-17 0:52 ` Chuck Lever
0 siblings, 0 replies; 4+ messages in thread
From: Chuck Lever @ 2012-07-17 0:52 UTC (permalink / raw)
To: Fengguang Wu; +Cc: kernel-janitors, Linux-NFS
On Jul 16, 2012, at 8:14 PM, Fengguang Wu wrote:
> Hi Chuck,
>
> There are new sparse warnings (likely a true locking bug) show up in
>
> tree: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git nfs-for-bryan
> head: 6a1a1e34dc55f17e7bd260809207442dbb7a0296
> commit: 6a1a1e34dc55f17e7bd260809207442dbb7a0296 [100/100] SUNRPC: Add rpcauth_list_flavors()
>
> All sparse warnings:
>
> net/sunrpc/auth_gss/gss_mech_switch.c:143:21: sparse: symbol '_gss_mech_get_by_name' was not declared. Should it be static?
> net/sunrpc/auth_gss/gss_mech_switch.c:208:21: sparse: symbol '_gss_mech_get_by_pseudoflavor' was not declared. Should it be static?
> + net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit
gss_mech_list_pseudoflavors() is missing a spin_unlock() before the "return -ENOMEM;".
> vim +252 net/sunrpc/auth_gss/gss_mech_switch.c
> 249 * The returned array is not sorted by any policy. Callers should not
> 250 * rely on the order of the items in the returned array.
> 251 */
>> 252 int gss_mech_list_pseudoflavors(rpc_authflavor_t *array_ptr, int size)
> 253 {
> 254 struct gss_api_mech *pos = NULL;
> 255 int j, i = 0;
--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-07-17 0:52 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-17 0:14 [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance i Fengguang Wu
2012-07-17 0:14 ` [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit Fengguang Wu
2012-07-17 0:52 ` [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalan Chuck Lever
2012-07-17 0:52 ` [nfs:nfs-for-bryan 100/100] net/sunrpc/auth_gss/gss_mech_switch.c:252:5: sparse: context imbalance in 'gss_mech_list_pseudoflavors' - wrong count at exit Chuck Lever
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.