[patch v2] Staging: iio: some uninitialized variable bugs

Jonathan Cameron jic23 at cam.ac.uk
Fri Jul 1 09:09:38 UTC 2011


On 06/29/11 20:50, Dan Carpenter wrote:
> There were some uninitialized variable warnings in iio.  Two of
> these came from the recent changes to how the private data was
> allocated in 83f0422dc6a16 "staging:iio:accel:sca3000: allocate
> state in iio_dev and use iio_priv to access."
> 
> drivers/staging/iio/accel/sca3000_core.c: In function 'sca3000_probe':
> drivers/staging/iio/accel/sca3000_core.c:1137:9: warning: 'st' may be used uninitialized in this function
> drivers/staging/iio/adc/ad7291.c: In function 'ad7291_probe':
> drivers/staging/iio/adc/ad7291.c:805:15: warning: 'chip' may be used uninitialized in this function
> drivers/staging/iio/dac/ad5624r_spi.c: In function 'ad5624r_probe':
> drivers/staging/iio/dac/ad5624r_spi.c:228:24: warning: 'st' may be used uninitialized in this function
> 
Thanks,
> Signed-off-by: Dan Carpenter <error27 at gmail.com>
Acked-by: Jonathan Cameron <jic23 at cam.ac.uk>
> ---
> v2:  I added a couple more bug related bug fixes.
> 
> diff --git a/drivers/staging/iio/accel/sca3000_core.c b/drivers/staging/iio/accel/sca3000_core.c
> index 4313f73..603f5bc 100644
> --- a/drivers/staging/iio/accel/sca3000_core.c
> +++ b/drivers/staging/iio/accel/sca3000_core.c
> @@ -1133,6 +1133,7 @@ static int __devinit sca3000_probe(struct spi_device *spi)
>  		goto error_ret;
>  	}
>  
> +	st = iio_priv(indio_dev);
>  	spi_set_drvdata(spi, indio_dev);
>  	st->us = spi;
>  	mutex_init(&st->lock);
> diff --git a/drivers/staging/iio/adc/ad7291.c b/drivers/staging/iio/adc/ad7291.c
> index f024026..96cbb17 100644
> --- a/drivers/staging/iio/adc/ad7291.c
> +++ b/drivers/staging/iio/adc/ad7291.c
> @@ -799,6 +799,7 @@ static int __devinit ad7291_probe(struct i2c_client *client,
>  		ret = -ENOMEM;
>  		goto error_ret;
>  	}
> +	chip = iio_priv(indio_dev);
>  	/* this is only used for device removal purposes */
>  	i2c_set_clientdata(client, indio_dev);
>  
> diff --git a/drivers/staging/iio/dac/ad5624r_spi.c b/drivers/staging/iio/dac/ad5624r_spi.c
> index 0175cc0..a5b3776 100644
> --- a/drivers/staging/iio/dac/ad5624r_spi.c
> +++ b/drivers/staging/iio/dac/ad5624r_spi.c
> @@ -276,7 +276,7 @@ error_free_dev:
>  	iio_free_device(indio_dev);
>  error_disable_reg:
>  	if (!IS_ERR(reg))
> -		regulator_disable(st->reg);
> +		regulator_disable(reg);
>  error_put_reg:
>  	if (!IS_ERR(reg))
>  		regulator_put(reg);
> 
> 
> 
> 




More information about the devel mailing list