diff options
| author | Rob Landley <rob@landley.net> | 2006-03-01 16:39:45 +0000 |
|---|---|---|
| committer | Rob Landley <rob@landley.net> | 2006-03-01 16:39:45 +0000 |
| commit | e7c43b66d74ee9902a6732122788a7a16bcfbf18 (patch) | |
| tree | ebb8583e9e1265588592614c0b4878daded44125 /e2fsprogs/ext2fs/test_io.c | |
| parent | 93f2286e6e59dab5eed14b5912a79254031c5a62 (diff) | |
| download | busybox-e7c43b66d74ee9902a6732122788a7a16bcfbf18.tar.gz | |
Cleanup patch from Denis Vlasenko. Mostly variants of removing the if(x)
from before "if(x) free(x)".
Diffstat (limited to 'e2fsprogs/ext2fs/test_io.c')
| -rw-r--r-- | e2fsprogs/ext2fs/test_io.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/e2fsprogs/ext2fs/test_io.c b/e2fsprogs/ext2fs/test_io.c index d4b36532f..45f9090cb 100644 --- a/e2fsprogs/ext2fs/test_io.c +++ b/e2fsprogs/ext2fs/test_io.c @@ -203,10 +203,8 @@ static errcode_t test_open(const char *name, int flags, io_channel *channel) return 0; cleanup: - if (io) - ext2fs_free_mem(&io); - if (data) - ext2fs_free_mem(&data); + ext2fs_free_mem(&io); + ext2fs_free_mem(&data); return retval; } @@ -229,8 +227,7 @@ static errcode_t test_close(io_channel channel) fclose(data->outfile); ext2fs_free_mem(&channel->private_data); - if (channel->name) - ext2fs_free_mem(&channel->name); + ext2fs_free_mem(&channel->name); ext2fs_free_mem(&channel); return retval; } |