Re: [PATCH 10/14] drivers/ata/libata: Move dereference after NULLtest

From: Jeff Garzik
Date: Tue Nov 03 2009 - 16:18:03 EST


On 10/17/2009 02:41 AM, Julia Lawall wrote:
From: Julia Lawall<julia@xxxxxxx>

In each case, if the NULL test on qc is needed, then the derefernce
should be after the NULL test.

A simplified version of the semantic match that detects this problem is as
follows (http://coccinelle.lip6.fr/):

//<smpl>
@match exists@
expression x, E;
identifier fld;
@@

* x->fld
... when != \(x = E\|&x\)
* x == NULL
//</smpl>

Signed-off-by: Julia Lawall<julia@xxxxxxx>

applied


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/