Re: [Patch 5/9] fs/binfmt_em86.c: fix resource leaks

Al Viro
2008-05-10T19:25:29Z

On Thu, May 08, 2008 at 09:52:30PM +0800, WANG Cong wrote:
> ->load_binary()'s caller only frees resources which ->load_binary() applied
> when it succeeded, so ->load_binary() itself should free the resources on
> failure.

No, it should not. do_execve() will free them just fine.