SFI: do not return freed pointer
authorDan Carpenter <error27@gmail.com>
Fri, 28 May 2010 10:33:15 +0000 (12:33 +0200)
committerLen Brown <len.brown@intel.com>
Tue, 1 Jun 2010 16:04:35 +0000 (12:04 -0400)
We never actually use the return value of sfi_sysfs_install_table() but
it still seems wrong to return a freed pointer.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/sfi/sfi_core.c

index 0051959..ceba593 100644 (file)
@@ -441,8 +441,10 @@ struct sfi_table_attr __init *sfi_sysfs_install_table(u64 pa)
 
        ret = sysfs_create_bin_file(tables_kobj,
                                  &tbl_attr->attr);
-       if (ret)
+       if (ret) {
                kfree(tbl_attr);
+               tbl_attr = NULL;
+       }
 
        sfi_unmap_table(th);
        return tbl_attr;