parport: fix possible memory leak in parport_gsc_probe_port()
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Mon, 3 Sep 2012 10:09:53 +0000 (18:09 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 5 Sep 2012 23:34:22 +0000 (16:34 -0700)
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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/parport/parport_gsc.c

index 5d6de380e42ba87526cdfa2b87f658431223be86..352f96180bc71f848464b6f9a9a5ad914d92004d 100644 (file)
@@ -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);