Re: [CHECKER] 32 Memory Leaks on Error Paths

From: David Howells
Date: Wed Sep 24 2003 - 02:12:18 EST



> * David Yu Chen (dychen@xxxxxxxxxxxx) wrote:
> > [FILE: 2.6.0-test5/fs/afs/cell.c]
> > START -->
> > 58: cell = kmalloc(sizeof(afs_cell_t) + strlen(name) + 1,GFP_KERNEL);
> > 59: if (!cell) {
> <snip>
> > 126: error:
> > 127: up_write(&afs_cells_sem);
> > 128: kfree(afs_cell_root);
> > END -->
>
> Yes, this looks like a bug/typo. Patch below. David, this look ok?

Yes. I've applied it to my local development version.

Thanks,
David
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/