diff options
| author | David Sterba <dsterba@suse.cz> | 2014-12-19 16:15:39 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.cz> | 2014-12-19 18:04:21 +0100 |
| commit | db0981936c945af4b3089aa8ed9981c075dca721 (patch) | |
| tree | 9162671c1cb47e143802d771f8d2680a26161329 | |
| parent | fbd8e04f4b83d75079973ce24f48593d873f2166 (diff) | |
btrfs-progs: fragments, close output file on error
Resolves-Coverity-CID: 1258794
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
| -rw-r--r-- | btrfs-fragments.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/btrfs-fragments.c b/btrfs-fragments.c index d03c2c3e..360f10f8 100644 --- a/btrfs-fragments.c +++ b/btrfs-fragments.c @@ -233,7 +233,7 @@ list_fragments(int fd, u64 flags, char *dir) ret = ioctl(fd, BTRFS_IOC_TREE_SEARCH, &args); if (ret < 0) { fprintf(stderr, "ERROR: can't perform the search\n"); - return ret; + goto out_close; } /* the ioctl returns the number of item it found in nr_items */ if (sk->nr_items == 0) @@ -373,7 +373,10 @@ skip:; fprintf(html, "</p>"); } fprintf(html, "</body></html>\n"); - + +out_close: + fclose(html); + return ret; } |