From 34167aa6b0d7d53016b8574b4fe1fac764ddf615 Mon Sep 17 00:00:00 2001 From: Russ Cox Date: Mon, 3 Mar 2008 23:37:12 -0500 Subject: lib9: gcc-4.2 bug in sprint --- src/lib9/fmt/sprint.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/lib9/fmt') diff --git a/src/lib9/fmt/sprint.c b/src/lib9/fmt/sprint.c index 9e3cb63e..4155d885 100644 --- a/src/lib9/fmt/sprint.c +++ b/src/lib9/fmt/sprint.c @@ -16,8 +16,11 @@ sprint(char *buf, char *fmt, ...) /* * on PowerPC, the stack is near the top of memory, so * we must be sure not to overflow a 32-bit pointer. + * + * careful! gcc-4.2 assumes buf+len < buf can never be true and + * optimizes the test away. casting to uintptr works around this bug. */ - if(buf+len < buf) + if((uintptr)buf+len < (uintptr)buf) len = -(uintptr)buf-1; va_start(args, fmt); -- cgit v1.2.3