diff options
| author | Dave Young <hidave.darkstar@gmail.com> | 2009-05-11 02:17:45 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-06-16 00:30:24 -0400 |
| commit | d9cd8f37855b012757818f31390e8f84db251c89 (patch) | |
| tree | 4785d05f5df93747b6aa3386c98784d441d61377 /lib | |
| parent | 59a3cd7f9da60ac4ba8ae5a4cddc48fe4a450129 (diff) | |
kobject: make kset_create check kobject_set_name return value
kset_create should check the kobject_set_name return value.
Add the return value checking code.
Signed-off-by: Dave Young <hidave.darkstar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/kobject.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/kobject.c b/lib/kobject.c index bacf6fe4f7a0..b512b746d2af 100644 --- a/lib/kobject.c +++ b/lib/kobject.c | |||
| @@ -793,11 +793,16 @@ static struct kset *kset_create(const char *name, | |||
| 793 | struct kobject *parent_kobj) | 793 | struct kobject *parent_kobj) |
| 794 | { | 794 | { |
| 795 | struct kset *kset; | 795 | struct kset *kset; |
| 796 | int retval; | ||
| 796 | 797 | ||
| 797 | kset = kzalloc(sizeof(*kset), GFP_KERNEL); | 798 | kset = kzalloc(sizeof(*kset), GFP_KERNEL); |
| 798 | if (!kset) | 799 | if (!kset) |
| 799 | return NULL; | 800 | return NULL; |
| 800 | kobject_set_name(&kset->kobj, name); | 801 | retval = kobject_set_name(&kset->kobj, name); |
| 802 | if (retval) { | ||
| 803 | kfree(kset); | ||
| 804 | return NULL; | ||
| 805 | } | ||
| 801 | kset->uevent_ops = uevent_ops; | 806 | kset->uevent_ops = uevent_ops; |
| 802 | kset->kobj.parent = parent_kobj; | 807 | kset->kobj.parent = parent_kobj; |
| 803 | 808 | ||
