diff options
| author | Dave Airlie <airlied@redhat.com> | 2011-10-20 11:00:43 +0100 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2011-12-06 16:01:50 +0000 |
| commit | 41229392b790f30a0f0ef1f4ed95647c5bca4001 (patch) | |
| tree | 1e760872aebe26576daf63d845523ced4d979042 /Xext | |
| parent | 1049139499d9132a20cd6d4d156fe9da9cddb6c2 (diff) | |
xv: test correct number of requests. (v2)
Pointed out by coverity.
v2: fix swapped as well, as pointed out by Alan
Signed-off-by: Dave Airlie <airlied@redhat.com>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'Xext')
| -rw-r--r-- | Xext/xvdisp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Xext/xvdisp.c b/Xext/xvdisp.c index 364a90cf2..0795a1475 100644 --- a/Xext/xvdisp.c +++ b/Xext/xvdisp.c @@ -1238,7 +1238,7 @@ ProcXvDispatch(ClientPtr client) UpdateCurrentTime(); - if (stuff->data > xvNumRequests) { + if (stuff->data >= xvNumRequests) { SendErrorToClient(client, XvReqCode, stuff->data, 0, BadRequest); return BadRequest; } @@ -1542,7 +1542,7 @@ SProcXvDispatch(ClientPtr client) UpdateCurrentTime(); - if (stuff->data > xvNumRequests) { + if (stuff->data >= xvNumRequests) { SendErrorToClient(client, XvReqCode, stuff->data, 0, BadRequest); return BadRequest; } |