Messages in this thread | | | Date | Mon, 9 Jun 2008 23:10:37 +1000 | From | Stephen Rothwell <> | Subject | Re: [PATCH 4/4] drivers/macintosh/therm_adt746x.c: add missing of_node_put |
| |
On Mon, 9 Jun 2008 14:21:51 +0200 (CEST) Julia Lawall <julia@diku.dk> wrote: > > From: Julia Lawall <julia@diku.dk> > > of_node_put is needed before discarding a value received from > of_find_node_by_name, eg in error handling code. > > The semantic patch that makes the change is as follows: > (http://www.emn.fr/x-info/coccinelle/) > > // <smpl> > @@ > struct device_node *n; > struct device_node *n1; > statement S; > identifier f; > expression E; > constant C; > @@ > > n = of_find_node_by_name(...) > ... > if (!n) S > ... when != of_node_put(n) > when != n1 = f(n,...) > when != E = n > when any > when strict > ( > + of_node_put(n); > return -C; > | > of_node_put(n); > | > n1 = f(n,...) > | > E = n > | > return ...; > ) > // </smpl> > > Signed-off-by: Julia Lawall <julia@diku.dk>
Acked-by: Stephen Rothwell <sfr@canb.auug.org.au>
-- Cheers, Stephen Rothwell sfr@canb.auug.org.au http://www.canb.auug.org.au/~sfr/ [unhandled content-type:application/pgp-signature] | |