Commit c04fb18412be3908678cf3fc68a899272e6835bb

  • avatar
  • Peter Hutterer <peter.hutterer @wh…t.net>
  • Mon Dec 21 23:19:26 CET 2009
Fix touch property out-of-bounds memory access.

4fd2af39a5432eaad03af6a48cd16988a165fbbb split out the touch setting from a
combined property into its own separate one. The index didn't get updated
though, which means wcmTouch was assigned random garbage.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Tested-by: Ping Cheng <pinglinux@gmail.com>
src/wcmXCommand.c
(2 / 2)
  
552552 if ((values[0] != 0) && (values[0] != 1))
553553 return BadValue;
554554
555 if (!checkonly && common->wcmTouch != values[1])
556 common->wcmTouch = values[1];
555 if (!checkonly && common->wcmTouch != values[0])
556 common->wcmTouch = values[0];
557557 } else if (property == prop_hover)
558558 {
559559 CARD8 *values = (CARD8*)prop->data;