diff options
| author | Sascha Hauer <s.hauer@pengutronix.de> | 2015-07-07 16:24:23 +0200 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2015-08-05 14:47:59 +0200 |
| commit | 4955c2c1bc2b0a5dca469797dd635130a0790281 (patch) | |
| tree | b9ef900bde44f4326a9db6b115b823df53136194 /drivers | |
| parent | db48d9d1906c26c8d9a83ff1d9cb7ae75a00ba83 (diff) | |
| download | barebox-4955c2c1bc2b0a5dca469797dd635130a0790281.tar.gz barebox-4955c2c1bc2b0a5dca469797dd635130a0790281.tar.xz | |
mtd: concat: initialize *retlen
In concat_write and concat_read *retlen is only added to, but it's never
initialized. If the caller hasn't initialized retlen doesn't contain
valid values. Fix this by initializing *retlen properly.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/mtd/mtdconcat.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/mtd/mtdconcat.c b/drivers/mtd/mtdconcat.c index 10d3dc4c36..837614fdc5 100644 --- a/drivers/mtd/mtdconcat.c +++ b/drivers/mtd/mtdconcat.c @@ -66,6 +66,8 @@ concat_read(struct mtd_info *mtd, loff_t from, size_t len, int ret = 0, err; int i; + *retlen = 0; + for (i = 0; i < concat->num_subdev; i++) { struct mtd_info *subdev = concat->subdev[i]; size_t size, retsize; @@ -118,6 +120,8 @@ concat_write(struct mtd_info *mtd, loff_t to, size_t len, int err = -EINVAL; int i; + *retlen = 0; + for (i = 0; i < concat->num_subdev; i++) { struct mtd_info *subdev = concat->subdev[i]; size_t size, retsize; |