The test here should be ">= ARRAY_SIZE()" instead of "> ARRAY_SIZE()".

Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
Acked-by: Maarten Lankhorst <maarten.lankho...@canonical.com>
---
Somehow this wasn't applied when I sent it earlier.

diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c 
b/drivers/gpu/drm/nouveau/nouveau_drm.c
index c95decf..e11f8e4 100644
--- a/drivers/gpu/drm/nouveau/nouveau_drm.c
+++ b/drivers/gpu/drm/nouveau/nouveau_drm.c
@@ -86,7 +86,7 @@ nouveau_drm_vblank_enable(struct drm_device *dev, int head)
        struct nouveau_drm *drm = nouveau_drm(dev);
        struct nouveau_disp *pdisp = nouveau_disp(drm->device);
 
-       if (WARN_ON_ONCE(head > ARRAY_SIZE(drm->vblank)))
+       if (WARN_ON_ONCE(head >= ARRAY_SIZE(drm->vblank)))
                return -EIO;
        WARN_ON_ONCE(drm->vblank[head].func);
        drm->vblank[head].func = nouveau_drm_vblank_handler;
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to