| commit | 7c19c1e80d348a5c79b65f9fc3d54bbbeea9cd0f | |
| author | Julia Lawall <julia@diku.dk> | |
| Fri, 28 Oct 2011 23:58:13 +0000 (29 01:58 +0200) | ||
| committer | Greg Kroah-Hartman <gregkh@suse.de> | |
| Sun, 27 Nov 2011 01:19:23 +0000 (26 17:19 -0800) | ||
| tree | 7849154a8e2bfb6c6b337643a9b65b6e6dd0db85 | treesnapshot (tar.gz zip) |
| parent | 7990b0d7ec449b10da383bab191eb487185aadce | commitdiff |
drivers/staging/rtl8712/rtl871x_mlme.c: eliminate a null pointer dereference
If ibss_wlan is NULL, it is not correct to memcpy into its field.
The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@r@
expression E, E1;
identifier f;
statement S1,S2,S3;
@@
if (E == NULL)
{
... when != if (E == NULL || ...) S1 else S2
when != E = E1
*E->f
... when any
return ...;
}
else S3
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
If ibss_wlan is NULL, it is not correct to memcpy into its field.
The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@r@
expression E, E1;
identifier f;
statement S1,S2,S3;
@@
if (E == NULL)
{
... when != if (E == NULL || ...) S1 else S2
when != E = E1
*E->f
... when any
return ...;
}
else S3
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| drivers/staging/rtl8712/rtl871x_mlme.c | diffblobblamehistory |