Re: [PATCH] drivers/media: Removes useless kfree()

2012-09-12 Thread Dan Carpenter
On Tue, Sep 11, 2012 at 08:00:32PM +0200, Peter Senna Tschudin wrote: diff --git a/drivers/media/dvb-frontends/lg2160.c b/drivers/media/dvb-frontends/lg2160.c index cc11260..748da5d 100644 --- a/drivers/media/dvb-frontends/lg2160.c +++ b/drivers/media/dvb-frontends/lg2160.c @@ -1451,7

Re: [PATCH] drivers/media: Removes useless kfree()

2012-09-12 Thread Peter Senna Tschudin
I'll split this patch in one patch for file and resend. On Wed, Sep 12, 2012 at 10:06 AM, Dan Carpenter dan.carpen...@oracle.com wrote: On Tue, Sep 11, 2012 at 08:00:32PM +0200, Peter Senna Tschudin wrote: diff --git a/drivers/media/dvb-frontends/lg2160.c

[PATCH] drivers/media: Removes useless kfree()

2012-09-11 Thread Peter Senna Tschudin
From: Peter Senna Tschudin peter.se...@gmail.com The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ... return ...; } @unchanged exists@ position r.p1,r.p2; expression e

Re: [PATCH] drivers/media: Removes useless kfree()

2012-09-11 Thread Marcos Souza
2012/9/11 Peter Senna Tschudin peter.se...@gmail.com: From: Peter Senna Tschudin peter.se...@gmail.com The semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // smpl @r exists@ position p1,p2; expression x; @@ if (x@p1 == NULL) { ... kfree@p2(x); ...