pinctrl: tegra-xusb: Don't leak configurations
authorThierry Reding <treding@nvidia.com>
Thu, 16 Oct 2014 10:12:22 +0000 (12:12 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 28 Oct 2014 15:59:31 +0000 (16:59 +0100)
The pinctrl config helpers make a separate copy of the configuration, so
callers must make sure to free any dynamically allocated memory that was
used to store it.

Signed-off-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-tegra-xusb.c

index 1631ec94fb023ee93d6d787684893d94286c1bc6..080ec7723ef2e4319810bef820a9578a25895557 100644 (file)
@@ -20,6 +20,7 @@
 #include <linux/pinctrl/pinmux.h>
 #include <linux/platform_device.h>
 #include <linux/reset.h>
+#include <linux/slab.h>
 
 #include <dt-bindings/pinctrl/pinctrl-tegra-xusb.h>
 
@@ -171,7 +172,7 @@ static int tegra_xusb_padctl_parse_subnode(struct tegra_xusb_padctl *padctl,
                        if (err == -EINVAL)
                                continue;
 
-                       return err;
+                       goto out;
                }
 
                config = TEGRA_XUSB_PADCTL_PACK(properties[i].param, value);
@@ -179,7 +180,7 @@ static int tegra_xusb_padctl_parse_subnode(struct tegra_xusb_padctl *padctl,
                err = pinctrl_utils_add_config(padctl->pinctrl, &configs,
                                               &num_configs, config);
                if (err < 0)
-                       return err;
+                       goto out;
        }
 
        if (function)
@@ -190,14 +191,14 @@ static int tegra_xusb_padctl_parse_subnode(struct tegra_xusb_padctl *padctl,
 
        err = of_property_count_strings(np, "nvidia,lanes");
        if (err < 0)
-               return err;
+               goto out;
 
        reserve *= err;
 
        err = pinctrl_utils_reserve_map(padctl->pinctrl, maps, reserved_maps,
                                        num_maps, reserve);
        if (err < 0)
-               return err;
+               goto out;
 
        of_property_for_each_string(np, "nvidia,lanes", prop, group) {
                if (function) {
@@ -205,7 +206,7 @@ static int tegra_xusb_padctl_parse_subnode(struct tegra_xusb_padctl *padctl,
                                        reserved_maps, num_maps, group,
                                        function);
                        if (err < 0)
-                               return err;
+                               goto out;
                }
 
                if (num_configs) {
@@ -214,11 +215,15 @@ static int tegra_xusb_padctl_parse_subnode(struct tegra_xusb_padctl *padctl,
                                        configs, num_configs,
                                        PIN_MAP_TYPE_CONFIGS_GROUP);
                        if (err < 0)
-                               return err;
+                               goto out;
                }
        }
 
-       return 0;
+       err = 0;
+
+out:
+       kfree(configs);
+       return err;
 }
 
 static int tegra_xusb_padctl_dt_node_to_map(struct pinctrl_dev *pinctrl,