summaryrefslogtreecommitdiff
path: root/src/XIProperties.c
AgeCommit message (Collapse)Author
2014-11-03Fix double unlock when _XiCheckExtInit return -1.Michal Srb
_XiCheckExtInit unlocks the display if it fails and returns -1. Most callers account for it properly, but few didn't. Signed-off-by: Michal Srb <msrb@suse.com> Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2013-05-23integer overflow in XIGetProperty() [CVE-2013-1984 5/8]Alan Coopersmith
If the number of items reported by the server is large enough that it overflows when multiplied by the size of the appropriate item type, then memory corruption can occur when more bytes are copied from the X server reply than the size of the buffer we allocated to hold them. Reported-by: Ilja Van Sprundel <ivansprundel@ioactive.com> Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2013-05-23Use _XEatDataWords to avoid overflow of rep.length bit shiftingAlan Coopersmith
rep.length is a CARD32, so rep.length << 2 could overflow in 32-bit builds Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2011-03-03Require XI 2 for all XI 2 calls.Peter Hutterer
Each XI2 call should check _XiCheckExtInit for XI version 2.0 or higher. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Daniel Stone <daniel@fooishbar.org>
2010-07-13Fix usage of uninitialized valuePauli Nieminen
In error case length of extra data could be uninitialized. This would result randomly sized request later in function. Signed-off-by: Pauli Nieminen <ext-pauli.nieminen@nokia.com> Reviewed-by: Julien Cristau <jcristau@debian.org>
2009-06-02Use Dont_Check for most XI2 requests.Peter Hutterer
If a client issues an XI2 request against a non-XI2 server it should generate a BadRequest error from the server. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2009-05-15Remove duplicated code paths in XIGetProperty.Peter Hutterer
XIGetProperty always returns in the specified format (8, 16, 32) and never in the Xlib's 64-bit long hackery. So we can clean this code up a bit. This doesn't affect XListProperties, it still returns an Atom list. Signed-off-by: Peter Hutterer <peter.hutterer@redhat.com>
2009-05-07Add XI2 property requests.Peter Hutterer
Without Xlib's 64-bit hilarity. A property claiming to be 32 bit is treated as 32 bit value.