]> git.ipfire.org Git - thirdparty/kmod.git/commitdiff
libkmod: Clarify function signature
authorTobias Stoeckmann <tobias@stoeckmann.org>
Thu, 24 Oct 2024 19:04:28 +0000 (21:04 +0200)
committerLucas De Marchi <lucas.de.marchi@gmail.com>
Tue, 29 Oct 2024 02:55:10 +0000 (21:55 -0500)
The module_get_dependencies_noref function may modify content of
given kmod_module struct, so clarify this fact.

Actually we would have to adjust kmod API for total clarification,
but let's start with internally used functions first.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Reviewed-by: Emil Velikov <emil.l.velikov@gmail.com>
Link: https://github.com/kmod-project/kmod/pull/211
Signed-off-by: Lucas De Marchi <lucas.de.marchi@gmail.com>
libkmod/libkmod-module.c

index 28ddee4e50fdd8ff378f42bfdd6354c01c483899..b410cc5e74da49e1bb6482763494db89a4b9231b 100644 (file)
@@ -506,14 +506,14 @@ KMOD_EXPORT int kmod_module_get_filtered_blacklist(const struct kmod_ctx *ctx,
        return kmod_module_apply_filter(ctx, KMOD_FILTER_BLACKLIST, input, output);
 }
 
-static void module_get_dependencies_noref(const struct kmod_module *mod)
+static void module_get_dependencies_noref(struct kmod_module *mod)
 {
        if (!mod->init.dep) {
                /* lazy init */
                char *line = kmod_search_moddep(mod->ctx, mod->name);
 
                if (line != NULL) {
-                       kmod_module_parse_depline((struct kmod_module *)mod, line);
+                       kmod_module_parse_depline(mod, line);
                        free(line);
                }
        }
@@ -526,7 +526,7 @@ KMOD_EXPORT struct kmod_list *kmod_module_get_dependencies(const struct kmod_mod
        if (mod == NULL)
                return NULL;
 
-       module_get_dependencies_noref(mod);
+       module_get_dependencies_noref((struct kmod_module *)mod);
 
        kmod_list_foreach(l, mod->dep) {
                l_new = kmod_list_append(list_new, kmod_module_ref(l->data));
@@ -575,7 +575,7 @@ KMOD_EXPORT const char *kmod_module_get_path(const struct kmod_module *mod)
                return NULL;
 
        /* lazy init */
-       module_get_dependencies_noref(mod);
+       module_get_dependencies_noref((struct kmod_module *)mod);
 
        return mod->path;
 }