syndaemon: don't compare against a null-property. (#37459)

synaptics_property is not set, touchpad_off_prop is the property we need to
check against.

False check for (nprops != 0) instead of (nprops < 0) would result in
syndaemon always reporting a touchpad device, even if none are present.

X.Org Bug 37459 <http://bugs.freedesktop.org/show_bug.cgi?id=37459>

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
This commit is contained in:
Peter Hutterer
2011-05-23 10:18:22 +10:00
parent 33775253a0
commit be83094bd9

View File

@@ -465,7 +465,6 @@ dp_get_device(Display *dpy)
XDeviceInfo *info = NULL;
int ndevices = 0;
Atom touchpad_type = 0;
Atom synaptics_property = 0;
Atom *properties = NULL;
int nprops = 0;
int error = 0;
@@ -495,10 +494,10 @@ dp_get_device(Display *dpy)
while(nprops--)
{
if (properties[nprops] == synaptics_property)
if (properties[nprops] == touchpad_off_prop)
break;
}
if (!nprops)
if (nprops < 0)
{
fprintf(stderr, "No synaptics properties on device '%s'.\n",
info[ndevices].name);