diff options
author | Anand Jain <anand.jain@oracle.com> | 2015-06-01 14:25:18 +0800 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2015-06-02 17:15:00 +0200 |
commit | d2d22a4f7bbf3dda752c418468a2c95a6ea11dbe (patch) | |
tree | 166f47468c335adab6076eea43d35bd8a048db08 /cmds-replace.c | |
parent | fbb356632c19c9262231e2062d2f708574f794ad (diff) |
btrfs-progs: use function is_block_device() instead
Here the delete code as below, is trying to check if the provided device
is a block device, there is a function for it. Use it.
Signed-off-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.cz>
Diffstat (limited to 'cmds-replace.c')
-rw-r--r-- | cmds-replace.c | 25 |
1 files changed, 1 insertions, 24 deletions
diff --git a/cmds-replace.c b/cmds-replace.c index 63d34f94..3a9485f8 100644 --- a/cmds-replace.c +++ b/cmds-replace.c @@ -132,14 +132,12 @@ static int cmd_start_replace(int argc, char **argv) int i; int c; int fdmnt = -1; - int fdsrcdev = -1; int fddstdev = -1; char *path; char *srcdev; char *dstdev = NULL; int avoid_reading_from_srcdev = 0; int force_using_targetdev = 0; - struct stat st; u64 dstdev_block_count; int do_not_background = 0; int mixed = 0; @@ -252,28 +250,9 @@ static int cmd_start_replace(int argc, char **argv) srcdev, path); goto leave_with_error; } - } else { - fdsrcdev = open(srcdev, O_RDWR); - if (fdsrcdev < 0) { - fprintf(stderr, "Error: Unable to open device '%s'\n", - srcdev); - fprintf(stderr, "\tTry using the devid instead of the path\n"); - goto leave_with_error; - } - ret = fstat(fdsrcdev, &st); - if (ret) { - fprintf(stderr, "Error: Unable to stat '%s'\n", srcdev); - goto leave_with_error; - } - if (!S_ISBLK(st.st_mode)) { - fprintf(stderr, "Error: '%s' is not a block device\n", - srcdev); - goto leave_with_error; - } + } else if (is_block_device(srcdev)) { strncpy((char *)start_args.start.srcdev_name, srcdev, BTRFS_DEVICE_PATH_NAME_MAX); - close(fdsrcdev); - fdsrcdev = -1; start_args.start.srcdevid = 0; } @@ -346,8 +325,6 @@ leave_with_error: free(dstdev); if (fdmnt != -1) close(fdmnt); - if (fdsrcdev != -1) - close(fdsrcdev); if (fddstdev != -1) close(fddstdev); return 1; |