On Fri, 2010-05-07 at 10:38 +0200, Dan Carpenter wrote:
> m == num_est3_modes is one past the end of the est3_modes[]. 
> 
> Signed-off-by: Dan Carpenter <erro...@gmail.com>
> 
> diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> index 7188674..46447b0 100644
> --- a/drivers/gpu/drm/drm_edid.c
> +++ b/drivers/gpu/drm/drm_edid.c
> @@ -1383,7 +1383,7 @@ drm_est3_modes(struct drm_connector *connector, struct 
> detailed_timing *timing)
>       for (i = 0; i < 6; i++) {
>               for (j = 7; j > 0; j--) {
>                       m = (i * 8) + (7 - j);
> -                     if (m > num_est3_modes)
> +                     if (m >= num_est3_modes)
>                               break;
>                       if (est[i] & (1 << j)) {
>                               mode = drm_find_dmt(connector->dev,

Reviewed-by: Adam Jackson <a...@redhat.com>

- ajax

Attachment: signature.asc
Description: This is a digitally signed message part

------------------------------------------------------------------------------

--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to