summaryrefslogtreecommitdiffstats
path: root/security
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2017-01-14 16:34:25 +0100
committerPaul Moore <paul@paul-moore.com>2017-03-23 22:45:29 +0100
commit3a0aa56518432a1a598ac3e48a93f2e99c66a393 (patch)
tree4f225e2731cd2b3166b2720acd4088b78cf14f8b /security
parentselinux: Delete an unnecessary return statement in policydb_destroy() (diff)
downloadlinux-3a0aa56518432a1a598ac3e48a93f2e99c66a393.tar.xz
linux-3a0aa56518432a1a598ac3e48a93f2e99c66a393.zip
selinux: Return directly after a failed next_entry() in genfs_read()
Return directly after a call of the function "next_entry" failed at the beginning. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security')
-rw-r--r--security/selinux/ss/policydb.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/security/selinux/ss/policydb.c b/security/selinux/ss/policydb.c
index bccc9acf6bc5..375e304070e1 100644
--- a/security/selinux/ss/policydb.c
+++ b/security/selinux/ss/policydb.c
@@ -2006,7 +2006,7 @@ static int genfs_read(struct policydb *p, void *fp)
rc = next_entry(buf, fp, sizeof(u32));
if (rc)
- goto out;
+ return rc;
nel = le32_to_cpu(buf[0]);
for (i = 0; i < nel; i++) {