diff options
author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2020-06-02 10:54:12 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2020-06-03 10:03:34 +0200 |
commit | aa2cd910c7465eaf6de04da1b4d110205ad63c77 (patch) | |
tree | 3b469e89caeb4e7e595d524c7609e27b01e93a69 /drivers | |
parent | 90f96b39cd851df684ec8d6a0c0d508c2503003f (diff) | |
download | barebox-aa2cd910c7465eaf6de04da1b4d110205ad63c77.tar.gz barebox-aa2cd910c7465eaf6de04da1b4d110205ad63c77.tar.xz |
mtd: nand: base: fix use of uninitialized struct member
ooblen is read in nand_do_read_ops, despite never having been assigned a
value. Because ooblen is used to indicate how many bytes of oobbuf are
usable and oobbuf is NULL, use 0 as initial value.
Found on ARCH=sandbox using clang-analyzer.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/mtd/nand/nand_base.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index 00f0f75884..3f4c787f49 100644 --- a/drivers/mtd/nand/nand_base.c +++ b/drivers/mtd/nand/nand_base.c @@ -1688,6 +1688,7 @@ static int nand_read(struct mtd_info *mtd, loff_t from, size_t len, nand_get_device(mtd, FL_READING); ops.len = len; ops.datbuf = buf; + ops.ooblen = 0; ops.oobbuf = NULL; ops.mode = MTD_OPS_PLACE_OOB; ret = nand_do_read_ops(mtd, from, &ops); |