| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 1 | /// Use memdup_user rather than duplicating its implementation | 
| Nicolas Palix | 0341745 | 2010-08-24 17:39:07 +0200 | [diff] [blame] | 2 | /// This is a little bit restricted to reduce false positives | 
 | 3 | /// | 
 | 4 | // Confidence: High | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 5 | // Copyright: (C) 2010-2012 Nicolas Palix.  GPLv2. | 
 | 6 | // Copyright: (C) 2010-2012 Julia Lawall, INRIA/LIP6.  GPLv2. | 
 | 7 | // Copyright: (C) 2010-2012 Gilles Muller, INRIA/LiP6.  GPLv2. | 
| Nicolas Palix | 0341745 | 2010-08-24 17:39:07 +0200 | [diff] [blame] | 8 | // URL: http://coccinelle.lip6.fr/ | 
 | 9 | // Comments: | 
 | 10 | // Options: -no_includes -include_headers | 
 | 11 |  | 
 | 12 | virtual patch | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 13 | virtual context | 
 | 14 | virtual org | 
 | 15 | virtual report | 
| Nicolas Palix | 0341745 | 2010-08-24 17:39:07 +0200 | [diff] [blame] | 16 |  | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 17 | @depends on patch@ | 
| Nicolas Palix | 0341745 | 2010-08-24 17:39:07 +0200 | [diff] [blame] | 18 | expression from,to,size,flag; | 
| Nicolas Palix | 0341745 | 2010-08-24 17:39:07 +0200 | [diff] [blame] | 19 | identifier l1,l2; | 
 | 20 | @@ | 
 | 21 |  | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 22 | -  to = \(kmalloc\|kzalloc\)(size,flag); | 
| Nicolas Palix | 0341745 | 2010-08-24 17:39:07 +0200 | [diff] [blame] | 23 | +  to = memdup_user(from,size); | 
 | 24 |    if ( | 
 | 25 | -      to==NULL | 
 | 26 | +      IS_ERR(to) | 
 | 27 |                  || ...) { | 
 | 28 |    <+... when != goto l1; | 
 | 29 | -  -ENOMEM | 
 | 30 | +  PTR_ERR(to) | 
 | 31 |    ...+> | 
 | 32 |    } | 
 | 33 | -  if (copy_from_user(to, from, size) != 0) { | 
 | 34 | -    <+... when != goto l2; | 
 | 35 | -    -EFAULT | 
 | 36 | -    ...+> | 
 | 37 | -  } | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 38 |  | 
 | 39 | @r depends on !patch@ | 
 | 40 | expression from,to,size,flag; | 
 | 41 | position p; | 
 | 42 | statement S1,S2; | 
 | 43 | @@ | 
 | 44 |  | 
 | 45 | *  to = \(kmalloc@p\|kzalloc@p\)(size,flag); | 
 | 46 |    if (to==NULL || ...) S1 | 
 | 47 |    if (copy_from_user(to, from, size) != 0) | 
 | 48 |    S2 | 
 | 49 |  | 
 | 50 | @script:python depends on org@ | 
 | 51 | p << r.p; | 
 | 52 | @@ | 
 | 53 |  | 
 | 54 | coccilib.org.print_todo(p[0], "WARNING opportunity for memdep_user") | 
 | 55 |  | 
 | 56 | @script:python depends on report@ | 
 | 57 | p << r.p; | 
 | 58 | @@ | 
 | 59 |  | 
 | 60 | coccilib.report.print_report(p[0], "WARNING opportunity for memdep_user") |