diff options
author | Brian Norris <computersforpeace@gmail.com> | 2014-07-21 19:06:39 -0700 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2014-08-19 11:53:07 -0700 |
commit | f7f0d358f5f2f1133b5a14337028ddab848cd74e (patch) | |
tree | 2aa983dc7a894d0801f7d9b9bf115f48768ccd85 /drivers | |
parent | 0c2b4e21444d0e274e91fc7db85caddb30988853 (diff) | |
download | lwn-f7f0d358f5f2f1133b5a14337028ddab848cd74e.tar.gz lwn-f7f0d358f5f2f1133b5a14337028ddab848cd74e.zip |
mtd: sm_ftl: initialize error code
There is one theoretical case that could fall through to using an
uninitialized value as the return code. Let's give it a value of 0.
Untested.
Caught by Coverity.
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/mtd/sm_ftl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mtd/sm_ftl.c b/drivers/mtd/sm_ftl.c index cf49c22673b9..c23184a47fc4 100644 --- a/drivers/mtd/sm_ftl.c +++ b/drivers/mtd/sm_ftl.c @@ -1058,7 +1058,7 @@ static int sm_write(struct mtd_blktrans_dev *dev, { struct sm_ftl *ftl = dev->priv; struct ftl_zone *zone; - int error, zone_num, block, boffset; + int error = 0, zone_num, block, boffset; BUG_ON(ftl->readonly); sm_break_offset(ftl, sec_no << 9, &zone_num, &block, &boffset); |