diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2012-10-22 07:15:00 -0400 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2012-11-02 05:43:47 -0400 |
commit | 8f1b7eb11422d28fbc7432f0dee2f97aead09417 (patch) | |
tree | eadfcfeb19e66b8d47ad9481dd9cf806a67a0930 /drivers/iio | |
parent | fad109452e0562e896ee0ede27552931c761b0ad (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.c | 2 |
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; |