Re: [PATCH 12/24] media/video: fix dangling pointers
From: Hans Verkuil
Date: Thu Apr 01 2010 - 04:32:57 EST
On Tuesday 30 March 2010 14:39:12 Wolfram Sang wrote:
> > But this just feels like an i2c core thing to me. After remove() is called
> > the core should just set the client data to NULL. If there are drivers that
> > rely on the current behavior, then those drivers should be reviewed first as
> > to the reason why they need it.
> It will be done this way now. As you have taken part in the discussion, I guess
> the media-subsystem never really considered picking those patches up ;)
I remember that there was one patch in your patch series where the client data
was set after it was freed.
That should still be fixed (by just removing the i2c_set_clientdata).
Can you post that one again?
Hans Verkuil - video4linux developer - sponsored by TANDBERG
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/