summaryrefslogtreecommitdiff
path: root/drivers/md
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2015-06-12 12:46:44 +0300
committerNeilBrown <neilb@suse.de>2015-06-12 13:16:16 +0300
commitc008f1d356277a5b7561040596a073d87e56b0c8 (patch)
tree6d7eb913a79b11f8ed9beee30c81eca81e271ed2 /drivers/md
parentc65b99f046843d2455aa231747b5a07a999a9f3d (diff)
downloadlinux-c008f1d356277a5b7561040596a073d87e56b0c8.tar.xz
md: don't return 0 from array_state_store
Returning zero from a 'store' function is bad. The return value should be either len length of the string or an error. So use 'len' if 'err' is zero. Fixes: 6791875e2e53 ("md: make reconfig_mutex optional for writes to md sysfs files.") Signed-off-by: NeilBrown <neilb@suse.de> Cc: stable@vger.kernel (v4.0+)
Diffstat (limited to 'drivers/md')
-rw-r--r--drivers/md/md.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c
index 27506302eb7a..dd59d71ade2f 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -3834,7 +3834,7 @@ array_state_store(struct mddev *mddev, const char *buf, size_t len)
err = -EBUSY;
}
spin_unlock(&mddev->lock);
- return err;
+ return err ?: len;
}
err = mddev_lock(mddev);
if (err)