6851ba1a1b
This just needs to find any reassignment of the loop iterator, and doesn't need such a thing on all execution paths, so use exists on the first rule. Signed-off-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
63 lines
1.4 KiB
Plaintext
63 lines
1.4 KiB
Plaintext
/// list_for_each_entry uses its first argument to get from one element of
|
|
/// the list to the next, so it is usually not a good idea to reassign it.
|
|
/// The first rule finds such a reassignment and the second rule checks
|
|
/// that there is a path from the reassignment back to the top of the loop.
|
|
///
|
|
// Confidence: High
|
|
// Copyright: (C) 2010 Nicolas Palix, DIKU. GPLv2.
|
|
// Copyright: (C) 2010 Julia Lawall, DIKU. GPLv2.
|
|
// Copyright: (C) 2010 Gilles Muller, INRIA/LiP6. GPLv2.
|
|
// URL: http://coccinelle.lip6.fr/
|
|
// Comments:
|
|
// Options: --no-includes --include-headers
|
|
|
|
virtual context
|
|
virtual org
|
|
virtual report
|
|
|
|
@r exists@
|
|
iterator name list_for_each_entry;
|
|
expression x,E;
|
|
position p1,p2;
|
|
@@
|
|
|
|
list_for_each_entry@p1(x,...) { <... x =@p2 E ...> }
|
|
|
|
@depends on context && !org && !report@
|
|
expression x,E;
|
|
position r.p1,r.p2;
|
|
statement S;
|
|
@@
|
|
|
|
*x =@p2 E
|
|
...
|
|
list_for_each_entry@p1(x,...) S
|
|
|
|
// ------------------------------------------------------------------------
|
|
|
|
@back depends on (org || report) && !context exists@
|
|
expression x,E;
|
|
position r.p1,r.p2;
|
|
statement S;
|
|
@@
|
|
|
|
x =@p2 E
|
|
...
|
|
list_for_each_entry@p1(x,...) S
|
|
|
|
@script:python depends on back && org@
|
|
p1 << r.p1;
|
|
p2 << r.p2;
|
|
@@
|
|
|
|
cocci.print_main("iterator",p1)
|
|
cocci.print_secs("update",p2)
|
|
|
|
@script:python depends on back && report@
|
|
p1 << r.p1;
|
|
p2 << r.p2;
|
|
@@
|
|
|
|
msg = "iterator with update on line %s" % (p2[0].line)
|
|
coccilib.report.print_report(p1[0],msg)
|