diff options
| author | Greg Kroah-Hartman <gregkh@suse.de> | 2006-01-20 14:08:59 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-02-06 12:17:17 -0800 |
| commit | c171fef5c8566cf5f57877e7832fa696ecdf5228 (patch) | |
| tree | c599efed9172ce2f31835d4df01936063fad3a77 /lib | |
| parent | e3f749c4af69c4344d89f11e2293e3790eb4eaca (diff) | |
| download | op-kernel-dev-c171fef5c8566cf5f57877e7832fa696ecdf5228.zip op-kernel-dev-c171fef5c8566cf5f57877e7832fa696ecdf5228.tar.gz | |
[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.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/kobject.c b/lib/kobject.c index 7a0e680..fe4ae36 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", |
