sysctl: remove impossible condition check

Remove checks for conditions that will never happen. If procname is NULL
the loop would already had bailed out, so there's no need to check it
again.

At the same time this also compacts the function find_in_table() by
refactoring it to be easier to read.

Signed-off-by: Lucas De Marchi <lucas.demarchi@profusion.mobi>
Reviewed-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>

diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
index a6b62173d4c310e4fa4b131a214043667fa67726..d82f4a8b4b80c1209a80bb150d2e81d8299b78fc 100644 (file)
--- a/fs/proc/proc_sysctl.c
+++ b/fs/proc/proc_sysctl.c
@@ -59,17 +59,11 @@ out:
 
 static struct ctl_table *find_in_table(struct ctl_table *p, struct qstr *name)
 {
-       int len;
        for ( ; p->procname; p++) {
-
-               if (!p->procname)
-                       continue;
-
-               len = strlen(p->procname);
-               if (len != name->len)
+               if (strlen(p->procname) != name->len)
                        continue;
 
-               if (memcmp(p->procname, name->name, len) != 0)
+               if (memcmp(p->procname, name->name, name->len) != 0)
                        continue;
 
                /* I have a match */
@@ -266,10 +260,6 @@ static int scan(struct ctl_table_header *head, ctl_table *table,
        for (; table->procname; table++, (*pos)++) {
                int res;
 
-               /* Can't do anything without a proc name */
-               if (!table->procname)
-                       continue;
-
                if (*pos < file->f_pos)
                        continue;