aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/iio
diff options
context:
space:
mode:
authorLars-Peter Clausen <lars@metafoo.de>2012-10-22 07:15:00 -0400
committerJonathan Cameron <jic23@kernel.org>2012-11-02 05:43:47 -0400
commit8f1b7eb11422d28fbc7432f0dee2f97aead09417 (patch)
treeeadfcfeb19e66b8d47ad9481dd9cf806a67a0930 /drivers/iio
parentfad109452e0562e896ee0ede27552931c761b0ad (diff)
iio: Do not accept multiple '.' in fixed point numbers
Currently when parsing a fix-point number we silently skip any additional '.' found in the string. E.g. '1.2.3.4' gets parsed as '1.234'. This patch disallows this and returns an error if more than one '.' is encountered. Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio')
-rw-r--r--drivers/iio/industrialio-core.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
index 37650a72b31f..0994239361ba 100644
--- a/drivers/iio/industrialio-core.c
+++ b/drivers/iio/industrialio-core.c
@@ -454,7 +454,7 @@ static ssize_t iio_write_channel_info(struct device *dev,
454 break; 454 break;
455 else 455 else
456 return -EINVAL; 456 return -EINVAL;
457 } else if (*buf == '.') { 457 } else if (*buf == '.' && integer_part) {
458 integer_part = false; 458 integer_part = false;
459 } else { 459 } else {
460 return -EINVAL; 460 return -EINVAL;