md.c:didn't unlock the mddev before return EINVAL in array_size_store
authorZhilong Liu <zlliu@suse.com>
Mon, 10 Apr 2017 06:15:55 +0000 (14:15 +0800)
committerShaohua Li <shli@fb.com>
Mon, 10 Apr 2017 17:50:24 +0000 (10:50 -0700)
md.c: it needs to release the mddev lock before
the array_size_store() returns.

Fixes: ab5a98b132fd ("md-cluster: change array_sectors and update size are not supported")

Signed-off-by: Zhilong Liu <zlliu@suse.com>
Reviewed-by: Guoqing Jiang <gqjiang@suse.com>
Signed-off-by: Shaohua Li <shli@fb.com>
drivers/md/md.c

index a612b9f83069bdb3c0c390ece95828a5698191be..5022be1833903f753e58ba6c0ddc22d0abd3bf9a 100644 (file)
@@ -4950,8 +4950,10 @@ array_size_store(struct mddev *mddev, const char *buf, size_t len)
                return err;
 
        /* cluster raid doesn't support change array_sectors */
-       if (mddev_is_clustered(mddev))
+       if (mddev_is_clustered(mddev)) {
+               mddev_unlock(mddev);
                return -EINVAL;
+       }
 
        if (strncmp(buf, "default", 7) == 0) {
                if (mddev->pers)