From 1c66e30e7e715f28c5b3fd3342425aac48ad7828 Mon Sep 17 00:00:00 2001 From: Mikael Pettersson Date: Sun, 14 Apr 2013 10:56:47 +0200 Subject: [PATCH] Fix memmove() [The lcc source] overrides the libc memmove() with its own implementation, but that implementation fails to follow the specification. In particular, it returns NULL rather than memmove()'s first parameter. GCC now optimizes based on this aspect of the specification, so things go wrong at runtime. [Text & patch from http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56881#c8] --- code/tools/lcc/cpp/unix.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/code/tools/lcc/cpp/unix.c b/code/tools/lcc/cpp/unix.c index 17986d84..bd879448 100644 --- a/code/tools/lcc/cpp/unix.c +++ b/code/tools/lcc/cpp/unix.c @@ -109,7 +109,7 @@ memmove(void *dp, const void *sp, size_t n) unsigned char *cdp, *csp; if (n<=0) - return 0; + return dp; cdp = dp; csp = (unsigned char *)sp; if (cdp < csp) { @@ -123,6 +123,6 @@ memmove(void *dp, const void *sp, size_t n) *--cdp = *--csp; } while (--n); } - return 0; + return dp; } #endif