diff options
| author | Andrew Morton <akpm@osdl.org> | 2006-09-27 04:51:02 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-27 11:26:19 -0400 |
| commit | 8b0e330b7720a206339887044fa275bf537a5264 (patch) | |
| tree | f76b1e82f09988aef047a1d8b2636bee93eab92e | |
| parent | 07563c711fbc25389e58ab9c9f0b9de2fce56760 (diff) | |
[PATCH] alloc_fdtable() cleanup
free_fdset(NULL, ...) is legal.
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | fs/file.c | 6 |
1 files changed, 2 insertions, 4 deletions
| @@ -281,10 +281,8 @@ static struct fdtable *alloc_fdtable(int nr) | |||
| 281 | out2: | 281 | out2: |
| 282 | nfds = fdt->max_fdset; | 282 | nfds = fdt->max_fdset; |
| 283 | out: | 283 | out: |
| 284 | if (new_openset) | 284 | free_fdset(new_openset, nfds); |
| 285 | free_fdset(new_openset, nfds); | 285 | free_fdset(new_execset, nfds); |
| 286 | if (new_execset) | ||
| 287 | free_fdset(new_execset, nfds); | ||
| 288 | kfree(fdt); | 286 | kfree(fdt); |
| 289 | return NULL; | 287 | return NULL; |
| 290 | } | 288 | } |
