diff options
| author | Julia Lawall <[email protected]> | 2011-12-26 17:38:01 +0000 |
|---|---|---|
| committer | David Woodhouse <[email protected]> | 2012-01-09 18:24:05 +0000 |
| commit | af32b36095147e1828496b58987c1e24ef40487d (patch) | |
| tree | 377f57e77ef19d5b4dd206c343259cc8a5e6a207 /drivers/mtd/lpddr/lpddr_cmds.c | |
| parent | mtd: gpio-nand: add device tree bindings (diff) | |
| download | kernel-af32b36095147e1828496b58987c1e24ef40487d.tar.gz kernel-af32b36095147e1828496b58987c1e24ef40487d.zip | |
mtd: lantiq-flash: drop iounmap for devm_ allocated data
Data allocated with devm_ioremap or devm_ioremap_nocache should not be
freed using iounmap, because doing so causes a dangling pointer, and a
subsequent double free.
The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@r@
expression x;
@@
(
x = devm_ioremap(...)
|
x = devm_ioremap_nocache(...)
)
@@
expression r.x;
@@
* iounmap(x)
// </smpl>
Signed-off-by: Julia Lawall <[email protected]>
Signed-off-by: Artem Bityutskiy <[email protected]>
Signed-off-by: David Woodhouse <[email protected]>
Diffstat (limited to 'drivers/mtd/lpddr/lpddr_cmds.c')
0 files changed, 0 insertions, 0 deletions