parport: fix possible memory leak in parport_gsc_probe_port()

Author: 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 
Signed-off-by: Greg Kroah-Hartman

---
 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);

parport: fix possible memory leak in parport_gsc_probe_port()

qlambert

Archives

Posts by Subject