commit e1863c05d82096aa2bfcddb436da1b54a41369e3
author Ray Donnelly <mingw.android@gmail.com> Sun Feb 02 10:51:05 2014 -0500
committer Paul Smith <psmith@gnu.org> Sat Feb 08 16:01:10 2014 -0500
tree c1a3ba1efc937120ca3a9c216999815ea4b8cf65
parent b981bfd197eb767e93739908e4ec49406c314894

* output.c: Ensure space for final nul byte in fmtbuf.

Copyright-paperwork-exempt: yes

output.c

diff --git a/output.c b/output.c
index 000e241..b2584f1 100644
--- a/output.c
+++ b/output.c
@@ -591,14 +591,14 @@
 static char *
 get_buffer (size_t need)
 {
-  /* Make sure we have room.  */
+  /* Make sure we have room.  NEED includes space for \0.  */
   if (need > fmtbuf.size)
     {
       fmtbuf.size += need * 2;
       fmtbuf.buffer = xrealloc (fmtbuf.buffer, fmtbuf.size);
     }
 
-  fmtbuf.buffer[need] = '\0';
+  fmtbuf.buffer[need-1] = '\0';
 
   return fmtbuf.buffer;
 }
@@ -629,7 +629,7 @@
 
   strcat (p, "\n");
 
-  assert (fmtbuf.buffer[len] == '\0');
+  assert (fmtbuf.buffer[len-1] == '\0');
   outputs (0, fmtbuf.buffer);
 }
 
@@ -660,7 +660,7 @@
 
   strcat (p, "\n");
 
-  assert (fmtbuf.buffer[len] == '\0');
+  assert (fmtbuf.buffer[len-1] == '\0');
   outputs (1, fmtbuf.buffer);
 }
 
@@ -692,7 +692,7 @@
 
   strcat (p, stop);
 
-  assert (fmtbuf.buffer[len] == '\0');
+  assert (fmtbuf.buffer[len-1] == '\0');
   outputs (1, fmtbuf.buffer);
 
   die (2);