platform/x86: hp-bioscfg: fix error reporting in hp_add_other_attributes()
authorDan Carpenter <dan.carpenter@linaro.org>
Tue, 18 Jul 2023 07:06:26 +0000 (10:06 +0300)
committerHans de Goede <hdegoede@redhat.com>
Wed, 26 Jul 2023 14:16:42 +0000 (16:16 +0200)
Return a negative error code instead of returning success.

Fixes: a34fc329b189 ("platform/x86: hp-bioscfg: bioscfg")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/138641cc-52c0-41a5-8176-ad01c7e28c67@moroto.mountain
Reviewed-by: Jorge Lopez <jorge.lopez2@hp.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/hp/hp-bioscfg/bioscfg.c

index 32d9c36ca55392ece40c87a5e583d3e437212c80..8c4f9e12f0181d950dd6b38b0069298f5f96bd4d 100644 (file)
@@ -630,13 +630,18 @@ static int hp_add_other_attributes(int attr_type)
        switch (attr_type) {
        case HPWMI_SECURE_PLATFORM_TYPE:
                ret = hp_populate_secure_platform_data(attr_name_kobj);
+               if (ret)
+                       goto err_other_attr_init;
                break;
 
        case HPWMI_SURE_START_TYPE:
                ret = hp_populate_sure_start_data(attr_name_kobj);
+               if (ret)
+                       goto err_other_attr_init;
                break;
 
        default:
+               ret = -EINVAL;
                goto err_other_attr_init;
        }