Messages in this thread | | | Subject | Re: [PATCH] TDA1997x: fix tda1997x_remove() | From | Hans Verkuil <> | Date | Thu, 29 Jul 2021 11:55:47 +0200 |
| |
On 26/07/2021 12:44, Krzysztof Hałasa wrote: > TDA1997x I2C "client data" pointer was never set in tda1997x_probe(), > then the code tried to use invalid pointer in tda1997x_remove(). > > Signed-off-by: Krzysztof Hałasa <khalasa@piap.pl> > > diff --git a/drivers/media/i2c/tda1997x.c b/drivers/media/i2c/tda1997x.c > index 71194746c874..043cc8275d00 100644 > --- a/drivers/media/i2c/tda1997x.c > +++ b/drivers/media/i2c/tda1997x.c > @@ -2771,6 +2771,7 @@ static int tda1997x_probe(struct i2c_client *client, > goto err_free_media; > } > > + i2c_set_clientdata(client, sd); > return 0; > > err_free_media: >
Actually, v4l2_i2c_subdev_init() sets this, and v4l2_i2c_subdev_init() *is* called. Does it really crash in tda1997x_remove() without this patch?
If so, then I suspect something else is going on.
Regards,
Hans
| |