On Mon, Jun 06, 2011 at 07:43:39PM +0200, Hans de Goede wrote: > Unplugging a pwc cam while an app has the /dev/video# node open leads > to an oops in pwc_video_close when the app closes the node, because > the disconnect handler has free-ed the pdev struct pwc_video_close > tries to use. Instead of adding some sort of bandaid for this. > fix it properly using the v4l2 core's new(ish) behavior of keeping the > v4l2_dev structure around until both unregister has been called, and > all file handles referring to it have been closed: > > Embed the v4l2_dev structure in the pdev structure and define a v4l2 dev > release callback releasing the pdev structure (and thus also the embedded > v4l2 dev structure. > > Signed-off-by: Hans de Goede <hdego...@redhat.com> > --- > drivers/media/video/pwc/pwc-ctrl.c | 2 +- > drivers/media/video/pwc/pwc-if.c | 152 > +++++++++++------------------------- > drivers/media/video/pwc/pwc.h | 4 +- > 3 files changed, 50 insertions(+), 108 deletions(-)
<formletter> This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read Documentation/stable_kernel_rules.txt for how to do this properly. </formletter> -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html