forked from Mirrors/btrfs-progs
btrfs-progs: check for open failure, don't close
Check for failure by testing for a negative file descriptor, not a descriptor of 0. And if it failed we have nothing to close(). Signed-off-by: Zach Brown <zab@redhat.com>master
parent
201238df26
commit
7c9f323a7c
|
@ -87,9 +87,8 @@ static int cmd_add_dev(int argc, char **argv)
|
|||
}
|
||||
|
||||
devfd = open(argv[i], O_RDWR);
|
||||
if (!devfd) {
|
||||
if (devfd < 0) {
|
||||
fprintf(stderr, "ERROR: Unable to open device '%s'\n", argv[i]);
|
||||
close(devfd);
|
||||
ret++;
|
||||
continue;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue