From 6ac1bccfe1fe424908e9fa001288b5fd070317d1 Mon Sep 17 00:00:00 2001 From: Tobias Stoeckmann Date: Thu, 24 Oct 2024 21:04:28 +0200 Subject: [PATCH] libkmod: Clarify function signature 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 Reviewed-by: Emil Velikov Link: https://github.com/kmod-project/kmod/pull/211 Signed-off-by: Lucas De Marchi --- libkmod/libkmod-module.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libkmod/libkmod-module.c b/libkmod/libkmod-module.c index 28ddee4e..b410cc5e 100644 --- a/libkmod/libkmod-module.c +++ b/libkmod/libkmod-module.c @@ -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; } -- 2.47.3