diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-09-27 16:17:14 -0400 |
---|---|---|
committer | Ben Skeggs <bskeggs@redhat.com> | 2013-11-08 00:36:02 -0500 |
commit | ca3562226a901ddf77c7da36077c25316c4c0bb7 (patch) | |
tree | 21907defb5c90cd19d00e5e33d5268cda2fff5b8 | |
parent | 1934a2add96bdd9922c6752b07244cbd11d6a7d7 (diff) |
drm/nouveau: off by one in nouveau_drm_vblank_enable()
The test here should be ">= ARRAY_SIZE()" instead of "> ARRAY_SIZE()".
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Maarten Lankhorst <maarten.lankhorst@canonical.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
-rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_drm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c index 428d818be775..20626b93130c 100644 --- a/drivers/gpu/drm/nouveau/nouveau_drm.c +++ b/drivers/gpu/drm/nouveau/nouveau_drm.c | |||
@@ -93,7 +93,7 @@ nouveau_drm_vblank_enable(struct drm_device *dev, int head) | |||
93 | struct nouveau_drm *drm = nouveau_drm(dev); | 93 | struct nouveau_drm *drm = nouveau_drm(dev); |
94 | struct nouveau_disp *pdisp = nouveau_disp(drm->device); | 94 | struct nouveau_disp *pdisp = nouveau_disp(drm->device); |
95 | 95 | ||
96 | if (WARN_ON_ONCE(head > ARRAY_SIZE(drm->vblank))) | 96 | if (WARN_ON_ONCE(head >= ARRAY_SIZE(drm->vblank))) |
97 | return -EIO; | 97 | return -EIO; |
98 | WARN_ON_ONCE(drm->vblank[head].func); | 98 | WARN_ON_ONCE(drm->vblank[head].func); |
99 | drm->vblank[head].func = nouveau_drm_vblank_handler; | 99 | drm->vblank[head].func = nouveau_drm_vblank_handler; |