Staging: iio: adc: fix dangling pointers
Fix I2C-drivers which missed setting clientdata to NULL before freeing the structure it points to. Also fix drivers which do this _after_ the structure was freed already. Signed-off-by: Wolfram Sang <w.sang@pengutronix.de> Acked-by: Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
parent
b36fc07dc3
commit
ee58cb2bcd
1 changed files with 2 additions and 0 deletions
|
@ -557,6 +557,7 @@ error_put_reg:
|
||||||
if (!IS_ERR(st->reg))
|
if (!IS_ERR(st->reg))
|
||||||
regulator_put(st->reg);
|
regulator_put(st->reg);
|
||||||
error_free_st:
|
error_free_st:
|
||||||
|
i2c_set_clientdata(client, NULL);
|
||||||
kfree(st);
|
kfree(st);
|
||||||
|
|
||||||
error_ret:
|
error_ret:
|
||||||
|
@ -574,6 +575,7 @@ static int max1363_remove(struct i2c_client *client)
|
||||||
regulator_disable(st->reg);
|
regulator_disable(st->reg);
|
||||||
regulator_put(st->reg);
|
regulator_put(st->reg);
|
||||||
}
|
}
|
||||||
|
i2c_set_clientdata(client, NULL);
|
||||||
kfree(st);
|
kfree(st);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
Reference in a new issue