Here needs a NULL check.

Issue found by coccinelle.

Signed-off-by: Qiujun Huang <hqjag...@gmail.com>
---
 arch/powerpc/platforms/powernv/opal.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/arch/powerpc/platforms/powernv/opal.c 
b/arch/powerpc/platforms/powernv/opal.c
index 2b3dfd0b6cdd..5b98c98817aa 100644
--- a/arch/powerpc/platforms/powernv/opal.c
+++ b/arch/powerpc/platforms/powernv/opal.c
@@ -808,9 +808,12 @@ static int opal_add_one_export(struct kobject *parent, 
const char *export_name,
 
        rc = of_property_read_u64_array(np, prop_name, &vals[0], 2);
        if (rc)
-               goto out;
+               return rc;
 
        attr = kzalloc(sizeof(*attr), GFP_KERNEL);
+       if (!attr)
+               return -ENOMEM;
+
        name = kstrdup(export_name, GFP_KERNEL);
        if (!name) {
                rc = -ENOMEM;
-- 
2.17.1

Reply via email to