Switch sequence comparisons to handle 32-bit sequence number wrap.

Create a macro, XCB_SEQUENCE_COMPARE, that accepts two sequence numbers and
a comparison operator and correctly handles 32-bit wrap around.
Rewrite all ordered sequence number comparisons to use this macro.
Also, caught one error where a sequence was stored in a signed int variable.
Push out a GetInputFocus request when the sequence number does wrap at 32
bits so that applications cannot see sequence 0 (as that is an error
indicator).

1 files changed, 2 insertions, 0 deletions

diff --git a/src/xcbint.h b/src/xcbint.h
index a6d92aa..9048e9d 100644
--- a/src/xcbint.h
+++ b/src/xcbint.h
@@ -43,6 +43,8 @@ enum workarounds {
 
 #define XCB_PAD(i) (-(i) & 3)
 
+#define XCB_SEQUENCE_COMPARE(a,op,b)	((int) ((a) - (b)) op 0)
+
 /* xcb_list.c */
 
 typedef void (*XCBListFreeFunc)(void *);