diff options
author | Eric Sandeen <sandeen@redhat.com> | 2013-03-04 16:39:52 -0600 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2013-03-10 15:48:47 +0100 |
commit | 9e769c4864234a22d2e01a0a1cf7d3bac2db8b5c (patch) | |
tree | 8f6ec915f21072b93d1a109acf8ec99489d7859e /convert.c | |
parent | 0b9e200099c83e92e49557aef8348301f225052f (diff) |
btrfs-progs: close fd on do_convert error returns
stops an fd leak that Coverity found.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Diffstat (limited to 'convert.c')
-rw-r--r-- | convert.c | 5 |
1 files changed, 4 insertions, 1 deletions
@@ -2277,7 +2277,8 @@ err: int do_convert(const char *devname, int datacsum, int packing, int noxattr) { - int i, fd, ret; + int i, ret; + int fd = -1; u32 blocksize; u64 blocks[7]; u64 total_bytes; @@ -2407,6 +2408,8 @@ int do_convert(const char *devname, int datacsum, int packing, int noxattr) printf("conversion complete.\n"); return 0; fail: + if (fd != -1) + close(fd); fprintf(stderr, "conversion aborted.\n"); return -1; } |