diff options
author | Julia Lawall <julia@diku.dk> | 2011-02-02 17:13:36 (GMT) |
---|---|---|
committer | David Woodhouse <David.Woodhouse@intel.com> | 2011-03-11 14:22:44 (GMT) |
commit | 3ee7451a023dd197fcfc1ba46dcce6d8ec5e1b78 (patch) | |
tree | b87527fb5eccede4c9236ecd1e521afa1ef02125 /drivers/mtd/maps/ts5500_flash.c | |
parent | f17f12ce9dd6ec0a8e1f415ecdbaebfce0207464 (diff) | |
download | linux-fsl-qoriq-3ee7451a023dd197fcfc1ba46dcce6d8ec5e1b78.tar.xz |
mtd: ts5500_flash: avoid calling map_destroy on NULL
map_destroy dereferences its argument. The call is furthermore only
reachable when this argument is NULL. Thus the call is dropped.
The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression *x;
@@
*if (x == NULL)
{ ...
* map_destroy(x);
...
return ...;
}
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'drivers/mtd/maps/ts5500_flash.c')
-rw-r--r-- | drivers/mtd/maps/ts5500_flash.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/mtd/maps/ts5500_flash.c b/drivers/mtd/maps/ts5500_flash.c index e2147bf..e02dfa9 100644 --- a/drivers/mtd/maps/ts5500_flash.c +++ b/drivers/mtd/maps/ts5500_flash.c @@ -94,7 +94,6 @@ static int __init init_ts5500_map(void) return 0; err1: - map_destroy(mymtd); iounmap(ts5500_map.virt); err2: return rc; |