public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [Patch] Remove needless checks in fs/9p/vfs_inode.c
@ 2006-06-13 20:07 Eric Sesterhenn
  0 siblings, 0 replies; only message in thread
From: Eric Sesterhenn @ 2006-06-13 20:07 UTC (permalink / raw)
  To: linux-kernel; +Cc: ericvh

hi,

coverity found two needless checks in vfs_inode.c (cid #1165 and #1164)
In both cases inode is always NULL when we goto error; either because it
is still initialized to NULL or is set to NULL explicitly. This patch
simply removes these checks to save some code.

Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>


--- linux-2.6.17-rc5/fs/9p/vfs_inode.c.orig	2006-06-13 22:01:56.000000000 +0200
+++ linux-2.6.17-rc5/fs/9p/vfs_inode.c	2006-06-13 22:02:18.000000000 +0200
@@ -530,9 +530,6 @@ error:
 	if (vfid)
 		v9fs_fid_destroy(vfid);
 
-	if (inode)
-		iput(inode);
-
 	return err;
 }
 
@@ -1171,9 +1168,6 @@ error:
 	if (vfid)
 		v9fs_fid_destroy(vfid);
 
-	if (inode)
-		iput(inode);
-
 	return err;
 
 }



^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2006-06-13 20:07 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-06-13 20:07 [Patch] Remove needless checks in fs/9p/vfs_inode.c Eric Sesterhenn

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox