clang complained (correctly):
warning: comparison of constant 768614336404564650 with expression
of type 'CARD32' (aka 'unsigned int') is always true
[-Wtautological-constant-out-of-range-compare]
---
 src/Font.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Font.c b/src/Font.c
index a32f740..ac18024 100644
--- a/src/Font.c
+++ b/src/Font.c
@@ -588,7 +588,7 @@ _XF86BigfontQueryFont (
            if (!(extcodes->serverCapabilities & CAP_VerifiedLocal)) {
                struct shmid_ds buf;
                if (!(shmctl(reply.shmid, IPC_STAT, &buf) >= 0
-                     && reply.nCharInfos < (LONG_MAX / sizeof(XCharStruct))
+                     && reply.nCharInfos < (INT_MAX / sizeof(XCharStruct))
                      && buf.shm_segsz >= reply.shmsegoffset + reply.nCharInfos 
* sizeof(XCharStruct) + sizeof(CARD32)
                      && *(CARD32 *)(addr + reply.shmsegoffset + 
reply.nCharInfos * sizeof(XCharStruct)) == extcodes->serverSignature)) {
                    shmdt(addr);
-- 
1.8.3.1

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to