summaryrefslogtreecommitdiffstats
path: root/xlators/cluster/stripe/src
diff options
context:
space:
mode:
authorshishir gowda <shishirng@gluster.com>2011-05-02 05:08:39 +0000
committerAnand Avati <avati@gluster.com>2011-05-11 19:18:08 -0700
commit276c0cecadbe763e98c5cfd18b3ad32822964789 (patch)
tree78057ddcabcd688c99fcdea84556a9b088393522 /xlators/cluster/stripe/src
parent0f020affb2cbbcf7c3a25f9f10fe8f14bf012b8e (diff)
Validate stripe-block-size option
Minimum size is 512 bytes and should be a multiple of 512 bytes. Signed-off-by: shishir gowda <shishirng@gluster.com> Signed-off-by: Anand Avati <avati@gluster.com> BUG: 2861 (Stripe block-size has no check for valid option) URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2861
Diffstat (limited to 'xlators/cluster/stripe/src')
-rw-r--r--xlators/cluster/stripe/src/stripe.c12
1 files changed, 12 insertions, 0 deletions
diff --git a/xlators/cluster/stripe/src/stripe.c b/xlators/cluster/stripe/src/stripe.c
index a06d275df..8f6790777 100644
--- a/xlators/cluster/stripe/src/stripe.c
+++ b/xlators/cluster/stripe/src/stripe.c
@@ -3750,6 +3750,18 @@ set_stripe_block_size (xlator_t *this, stripe_private_t *priv, char *data)
"invalid number format \"%s\"", num);
goto out;
}
+
+ if (stripe_opt->block_size < 512) {
+ gf_log (this->name, GF_LOG_ERROR, "Invalid Block-size: "
+ "%s. Should be atleast 512 bytes", num);
+ goto out;
+ }
+ if (stripe_opt->block_size % 512) {
+ gf_log (this->name, GF_LOG_ERROR, "Block-size: %s should"
+ " be a multiple of 512 bytes", num);
+ goto out;
+ }
+
memcpy (stripe_opt->path_pattern, pattern, strlen (pattern));
gf_log (this->name, GF_LOG_DEBUG,