summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLi Zefan <lizf@cn.fujitsu.com>2008-06-13 11:09:16 +0800
committerGreg Kroah-Hartman <gregkh@suse.de>2008-07-21 21:54:56 -0700
commit185000fc556372b7fb7f26516c325f212030dbd3 (patch)
tree522e1a72bcf51c8160f48eb02e79c064ee7d7c9a
parenta231934bdf086a4fefc0df06e669499125a9db6f (diff)
downloadlinux-185000fc556372b7fb7f26516c325f212030dbd3.tar.gz
linux-185000fc556372b7fb7f26516c325f212030dbd3.tar.bz2
linux-185000fc556372b7fb7f26516c325f212030dbd3.zip
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>
-rw-r--r--samples/kobject/kset-example.c2
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)
*/
retval = kobject_init_and_add(&foo->kobj, &foo_ktype, NULL, "%s", name);
if (retval) {
- kfree(foo);
+ kobject_put(&foo->kobj);
return NULL;
}