* alloc.c (allocate_string_data) [GC_CHECK_STRING_OVERRUN]: Fix typo that

can cause Emacs to crash when string overrun checking is enabled.

diff --git a/src/ChangeLog b/src/ChangeLog
index d51052f0e3da7d159099ee6a58ad6dfa81ec0d0e..fa2aa03233d19a5c3553bda1fca236b78214c3ae 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -4,6 +4,8 @@
        This doesn't fix a bug but makes the code clearer.
        (string_overrun_cookie): Now const.  Use initializers that
        don't formally overflow signed char, to avoid warnings.
+       (allocate_string_data) [GC_CHECK_STRING_OVERRUN]: Fix typo that
+       can cause Emacs to crash when string overrun checking is enabled.
 
 2011-04-17  Chong Yidong  <cyd@stupidchicken.com>
 

diff --git a/src/alloc.c b/src/alloc.c
index 2029383dec86257eee61efb8971e7ca4f7ee0ed9..2af75e3c4717dd86933b94c0c86a11c17d884f91 100644 (file)
--- a/src/alloc.c
+++ b/src/alloc.c
@@ -1937,7 +1937,8 @@ allocate_string_data (struct Lisp_String *s,
   s->size_byte = nbytes;
   s->data[nbytes] = '\0';
 #ifdef GC_CHECK_STRING_OVERRUN
-  memcpy (data + needed, string_overrun_cookie, GC_STRING_OVERRUN_COOKIE_SIZE);
+  memcpy ((char *) data + needed, string_overrun_cookie,
+         GC_STRING_OVERRUN_COOKIE_SIZE);
 #endif
 
   /* If S had already data assigned, mark that as free by setting its