diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2008-06-12 23:09:16 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-07-22 00:54:56 -0400 |
commit | 185000fc556372b7fb7f26516c325f212030dbd3 (patch) | |
tree | 522e1a72bcf51c8160f48eb02e79c064ee7d7c9a /samples/kobject | |
parent | a231934bdf086a4fefc0df06e669499125a9db6f (diff) |
kobject: should use kobject_put() in kset-example
We should call kobject_put() instead of kfree() if kobject_init_and_add()
returns an error, shouldn't we? Don't set up a bad example ;)
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'samples/kobject')
-rw-r--r-- | samples/kobject/kset-example.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/samples/kobject/kset-example.c b/samples/kobject/kset-example.c index b0a1b4fe6584..7395c0bbae18 100644 --- a/samples/kobject/kset-example.c +++ b/samples/kobject/kset-example.c | |||
@@ -211,7 +211,7 @@ static struct foo_obj *create_foo_obj(const char *name) | |||
211 | */ | 211 | */ |
212 | retval = kobject_init_and_add(&foo->kobj, &foo_ktype, NULL, "%s", name); | 212 | retval = kobject_init_and_add(&foo->kobj, &foo_ktype, NULL, "%s", name); |
213 | if (retval) { | 213 | if (retval) { |
214 | kfree(foo); | 214 | kobject_put(&foo->kobj); |
215 | return NULL; | 215 | return NULL; |
216 | } | 216 | } |
217 | 217 | ||