mirror of
https://github.com/kdave/btrfs-progs
synced 2025-04-01 14:38:08 +00:00
btrfs-progs: Remove fprintf() in find_mount_root().
find_mount_root() function in utils.c should not print error string. Caller should be responsible to print error string. This patch will remove the only fprintf in find_mount_root() and modify the caller a little to use strerror() to prompt users. Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com> Signed-off-by: David Sterba <dsterba@suse.cz>
This commit is contained in:
parent
b2d0e10d49
commit
10c8f34f51
@ -842,9 +842,10 @@ static int do_receive(struct btrfs_receive *r, const char *tomnt, int r_fd,
|
||||
|
||||
ret = find_mount_root(dest_dir_full_path, &r->root_path);
|
||||
if (ret < 0) {
|
||||
fprintf(stderr,
|
||||
"ERROR: failed to determine mount point for %s: %s\n",
|
||||
dest_dir_full_path, strerror(-ret));
|
||||
ret = -EINVAL;
|
||||
fprintf(stderr, "ERROR: failed to determine mount point "
|
||||
"for %s\n", dest_dir_full_path);
|
||||
goto out;
|
||||
}
|
||||
r->mnt_fd = open(r->root_path, O_RDONLY | O_NOATIME);
|
||||
|
@ -349,9 +349,10 @@ static int init_root_path(struct btrfs_send *s, const char *subvol)
|
||||
|
||||
ret = find_mount_root(subvol, &s->root_path);
|
||||
if (ret < 0) {
|
||||
fprintf(stderr,
|
||||
"ERROR: failed to determine mount point for %s: %s\n",
|
||||
subvol, strerror(-ret));
|
||||
ret = -EINVAL;
|
||||
fprintf(stderr, "ERROR: failed to determine mount point "
|
||||
"for %s\n", subvol);
|
||||
goto out;
|
||||
}
|
||||
|
||||
|
6
utils.c
6
utils.c
@ -2356,12 +2356,8 @@ int find_mount_root(const char *path, char **mount_root)
|
||||
}
|
||||
endmntent(mnttab);
|
||||
|
||||
if (!longest_match) {
|
||||
fprintf(stderr,
|
||||
"ERROR: Failed to find mount root for path %s.\n",
|
||||
path);
|
||||
if (!longest_match)
|
||||
return -ENOENT;
|
||||
}
|
||||
|
||||
ret = 0;
|
||||
*mount_root = realpath(longest_match, NULL);
|
||||
|
Loading…
Reference in New Issue
Block a user