livepatch: Call klp_match_callback() in klp_find_callback() to avoid code duplication
authorZhen Lei <thunder.leizhen@huawei.com>
Wed, 7 Dec 2022 03:23:04 +0000 (11:23 +0800)
committerLuis Chamberlain <mcgrof@kernel.org>
Tue, 13 Dec 2022 02:30:58 +0000 (18:30 -0800)
The implementation of function klp_match_callback() is identical to the
partial implementation of function klp_find_callback(). So call function
klp_match_callback() in function klp_find_callback() instead of the
duplicated code.

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Acked-by: Song Liu <song@kernel.org>
Reviewed-by: Petr Mladek <pmladek@suse.com>
Suggested-by: Petr Mladek <pmladek@suse.com>
Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
kernel/livepatch/core.c

index 50bfc34..201f0c0 100644 (file)
@@ -125,20 +125,10 @@ struct klp_find_arg {
        unsigned long pos;
 };
 
-static int klp_find_callback(void *data, const char *name,
-                            struct module *mod, unsigned long addr)
+static int klp_match_callback(void *data, unsigned long addr)
 {
        struct klp_find_arg *args = data;
 
-       if ((mod && !args->objname) || (!mod && args->objname))
-               return 0;
-
-       if (strcmp(args->name, name))
-               return 0;
-
-       if (args->objname && strcmp(args->objname, mod->name))
-               return 0;
-
        args->addr = addr;
        args->count++;
 
@@ -153,22 +143,21 @@ static int klp_find_callback(void *data, const char *name,
        return 0;
 }
 
-static int klp_match_callback(void *data, unsigned long addr)
+static int klp_find_callback(void *data, const char *name,
+                            struct module *mod, unsigned long addr)
 {
        struct klp_find_arg *args = data;
 
-       args->addr = addr;
-       args->count++;
+       if ((mod && !args->objname) || (!mod && args->objname))
+               return 0;
 
-       /*
-        * Finish the search when the symbol is found for the desired position
-        * or the position is not defined for a non-unique symbol.
-        */
-       if ((args->pos && (args->count == args->pos)) ||
-           (!args->pos && (args->count > 1)))
-               return 1;
+       if (strcmp(args->name, name))
+               return 0;
 
-       return 0;
+       if (args->objname && strcmp(args->objname, mod->name))
+               return 0;
+
+       return klp_match_callback(data, addr);
 }
 
 static int klp_find_object_symbol(const char *objname, const char *name,