(substr): Don't allocate a byte for trailing NUL in result

since the result needn't be NUL-terminated.  Don't NUL terminate it.

1 files changed, 1 insertions, 3 deletions

diff --git a/src/tr.c b/src/tr.c
index 03a8a2984..b1f701dbe 100644
--- a/src/tr.c
+++ b/src/tr.c
@@ -811,13 +811,11 @@ substr (p, first_idx, last_idx)
      int last_idx;
 {
   int len = last_idx - first_idx + 1;
-  unsigned char *tmp = (unsigned char *) xmalloc (len + 1);
+  unsigned char *tmp = (unsigned char *) xmalloc (len);
 
   assert (first_idx <= last_idx);
   /* Use memcpy rather than strncpy because `p' may contain zero-bytes.  */
   memcpy (tmp, p + first_idx, len);
-  /* FIXME: then why nul-terminate.  */
-  tmp[len] = '\0';
   return tmp;
 }