btrfs-progs: remove uneccessary subvol name check for subvol deletion

When we want to delete a subvol, we first check to see whether it is
a subvolume or not. After the check, we are sure that it is a valid
subvol, don't have to check its name.

Signed-off-by: Gui Hecheng <guihc.fnst@cn.fujitsu.com>
[removed unused variable 'len']
Signed-off-by: David Sterba <dsterba@suse.cz>

diff --git a/cmds-subvolume.c b/cmds-subvolume.c
index 9618cae..3895d11 100644 (file)
--- a/cmds-subvolume.c
+++ b/cmds-subvolume.c
@@ -208,7 +208,7 @@ static const char * const cmd_subvol_delete_usage[] = {
 
 static int cmd_subvol_delete(int argc, char **argv)
 {
-       int     res, len, e, ret = 0;
+       int     res, e, ret = 0;
        int cnt;
        int fd = -1;
        struct btrfs_ioctl_vol_args     args;
@@ -287,21 +287,6 @@ again:
        vname = basename(dupvname);
        free(cpath);
 
-       if (!test_issubvolname(vname)) {
-               fprintf(stderr, "ERROR: incorrect subvolume name '%s'\n",
-                       vname);
-               ret = 1;
-               goto out;
-       }
-
-       len = strlen(vname);
-       if (len == 0 || len >= BTRFS_VOL_NAME_MAX) {
-               fprintf(stderr, "ERROR: snapshot name too long '%s'\n",
-                       vname);
-               ret = 1;
-               goto out;
-       }
-
        fd = open_file_or_dir(dname, &dirstream);
        if (fd < 0) {
                fprintf(stderr, "ERROR: can't access '%s'\n", dname);