Messages in this thread Patch in this message | | | From | Caizhiyong <> | Subject | [PATCH] regmap: Fix 'ret' would return an uninitialized value | Date | Fri, 15 Nov 2013 11:30:55 +0000 |
| |
From: Cai Zhiyong <caizhiyong@huawei.com> Date: Mon, 11 Nov 2013 19:26:14 +0800 Subject: [PATCH] regmap: Fix 'ret' would return an uninitialized value
This patch give a warning when calling regmap_register_patch with parameter num_regs <= 0.
When the num_regs parameter is zero and krealloc doesn't fail, then the code would return an uninitialized value. However, calling this function with num_regs == 0, would be a waste as it essentially does nothing.
Signed-off-by: Cai Zhiyong <caizhiyong@huawei.com> --- drivers/base/regmap/regmap.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c index 9c021d9..7b5c28a 100644 --- a/drivers/base/regmap/regmap.c +++ b/drivers/base/regmap/regmap.c @@ -2170,9 +2170,12 @@ int regmap_register_patch(struct regmap *map, const struct reg_default *regs, int num_regs) { struct reg_default *p; int i, ret; bool bypass; + if (WARN_ONCE(num_regs <= 0)) + return 0; + map->lock(map->lock_arg); bypass = map->cache_bypass; -- 1.8.1.5
| |