Thomas Gleixner | 7f904d7 | 2019-06-04 10:11:38 +0200 | [diff] [blame] | 1 | // SPDX-License-Identifier: GPL-2.0-only |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 2 | /// Find &&/|| operations that include the same argument more than once |
Julia Lawall | 09d4d96 | 2018-08-03 18:32:11 +0200 | [diff] [blame] | 3 | //# A common source of false positives is when the expression, or |
| 4 | //# another expresssion in the same && or || operation, performs a |
| 5 | //# side effect. |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 6 | /// |
| 7 | // Confidence: Moderate |
Thomas Gleixner | 7f904d7 | 2019-06-04 10:11:38 +0200 | [diff] [blame] | 8 | // Copyright: (C) 2010 Nicolas Palix, DIKU. |
| 9 | // Copyright: (C) 2010 Julia Lawall, DIKU. |
| 10 | // Copyright: (C) 2010 Gilles Muller, INRIA/LiP6. |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 11 | // URL: http://coccinelle.lip6.fr/ |
| 12 | // Comments: |
Nicolas Palix | 93f1446 | 2013-06-20 13:10:56 +0200 | [diff] [blame] | 13 | // Options: --no-includes --include-headers |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 14 | |
| 15 | virtual context |
| 16 | virtual org |
| 17 | virtual report |
| 18 | |
| 19 | @r expression@ |
| 20 | expression E; |
| 21 | position p; |
| 22 | @@ |
| 23 | |
| 24 | ( |
Julia Lawall | 09d4d96 | 2018-08-03 18:32:11 +0200 | [diff] [blame] | 25 | E@p || ... || E |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 26 | | |
Julia Lawall | 09d4d96 | 2018-08-03 18:32:11 +0200 | [diff] [blame] | 27 | E@p && ... && E |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 28 | ) |
| 29 | |
Julia Lawall | 09d4d96 | 2018-08-03 18:32:11 +0200 | [diff] [blame] | 30 | @bad@ |
| 31 | expression r.E,e1,e2,fn; |
| 32 | position r.p; |
| 33 | assignment operator op; |
| 34 | @@ |
| 35 | |
| 36 | ( |
| 37 | E@p |
| 38 | & |
| 39 | <+... \(fn(...)\|e1 op e2\|e1++\|e1--\|++e1\|--e1\) ...+> |
| 40 | ) |
| 41 | |
| 42 | @depends on context && !bad@ |
| 43 | expression r.E; |
| 44 | position r.p; |
| 45 | @@ |
| 46 | |
| 47 | *E@p |
| 48 | |
| 49 | @script:python depends on org && !bad@ |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 50 | p << r.p; |
| 51 | @@ |
| 52 | |
| 53 | cocci.print_main("duplicated argument to && or ||",p) |
| 54 | |
Julia Lawall | 09d4d96 | 2018-08-03 18:32:11 +0200 | [diff] [blame] | 55 | @script:python depends on report && !bad@ |
Julia Lawall | 97c1cf8 | 2010-10-24 23:37:32 +0200 | [diff] [blame] | 56 | p << r.p; |
| 57 | @@ |
| 58 | |
| 59 | coccilib.report.print_report(p[0],"duplicated argument to && or ||") |