pstore: fix build warning for unused return value from sysfs_create_file
authorTony Luck <tony.luck@intel.com>
Fri, 7 Jan 2011 00:58:58 +0000 (16:58 -0800)
committerTony Luck <tony.luck@intel.com>
Fri, 7 Jan 2011 00:58:58 +0000 (16:58 -0800)
fs/pstore/inode.c: In function 'init_pstore_fs':
fs/pstore/inode.c:266: warning: ignoring return value of 'sysfs_create_file', declared with attribute warn_unused_result

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Tony Luck <tony.luck@intel.com>
fs/pstore/inode.c

index 0e806aa..549d245 100644 (file)
@@ -256,23 +256,28 @@ static struct file_system_type pstore_fs_type = {
 
 static int __init init_pstore_fs(void)
 {
 
 static int __init init_pstore_fs(void)
 {
-       int ret = 0;
+       int rc = 0;
        struct kobject *pstorefs_kobj;
 
        pstorefs_kobj = kobject_create_and_add("pstore", fs_kobj);
        struct kobject *pstorefs_kobj;
 
        pstorefs_kobj = kobject_create_and_add("pstore", fs_kobj);
-       if (!pstorefs_kobj)
-               return -ENOMEM;
+       if (!pstorefs_kobj) {
+               rc = -ENOMEM;
+               goto done;
+       }
 
 
-       sysfs_create_file(pstorefs_kobj, &pstore_kmsg_bytes_attr.attr);
+       rc = sysfs_create_file(pstorefs_kobj, &pstore_kmsg_bytes_attr.attr);
+       if (rc)
+               goto done1;
 
 
-       ret = register_filesystem(&pstore_fs_type);
+       rc = register_filesystem(&pstore_fs_type);
+       if (rc == 0)
+               goto done;
 
 
-       if (ret) {
-               sysfs_remove_file(pstorefs_kobj, &pstore_kmsg_bytes_attr.attr);
-               kobject_put(pstorefs_kobj);
-       }
-
-       return ret;
+       sysfs_remove_file(pstorefs_kobj, &pstore_kmsg_bytes_attr.attr);
+done1:
+       kobject_put(pstorefs_kobj);
+done:
+       return rc;
 }
 module_init(init_pstore_fs)
 
 }
 module_init(init_pstore_fs)