btrfs-progs: make get_btrfs_mount callable
get_btrfs_mount is reusable function but it is printing errors, this removes it. Here the parent function of open_path_or_dev_mnt does print error msg on error. Signed-off-by: Anand Jain <anand.jain@oracle.com> Signed-off-by: Chris Mason <chris.mason@fusionio.com>
This commit is contained in:
parent
f3fb002051
commit
811e993db4
6
utils.c
6
utils.c
|
@ -692,7 +692,8 @@ int is_block_device(const char *path) {
|
|||
* On failure, returns -errno (not mounted yields -EINVAL)
|
||||
* Is noisy on failures, expects to be given a mounted device.
|
||||
*/
|
||||
static int get_btrfs_mount(const char *dev, char *mp, size_t mp_size) {
|
||||
int get_btrfs_mount(const char *dev, char *mp, size_t mp_size)
|
||||
{
|
||||
int ret;
|
||||
int fd = -1;
|
||||
|
||||
|
@ -717,7 +718,6 @@ static int get_btrfs_mount(const char *dev, char *mp, size_t mp_size) {
|
|||
|
||||
ret = check_mounted_where(fd, dev, mp, mp_size, NULL);
|
||||
if (!ret) {
|
||||
fprintf(stderr, "%s is not a mounted btrfs device\n", dev);
|
||||
ret = -EINVAL;
|
||||
} else { /* mounted, all good */
|
||||
ret = 0;
|
||||
|
@ -725,8 +725,6 @@ static int get_btrfs_mount(const char *dev, char *mp, size_t mp_size) {
|
|||
out:
|
||||
if (fd != -1)
|
||||
close(fd);
|
||||
if (ret)
|
||||
fprintf(stderr, "Could not get mountpoint for %s\n", dev);
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue