aboutsummaryrefslogtreecommitdiff
path: root/include/fcall.h
diff options
context:
space:
mode:
authorrsc <devnull@localhost>2006-11-04 20:41:42 +0000
committerrsc <devnull@localhost>2006-11-04 20:41:42 +0000
commit9a5678a8a95c46913c15a5fc19f73c113e8ed036 (patch)
tree0a4ea2fc4ec7ada7397ac05b684ecec5e78a4be9 /include/fcall.h
parent3a19470202c5c0f6e9375e5d57535c3d508f2edf (diff)
downloadplan9port-9a5678a8a95c46913c15a5fc19f73c113e8ed036.tar.gz
plan9port-9a5678a8a95c46913c15a5fc19f73c113e8ed036.tar.bz2
plan9port-9a5678a8a95c46913c15a5fc19f73c113e8ed036.zip
Change GET/GBIT32 macros to explicitly cast final result to u32int.
Otherwise, the type of the expression will be int, and if written to a uvlong (or just a ulong on most 64-bit systems), it will be sign-extended. This was going on in drawfcall with Mouse.msec, which is declared as a ulong. This fix assumes that the protocols only send unsigned numbers, which is true.
Diffstat (limited to 'include/fcall.h')
-rw-r--r--include/fcall.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/fcall.h b/include/fcall.h
index aebb4515..a5dae337 100644
--- a/include/fcall.h
+++ b/include/fcall.h
@@ -50,7 +50,7 @@ struct Fcall
#define GBIT8(p) ((p)[0])
#define GBIT16(p) ((p)[0]|((p)[1]<<8))
-#define GBIT32(p) ((p)[0]|((p)[1]<<8)|((p)[2]<<16)|((p)[3]<<24))
+#define GBIT32(p) ((u32int)((p)[0]|((p)[1]<<8)|((p)[2]<<16)|((p)[3]<<24)))
#define GBIT64(p) ((u32int)((p)[0]|((p)[1]<<8)|((p)[2]<<16)|((p)[3]<<24)) |\
((vlong)((p)[4]|((p)[5]<<8)|((p)[6]<<16)|((p)[7]<<24)) << 32))