On Wed, 2 Feb 2011, Guennadi Liakhovetski wrote:

> On Mon, 31 Jan 2011, Anatolij Gustschin wrote:
> 
> > Running with enabled debug output in the mx3_camera driver results
> > in a kernel crash:
> > ...
> > mx3-camera mx3-camera.0: Providing format Bayer BGGR (sRGB) 8 bit using 
> > code 11
> > Unable to handle kernel NULL pointer dereference at virtual address 00000004
> > ...
> > 
> > Fix it by incrementing 'xlate' after usage.
> > 
> > Signed-off-by: Anatolij Gustschin <[email protected]>
> > ---
> >  drivers/media/video/mx3_camera.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/drivers/media/video/mx3_camera.c 
> > b/drivers/media/video/mx3_camera.c
> > index b9cb4a4..7bcaaf7 100644
> > --- a/drivers/media/video/mx3_camera.c
> > +++ b/drivers/media/video/mx3_camera.c
> > @@ -734,9 +734,9 @@ static int mx3_camera_get_formats(struct 
> > soc_camera_device *icd, unsigned int id
> >     if (xlate) {
> >             xlate->host_fmt = fmt;
> >             xlate->code     = code;
> > -           xlate++;
> >             dev_dbg(dev, "Providing format %x in pass-through mode\n",
> >                     xlate->host_fmt->fourcc);
> > +           xlate++;
> 
> Let's take an even easier path:
> 
> -                     xlate->host_fmt->fourcc);
> +                     fmt->fourcc);

Ok, sorry, forget it - this entire patch is superseded by another one.

Guennadi

> 
> 
> >     }
> >  
> >     return formats;
> > -- 
> > 1.7.1
> 
> Thanks
> Guennadi
> ---
> Guennadi Liakhovetski
> 

---
Guennadi Liakhovetski
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to