* coding.c (make_subsidiaries): Don't assume string length fits in int.

diff --git a/src/ChangeLog b/src/ChangeLog
index bcf40a4e470430f0087f80ac8c37212522f22695..99d9cad7628b109aa321765de2f50c19b09371df 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,7 @@
 2011-06-19  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * coding.c (make_subsidiaries): Don't assume string length fits in int.
+
        * callproc.c (child_setup): Rewrite to avoid two strlen calls.
 
        * process.c (Fformat_network_address): Use EMACS_INT, not EMACS_UINT.

diff --git a/src/coding.c b/src/coding.c
index 04985ab3c7495a7b72b24d2b44f305cc7a34d997..9939774ea825d323e5a3253facf853dc9c10c04b 100644 (file)
--- a/src/coding.c
+++ b/src/coding.c
@@ -9442,7 +9442,7 @@ static Lisp_Object
 make_subsidiaries (Lisp_Object base)
 {
   Lisp_Object subsidiaries;
-  int base_name_len = SBYTES (SYMBOL_NAME (base));
+  ptrdiff_t base_name_len = SBYTES (SYMBOL_NAME (base));
   char *buf = (char *) alloca (base_name_len + 6);
   int i;
 
@@ -9450,7 +9450,7 @@ make_subsidiaries (Lisp_Object base)
   subsidiaries = Fmake_vector (make_number (3), Qnil);
   for (i = 0; i < 3; i++)
     {
-      memcpy (buf + base_name_len, suffixes[i], strlen (suffixes[i]) + 1);
+      strcpy (buf + base_name_len, suffixes[i]);
       ASET (subsidiaries, i, intern (buf));
     }
   return subsidiaries;