From: Andrew Morton free_fdset(NULL, ...) is legal. Signed-off-by: Andrew Morton --- fs/file.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff -puN fs/file.c~alloc_fdtable-cleanup fs/file.c --- a/fs/file.c~alloc_fdtable-cleanup +++ a/fs/file.c @@ -281,10 +281,8 @@ static struct fdtable *alloc_fdtable(int out2: nfds = fdt->max_fdset; out: - if (new_openset) - free_fdset(new_openset, nfds); - if (new_execset) - free_fdset(new_execset, nfds); + free_fdset(new_openset, nfds); + free_fdset(new_execset, nfds); kfree(fdt); return NULL; } _