From: Thomas H. <th...@sh...> - 2010-04-28 08:18:15
|
ak...@li... wrote: > From: Dan Carpenter <er...@gm...> > > "fx->lock" is used as the index in "dev_priv->decoder_queue[fx->lock]" > which is an array of "VIA_NR_XVMC_LOCKS" elements. > > Signed-off-by: Dan Carpenter <er...@gm...> > Cc: David Airlie <ai...@li...> > Signed-off-by: Andrew Morton <ak...@li...> > Acked-by: Thomas Hellstrom <the...@vm...> > --- > > drivers/gpu/drm/via/via_video.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff -puN drivers/gpu/drm/via/via_video.c~drivers-gpu-drm-via-via_videoc-fix-off-by-one-issue drivers/gpu/drm/via/via_video.c > --- a/drivers/gpu/drm/via/via_video.c~drivers-gpu-drm-via-via_videoc-fix-off-by-one-issue > +++ a/drivers/gpu/drm/via/via_video.c > @@ -75,7 +75,7 @@ int via_decoder_futex(struct drm_device > > DRM_DEBUG("\n"); > > - if (fx->lock > VIA_NR_XVMC_LOCKS) > + if (fx->lock >= VIA_NR_XVMC_LOCKS) > return -EFAULT; > > lock = (volatile int *)XVMCLOCKPTR(sAPriv, fx->lock); > _ > > ------------------------------------------------------------------------------ > -- > _______________________________________________ > Dri-devel mailing list > Dri...@li... > https://lists.sourceforge.net/lists/listinfo/dri-devel > |