On 09/12, Venu Byravarasu wrote: > > > --- a/fs/binfmt_elf.c > > +++ b/fs/binfmt_elf.c > > @@ -1695,30 +1695,19 @@ static int elf_note_info_init(struct elf_note_info > > *info) > > return 0; > > info->psinfo = kmalloc(sizeof(*info->psinfo), GFP_KERNEL); > > Why don't you change kmalloc to devm_kzalloc, so that free can be ignored > altogether.
kzalloc(), I guess... Still I can't understand what did you mean. If kmalloc() fails we return "fail" and ->psinfo = NULL, the subsequent kfree(NULL) is nop. Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
