aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/base/regmap
diff options
context:
space:
mode:
authorXiubo Li <Li.Xiubo@freescale.com>2014-09-28 05:09:54 -0400
committerMark Brown <broonie@kernel.org>2014-09-28 06:55:07 -0400
commitd6b41cb06044a7d895db82bdd54f6e4219970510 (patch)
treea3dc551bd4d54b98fc4cf58b2f90cba92aeef63c /drivers/base/regmap
parent5336be8416a71b5568d2cf54a2f2066abe9f2a53 (diff)
regmap: fix possible ZERO_SIZE_PTR pointer dereferencing error.
Since we cannot make sure the 'val_count' will always be none zero here, and then if it equals to zero, the kmemdup() will return ZERO_SIZE_PTR, which equals to ((void *)16). So this patch fix this with just doing the zero check before calling kmemdup(). Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com> Signed-off-by: Mark Brown <broonie@kernel.org> Cc: stable@vger.kernel.org
Diffstat (limited to 'drivers/base/regmap')
-rw-r--r--drivers/base/regmap/regmap.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c
index b903aff59af3..e677a1b79f95 100644
--- a/drivers/base/regmap/regmap.c
+++ b/drivers/base/regmap/regmap.c
@@ -1659,6 +1659,9 @@ out:
1659 } else { 1659 } else {
1660 void *wval; 1660 void *wval;
1661 1661
1662 if (!val_count)
1663 return -EINVAL;
1664
1662 wval = kmemdup(val, val_count * val_bytes, GFP_KERNEL); 1665 wval = kmemdup(val, val_count * val_bytes, GFP_KERNEL);
1663 if (!wval) { 1666 if (!wval) {
1664 dev_err(map->dev, "Error in memory allocation\n"); 1667 dev_err(map->dev, "Error in memory allocation\n");