Re: [ATM] [UPDATE] unbalanced exit path in Forerunner HE he_init_one() (and an iphase patch too!)

Christoph Hellwig (hch@infradead.org)
Sat, 10 May 2003 15:43:46 +0100


On Sat, May 10, 2003 at 09:52:49AM -0400, chas williams wrote:
> class/usb-midi.c: if ( u ) kfree(u);
> class/usblp.c: if (usblp->statusbuf) kfree(usblp->statusbuf);
> class/usblp.c: if (usblp->device_id_string) kfree(usblp->device_id_string);
> image/mdc800.c:#define try_free_mem(A) if (A != 0) { kfree (A); A=0; }

It's wrong there, too. usb is not exactly the right example if you look
for code that matches the style guidelines..
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/