diff options
author | Zhilong Liu <zlliu@suse.com> | 2017-04-10 02:15:55 -0400 |
---|---|---|
committer | Shaohua Li <shli@fb.com> | 2017-04-10 13:50:24 -0400 |
commit | b670883bb9e55ba63a278d83e034faefc01ce2cf (patch) | |
tree | 9dd671e38507926c396fd20a32e095afdf944087 | |
parent | 065e519e71b2c1f41936cce75b46b5ab34adb588 (diff) |
md.c:didn't unlock the mddev before return EINVAL in array_size_store
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>
-rw-r--r-- | drivers/md/md.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index a612b9f83069..5022be183390 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c | |||
@@ -4950,8 +4950,10 @@ array_size_store(struct mddev *mddev, const char *buf, size_t len) | |||
4950 | return err; | 4950 | return err; |
4951 | 4951 | ||
4952 | /* cluster raid doesn't support change array_sectors */ | 4952 | /* cluster raid doesn't support change array_sectors */ |
4953 | if (mddev_is_clustered(mddev)) | 4953 | if (mddev_is_clustered(mddev)) { |
4954 | mddev_unlock(mddev); | ||
4954 | return -EINVAL; | 4955 | return -EINVAL; |
4956 | } | ||
4955 | 4957 | ||
4956 | if (strncmp(buf, "default", 7) == 0) { | 4958 | if (strncmp(buf, "default", 7) == 0) { |
4957 | if (mddev->pers) | 4959 | if (mddev->pers) |