summaryrefslogtreecommitdiff
path: root/Xext
diff options
context:
space:
mode:
authorAdam Jackson <ajax@redhat.com>2016-03-01 14:09:29 -0500
committerAdam Jackson <ajax@redhat.com>2016-03-11 13:14:32 -0500
commit0e3cc3d042a2da1ed781f6691dd7aca72d87046b (patch)
tree78ac2714880c0cdede437ab70969dca85db21a6f /Xext
parent3d7bfe02cc3ad0f16403a073f1182d7019c98be3 (diff)
xv: Fix unchecked AddResource
Reviewed-by: Julien Cristau <jcristau@debian.org> Signed-off-by: Adam Jackson <ajax@redhat.com> (cherry picked from commit 093f9505c12565cc19bdf6e33b263f31d104c3ef)
Diffstat (limited to 'Xext')
-rw-r--r--Xext/xvmain.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/Xext/xvmain.c b/Xext/xvmain.c
index 0c6f25b5a..3a0263427 100644
--- a/Xext/xvmain.c
+++ b/Xext/xvmain.c
@@ -844,7 +844,8 @@ XvdiSelectVideoNotify(ClientPtr client, DrawablePtr pDraw, BOOL onoff)
tpn->client = NULL;
tpn->id = FakeClientID(client->index);
- AddResource(tpn->id, XvRTVideoNotify, tpn);
+ if (!AddResource(tpn->id, XvRTVideoNotify, tpn))
+ return BadAlloc;
tpn->client = client;
return Success;
@@ -893,7 +894,8 @@ XvdiSelectPortNotify(ClientPtr client, XvPortPtr pPort, BOOL onoff)
tpn->client = client;
tpn->id = FakeClientID(client->index);
- AddResource(tpn->id, XvRTPortNotify, tpn);
+ if (!AddResource(tpn->id, XvRTPortNotify, tpn))
+ return BadAlloc;
return Success;