Re: [PATCH] nfs: avoid dereferencing null pointer ininitiate_bulk_draining

From: Myklebust, Trond
Date: Sat Jan 05 2013 - 14:29:16 EST

On Sat, 2013-01-05 at 14:19 -0500, Nickolai Zeldovich wrote:
+AD4- Fix an inverted null pointer check in initiate+AF8-bulk+AF8-draining().
+AD4- Signed-off-by: Nickolai Zeldovich
+AD4- ---
+AD4- fs/nfs/callback+AF8-proc.c +AHw- 2 +--
+AD4- 1 file changed, 1 insertion(+-), 1 deletion(-)
+AD4- diff --git a/fs/nfs/callback+AF8-proc.c b/fs/nfs/callback+AF8-proc.c
+AD4- index c89b26b..264d1aa 100644
+AD4- --- a/fs/nfs/callback+AF8-proc.c
+AD4- +-+-+- b/fs/nfs/callback+AF8-proc.c
+AD4- +AEAAQA- -206,7 +-206,7 +AEAAQA- static u32 initiate+AF8-bulk+AF8-draining(struct nfs+AF8-client +ACo-clp,
+AD4- list+AF8-for+AF8-each+AF8-entry(lo, +ACY-server-+AD4-layouts, plh+AF8-layouts) +AHs-
+AD4- ino +AD0- igrab(lo-+AD4-plh+AF8-inode)+ADs-
+AD4- - if (ino)
+AD4- +- if (+ACE-ino)
+AD4- continue+ADs-
+AD4- spin+AF8-lock(+ACY-ino-+AD4-i+AF8-lock)+ADs-
+AD4- /+ACo- Is this layout in the process of being freed? +ACo-/

Thanks for spotting. Applied to the 'bugfixes' branch.

Trond Myklebust
Linux NFS client maintainer

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at