summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorGreg Kroah-Hartman <gregkh@suse.de>2006-01-20 14:08:59 -0800
committerGreg Kroah-Hartman <gregkh@suse.de>2006-02-06 12:17:17 -0800
commitc171fef5c8566cf5f57877e7832fa696ecdf5228 (patch)
treec599efed9172ce2f31835d4df01936063fad3a77 /lib
parente3f749c4af69c4344d89f11e2293e3790eb4eaca (diff)
downloadlinux-3.10-c171fef5c8566cf5f57877e7832fa696ecdf5228.tar.gz
linux-3.10-c171fef5c8566cf5f57877e7832fa696ecdf5228.tar.bz2
linux-3.10-c171fef5c8566cf5f57877e7832fa696ecdf5228.zip
[PATCH] kobject_add() must have a valid name in order to succeed.
So we might as well check to verify this, and let the user know that something is wrong if they didn't do it correctly, instead of oopsing later on in kobject_get_name() or somewhere else. Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'lib')
-rw-r--r--lib/kobject.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/kobject.c b/lib/kobject.c
index 7a0e6809490..fe4ae36ce96 100644
--- a/lib/kobject.c
+++ b/lib/kobject.c
@@ -162,6 +162,11 @@ int kobject_add(struct kobject * kobj)
return -ENOENT;
if (!kobj->k_name)
kobj->k_name = kobj->name;
+ if (!kobj->k_name) {
+ pr_debug("kobject attempted to be registered with no name!\n");
+ WARN_ON(1);
+ return -EINVAL;
+ }
parent = kobject_get(kobj->parent);
pr_debug("kobject %s: registering. parent: %s, set: %s\n",