Commit e278df1c authored by Peter Meerwald's avatar Peter Meerwald Committed by Greg Kroah-Hartman

staging: iio: add check for measurement configuration value passed to hmc5843

Signed-off-by: default avatarPeter Meerwald <pmeerw@pmeerw.net>
Acked-by: default avatarJonathan Cameron <jic23@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 13f6eb69
...@@ -281,9 +281,14 @@ static ssize_t hmc5843_set_measurement_configuration(struct device *dev, ...@@ -281,9 +281,14 @@ static ssize_t hmc5843_set_measurement_configuration(struct device *dev,
struct i2c_client *client = to_i2c_client(indio_dev->dev.parent); struct i2c_client *client = to_i2c_client(indio_dev->dev.parent);
struct hmc5843_data *data = iio_priv(indio_dev); struct hmc5843_data *data = iio_priv(indio_dev);
unsigned long meas_conf = 0; unsigned long meas_conf = 0;
int error = kstrtoul(buf, 10, &meas_conf); int error;
error = kstrtoul(buf, 10, &meas_conf);
if (error) if (error)
return error; return error;
if (meas_conf >= HMC5843_MEAS_CONF_NOT_USED)
return -EINVAL;
mutex_lock(&data->lock); mutex_lock(&data->lock);
dev_dbg(dev, "set mode to %lu\n", meas_conf); dev_dbg(dev, "set mode to %lu\n", meas_conf);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment