Commit 1f310f77 authored by Dan Carpenter's avatar Dan Carpenter Committed by Jonathan Cameron

iio: adc: intel_mrfld_adc: Allocating too much data in probe()

This probe function is passing the wrong size to devm_iio_device_alloc().
It is supposed to be the size of the private data.  Fortunately,
sizeof(*indio_dev) is larger than sizeof(struct mrfld_adc) so it doesn't
cause a runtime problem.

Fixes: a7118662 ("iio: adc: intel_mrfld_adc: Add Basin Cove ADC driver")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 11d7c8d3
...@@ -185,7 +185,7 @@ static int mrfld_adc_probe(struct platform_device *pdev) ...@@ -185,7 +185,7 @@ static int mrfld_adc_probe(struct platform_device *pdev)
int irq; int irq;
int ret; int ret;
indio_dev = devm_iio_device_alloc(dev, sizeof(*indio_dev)); indio_dev = devm_iio_device_alloc(dev, sizeof(struct mrfld_adc));
if (!indio_dev) if (!indio_dev)
return -ENOMEM; return -ENOMEM;
......
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