diff options
| author | Brian Norris <computersforpeace@gmail.com> | 2015-02-28 02:02:27 -0800 |
|---|---|---|
| committer | Brian Norris <computersforpeace@gmail.com> | 2015-03-11 15:21:48 -0700 |
| commit | b1a2348a1ac1380adf429035dbf3fdad0d5367bc (patch) | |
| tree | 4c572468b5ea194554e4eebeb9cd3ebd674d1dd6 /drivers | |
| parent | b9da8bae416efda5ad61c7c92edbb30de15ff7ee (diff) | |
| download | lwn-b1a2348a1ac1380adf429035dbf3fdad0d5367bc.tar.gz lwn-b1a2348a1ac1380adf429035dbf3fdad0d5367bc.zip | |
mtd: nand: fixup bounds checks for nand_{lock,unlock}()
Coverity noticed that these 'ret' assignments weren't being used. Let's
use them.
Note that nand_lock() and nand_unlock() are still not officially used by
any drivers.
Coverity CIDs #1227054 and #1227037
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/mtd/nand/nand_base.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index df7eb4ff07d1..5488a7ad63e7 100644 --- a/drivers/mtd/nand/nand_base.c +++ b/drivers/mtd/nand/nand_base.c @@ -968,7 +968,7 @@ int nand_unlock(struct mtd_info *mtd, loff_t ofs, uint64_t len) __func__, (unsigned long long)ofs, len); if (check_offs_len(mtd, ofs, len)) - ret = -EINVAL; + return -EINVAL; /* Align to last block address if size addresses end of the device */ if (ofs + len == mtd->size) @@ -1031,7 +1031,7 @@ int nand_lock(struct mtd_info *mtd, loff_t ofs, uint64_t len) __func__, (unsigned long long)ofs, len); if (check_offs_len(mtd, ofs, len)) - ret = -EINVAL; + return -EINVAL; nand_get_device(mtd, FL_LOCKING); |