diff options
| author |   Julia Lawall <julia@diku.dk> | 2011-12-26 18:38:01 +0100 |
|---|---|---|
| committer |   David Woodhouse <David.Woodhouse@intel.com> | 2012-01-09 18:24:05 +0000 |
| commit | af32b36095147e1828496b58987c1e24ef40487d (patch) | |
| tree | 377f57e77ef19d5b4dd206c343259cc8a5e6a207 /crypto/api.c | |
| parent | mtd: gpio-nand: add device tree bindings (diff) | |
| download | wireguard-linux-af32b36095147e1828496b58987c1e24ef40487d.tar.xz wireguard-linux-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 <julia@diku.dk>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Diffstat (limited to 'crypto/api.c')
0 files changed, 0 insertions, 0 deletions