diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-09-27 23:17:14 +0300 |
|---|---|---|
| committer | Ben Skeggs <bskeggs@redhat.com> | 2013-11-05 16:15:42 +1000 |
| commit | 98cbe24b0dee21fb7a4b34fa75dd7d38c26de6fe (patch) | |
| tree | cda09912d6005beff703b298f0f649f49e3c7c1b /drm/nouveau_drm.c | |
| parent | f26e4e7bc3802c52fdbb65cf2da06a66bbcd3082 (diff) | |
| download | nouveau-98cbe24b0dee21fb7a4b34fa75dd7d38c26de6fe.tar.gz | |
drm: 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>
Diffstat (limited to 'drm/nouveau_drm.c')
| -rw-r--r-- | drm/nouveau_drm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drm/nouveau_drm.c b/drm/nouveau_drm.c index e893c5362..c79c805e1 100644 --- a/drm/nouveau_drm.c +++ b/drm/nouveau_drm.c @@ -93,7 +93,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; |