Commit ee61abb3 authored by Linus Torvalds's avatar Linus Torvalds

module: fix missing module_mutex unlock

Commit 1fb9341a ("module: put modules in list much earlier") moved
some of the module initialization code around, and in the process
changed the exit paths too.  But for the duplicate export symbol error
case the change made the ddebug_cleanup path jump to after the module
mutex unlock, even though it happens with the mutex held.

Rusty has some patches to split this function up into some helper
functions, hopefully the mess of complex goto targets will go away
eventually.
Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Cc: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 22636476
...@@ -3274,8 +3274,8 @@ static int load_module(struct load_info *info, const char __user *uargs, ...@@ -3274,8 +3274,8 @@ static int load_module(struct load_info *info, const char __user *uargs,
/* module_bug_cleanup needs module_mutex protection */ /* module_bug_cleanup needs module_mutex protection */
mutex_lock(&module_mutex); mutex_lock(&module_mutex);
module_bug_cleanup(mod); module_bug_cleanup(mod);
mutex_unlock(&module_mutex);
ddebug_cleanup: ddebug_cleanup:
mutex_unlock(&module_mutex);
dynamic_debug_remove(info->debug); dynamic_debug_remove(info->debug);
synchronize_sched(); synchronize_sched();
kfree(mod->args); kfree(mod->args);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment