]> bbs.cooldavid.org Git - net-next-2.6.git/commit
UBI: eliminate update of list_for_each_entry loop cursor
authorJulia Lawall <julia@diku.dk>
Sat, 7 Aug 2010 09:09:29 +0000 (11:09 +0200)
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Mon, 30 Aug 2010 07:19:11 +0000 (10:19 +0300)
commit01ebc12f5f2e88a1c6a5436b71a506ac2bf66d6b
tree7c3e748518efe1f42c82eb922d1d6625964282a2
parent2bfc96a127bc1cc94d26bfaa40159966064f9c8c
UBI: eliminate update of list_for_each_entry loop cursor

list_for_each_entry uses its first argument to move from one element to the
next, so modifying it can break the iteration.  The variable re1 is already
used within the loop as a temporary variable, and is not live here.

The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@r@
iterator name list_for_each_entry;
expression x,E;
position p1,p2;
@@

list_for_each_entry@p1(x,...) { <... x =@p2 E ...> }

@@
expression x,E;
position r.p1,r.p2;
statement S;
@@

*x =@p2 E
...
list_for_each_entry@p1(x,...) S
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
drivers/mtd/ubi/cdev.c