Messages in this thread Patch in this message | | | Date | Mon, 3 Sep 2012 18:09:53 +0800 | Subject | [PATCH] parport: fix possible memory leak in parport_gsc_probe_port() | From | Wei Yongjun <> |
| |
From: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
ops has been allocated in this function and should be freed before leaving from the error handling cases.
spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/)
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> --- drivers/parport/parport_gsc.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/parport/parport_gsc.c b/drivers/parport/parport_gsc.c index 5d6de38..352f961 100644 --- a/drivers/parport/parport_gsc.c +++ b/drivers/parport/parport_gsc.c @@ -271,6 +271,7 @@ struct parport *__devinit parport_gsc_probe_port (unsigned long base, if (!parport_SPP_supported (p)) { /* No port. */ kfree (priv); + kfree(ops); return NULL; } parport_PS2_supported (p);
| |