Reviewed-By: Jasper St. Pierre <<a href="mailto:jstpierre@mecheye.net">jstpierre@mecheye.net</a>><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 16, 2012 at 12:36 AM, Peter Hutterer <span dir="ltr"><<a href="mailto:peter.hutterer@who-t.net" target="_blank">peter.hutterer@who-t.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">"Fixed" up Jasper's original patch [1] to use pointers instead of<br>
zero-length arrays but didn't actually set the pointer to anywhere.<br>
<br>
[1] 9a536820f6174befb22431bab375ef1af0a1ec29<br>
<br>
Signed-off-by: Peter Hutterer <<a href="mailto:peter.hutterer@who-t.net">peter.hutterer@who-t.net</a>><br>
---<br>
device_ids[0] is a gnu-ism, having device_ids[] is C99, so let's go with the<br>
old, boring style.<br>
<br>
 xfixes/cursor.c | 4 ++++<br>
 1 file changed, 4 insertions(+)<br>
<br>
diff --git a/xfixes/cursor.c b/xfixes/cursor.c<br>
index 7c223dd..bd175b4 100644<br>
--- a/xfixes/cursor.c<br>
+++ b/xfixes/cursor.c<br>
@@ -1300,6 +1300,10 @@ CreatePointerBarrierClient(ScreenPtr screen, ClientPtr client,<br>
<br>
     ret->screen = screen;<br>
     ret->num_devices = stuff->num_devices;<br>
+    if (ret->num_devices > 0)<br>
+        ret->device_ids = (int*)&ret[1];<br>
+    else<br>
+        ret->device_ids = NULL;<br>
<br>
     in_devices = (CARD16 *) &stuff[1];<br>
     for (i = 0; i < stuff->num_devices; i++) {<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.11.7<br>
<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>  Jasper<br><br>
</div>