On 10/20/11 03:44, Dave Airlie wrote:
From: Dave Airlie<[email protected]>
Pointed out by coverity.
Signed-off-by: Dave Airlie<[email protected]>
---
Xext/xvdisp.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/Xext/xvdisp.c b/Xext/xvdisp.c
index 364a90c..71bf722 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;
}
Looks like the same fix is needed at line 1545 in the swapped version.
--
-Alan Coopersmith- [email protected]
Oracle Solaris Platform Engineering: X Window System
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
