xserver: Branch 'master'

Keith Packard keithp at kemper.freedesktop.org
Sun Nov 22 19:43:23 PST 2009


 Xi/xiproperty.c |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit a30e739a144912a68adcaa9f426d600c6ecbd529
Author: Peter Hutterer <peter.hutterer at who-t.net>
Date:   Mon Nov 23 13:07:56 2009 +1000

    Xi: don't crash when deleting invalid device properties.
    
    Deleting a property that was not set on a device leads to a null-pointer
    reference. The protocol allows deleting those properties - it has to be a
    noop.
    
    Reproducible:
    xinput --set-prop "My device" --type=int --format=8 "my property" 1
    xinput --delete-prop "My other device" "my property"
    
    Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
    Signed-off-by: Keith Packard <keithp at keithp.com>

diff --git a/Xi/xiproperty.c b/Xi/xiproperty.c
index 024dc44..4821857 100644
--- a/Xi/xiproperty.c
+++ b/Xi/xiproperty.c
@@ -643,6 +643,9 @@ XIDeleteDeviceProperty (DeviceIntPtr device, Atom property, Bool fromClient)
         if (prop->propertyName == property)
             break;
 
+    if (!prop)
+        return Success;
+
     if (fromClient && !prop->deletable)
         return BadAccess;
 


More information about the xorg-commit mailing list