pinctrl: qcom: qdf2xxx: improve error checking and reporting
authorTimur Tabi <timur@codeaurora.org>
Tue, 10 Nov 2015 15:57:10 +0000 (09:57 -0600)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 1 Dec 2015 08:59:02 +0000 (09:59 +0100)
The driver doesn't report an error message if the ACPI tables are missing
the num-gpios property (which indicates how many GPIOs there are on this
SOC), and it didn't check to ensure that the mallocs didn't fail.

Signed-off-by: Timur Tabi <timur@codeaurora.org>
Reviewed-by: Björn Andersson <bjorn.andersson@sonymobile.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/qcom/pinctrl-qdf2xxx.c

index e9ff3bc150bbc7ee441c91f591dbc19f3a4bf438..f448534edf467d31c6fb3cc3a59adcccf8eea797 100644 (file)
@@ -32,6 +32,9 @@
 
 static struct msm_pinctrl_soc_data qdf2xxx_pinctrl;
 
+/* A reasonable limit to the number of GPIOS */
+#define MAX_GPIOS      256
+
 static int qdf2xxx_pinctrl_probe(struct platform_device *pdev)
 {
        struct pinctrl_pin_desc *pins;
@@ -42,11 +45,13 @@ static int qdf2xxx_pinctrl_probe(struct platform_device *pdev)
 
        /* Query the number of GPIOs from ACPI */
        ret = device_property_read_u32(&pdev->dev, "num-gpios", &num_gpios);
-       if (ret < 0)
+       if (ret < 0) {
+               dev_warn(&pdev->dev, "missing num-gpios property\n");
                return ret;
+       }
 
-       if (!num_gpios) {
-               dev_warn(&pdev->dev, "missing num-gpios property\n");
+       if (!num_gpios || num_gpios > MAX_GPIOS) {
+               dev_warn(&pdev->dev, "invalid num-gpios property\n");
                return -ENODEV;
        }
 
@@ -55,6 +60,9 @@ static int qdf2xxx_pinctrl_probe(struct platform_device *pdev)
        groups = devm_kcalloc(&pdev->dev, num_gpios,
                sizeof(struct msm_pingroup), GFP_KERNEL);
 
+       if (!pins || !groups)
+               return -ENOMEM;
+
        for (i = 0; i < num_gpios; i++) {
                pins[i].number = i;