diff options
| author | Akinobu Mita <akinobu.mita@gmail.com> | 2007-04-26 00:12:09 -0700 | 
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-04-27 10:57:34 -0700 | 
| commit | 240936e18b75937e7866934df723c2db0011d24f (patch) | |
| tree | 0110cc4eecf98a3dec75e6a9d824907086591fe4 | |
| parent | a53c46dc8253cc613ad66a2ca7aad6de8b7e61b9 (diff) | |
| download | olio-linux-3.10-240936e18b75937e7866934df723c2db0011d24f.tar.xz olio-linux-3.10-240936e18b75937e7866934df723c2db0011d24f.zip  | |
mod_sysfs_setup() doesn't return errno when kobject_add_dir() failure occurs
mod_sysfs_setup() doesn't return an errno when kobject_add_dir() for module
"holders" directory fails.  So caller of mod_sysfs_setup() will keep going
and get oops.
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | kernel/module.c | 4 | 
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/module.c b/kernel/module.c index dcdb32b8b13..9da5af668a2 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -1148,8 +1148,10 @@ int mod_sysfs_setup(struct module *mod,  		goto out;  	mod->holders_dir = kobject_add_dir(&mod->mkobj.kobj, "holders"); -	if (!mod->holders_dir) +	if (!mod->holders_dir) { +		err = -ENOMEM;  		goto out_unreg; +	}  	err = module_param_sysfs_setup(mod, kparam, num_params);  	if (err)  |