mirror of
https://github.com/kdave/btrfs-progs
synced 2024-12-25 15:42:23 +00:00
btrfs-progs: fix close of error fd in scrub cancel
If we retry opening the mountpoint and fail, we'll call close on a filehandle w/ value -1. Rearrange so the retry uses the same open and same error handling. Signed-off-by: Eric Sandeen <sandeen@redhat.com>
This commit is contained in:
parent
67c631329a
commit
5eaeb577b5
13
cmds-scrub.c
13
cmds-scrub.c
@ -1448,13 +1448,13 @@ static int cmd_scrub_cancel(int argc, char **argv)
|
||||
|
||||
path = argv[1];
|
||||
|
||||
again:
|
||||
fdmnt = open_file_or_dir(path);
|
||||
if (fdmnt < 0) {
|
||||
fprintf(stderr, "ERROR: scrub cancel failed\n");
|
||||
return 12;
|
||||
return 1;
|
||||
}
|
||||
|
||||
again:
|
||||
ret = ioctl(fdmnt, BTRFS_IOC_SCRUB_CANCEL, NULL);
|
||||
err = errno;
|
||||
|
||||
@ -1463,13 +1463,10 @@ again:
|
||||
ret = check_mounted_where(fdmnt, path, mp, sizeof(mp),
|
||||
&fs_devices_mnt);
|
||||
if (ret) {
|
||||
/* It is a device; open the mountpoint. */
|
||||
/* It is a device; try again with the mountpoint. */
|
||||
close(fdmnt);
|
||||
fdmnt = open_file_or_dir(mp);
|
||||
if (fdmnt >= 0) {
|
||||
path = mp;
|
||||
goto again;
|
||||
}
|
||||
path = mp;
|
||||
goto again;
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user