commit e67eab39bee26f509d38d00ca1a8f24b63f46a31
author Alan Cox <alan@linux.intel.com> Thu Dec 20 15:05:54 2012 -0800
committer Linus Torvalds <torvalds@linux-foundation.org> Thu Dec 20 17:40:21 2012 -0800
tree 252072ae63bd5ecb8186242ace12e7df3a1311ad
parent a68c2f12b4b28994aaf622bbe5724b7258cc2fcf

keys: fix unreachable code

We set ret to NULL then test it. Remove the bogus test

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

security/keys/process_keys.c

diff --git a/security/keys/process_keys.c b/security/keys/process_keys.c
index 58dfe08..20e4bf5 100644
--- a/security/keys/process_keys.c
+++ b/security/keys/process_keys.c
@@ -367,8 +367,6 @@
 
 		switch (PTR_ERR(key_ref)) {
 		case -EAGAIN: /* no key */
-			if (ret)
-				break;
 		case -ENOKEY: /* negative key */
 			ret = key_ref;
 			break;