bus: arm-ccn: Check memory allocation failure
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sun, 27 Aug 2017 10:06:50 +0000 (11:06 +0100)
committerPawel Moll <pawel.moll@arm.com>
Mon, 4 Dec 2017 16:48:53 +0000 (16:48 +0000)
Check memory allocation failures and return -ENOMEM in such cases

This avoids a potential NULL pointer dereference.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Acked-by: Scott Branden <scott.branden@broadcom.com>
Cc: stable@vger.kernel.org # 3.17+
Signed-off-by: Pawel Moll <pawel.moll@arm.com>
drivers/bus/arm-ccn.c

index 7625bf762acbf0bedd7da007fc51646e60f36e9b..a7951662f85b74cfaefc88067b1ebb15102613c3 100644 (file)
@@ -1271,6 +1271,10 @@ static int arm_ccn_pmu_init(struct arm_ccn *ccn)
                int len = snprintf(NULL, 0, "ccn_%d", ccn->dt.id);
 
                name = devm_kzalloc(ccn->dev, len + 1, GFP_KERNEL);
+               if (!name) {
+                       err = -ENOMEM;
+                       goto error_choose_name;
+               }
                snprintf(name, len + 1, "ccn_%d", ccn->dt.id);
        }
 
@@ -1319,6 +1323,7 @@ static int arm_ccn_pmu_init(struct arm_ccn *ccn)
 
 error_pmu_register:
 error_set_affinity:
+error_choose_name:
        ida_simple_remove(&arm_ccn_pmu_ida, ccn->dt.id);
        for (i = 0; i < ccn->num_xps; i++)
                writel(0, ccn->xp[i].base + CCN_XP_DT_CONTROL);