[PATCH v7 10/15] Log messages in TouchBeginDDXTouch() in a signal-safe manner

Peter Hutterer peter.hutterer at who-t.net
Wed Jun 20 22:56:41 PDT 2012


From: Chase Douglas <chase.douglas at canonical.com>

Signed-off-by: Chase Douglas <chase.douglas at canonical.com>
Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>
---
 dix/touch.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/dix/touch.c b/dix/touch.c
index aa17faf..a01f152 100644
--- a/dix/touch.c
+++ b/dix/touch.c
@@ -198,8 +198,9 @@ TouchBeginDDXTouch(DeviceIntPtr dev, uint32_t ddx_id)
     /* If we get here, then we've run out of touches and we need to drop the
      * event (we're inside the SIGIO handler here) schedule a WorkProc to
      * grow the queue for us for next time. */
-    ErrorF("%s: not enough space for touch events (max %d touchpoints). "
-           "Dropping this event.\n", dev->name, dev->last.num_touches);
+    ErrorFSigSafe("%s: not enough space for touch events (max %u touchpoints). "
+                  "Dropping this event.\n", dev->name, dev->last.num_touches);
+
     if (!BitIsOn(resize_waiting, dev->id)) {
         SetBit(resize_waiting, dev->id);
         QueueWorkProc(TouchResizeQueue, serverClient, NULL);
-- 
1.7.10.2



More information about the xorg-devel mailing list