ParserNG: fix typo in rev.13688

Squid xstrncpy() assumes \0-termination and enforces it by overwriting
the last byte of the output. Un-terminated strings must have +1 length
delivered or they will be truncated.

diff --git a/src/http/one/Parser.cc b/src/http/one/Parser.cc
index 07696c80cf2af7571f3aa84a27b837b84844ee9e..10ed3603dc71cc34bb45e4507bc6f6fe96fbbce3 100644 (file)
--- a/src/http/one/Parser.cc
+++ b/src/http/one/Parser.cc
@@ -71,7 +71,7 @@ Http::One::Parser::getHeaderField(const char *name)
         p.chop(0, sizeof(header)-1);
 
         // return the header field-value
-        xstrncpy(header, p.rawContent(), p.length());
+        xstrncpy(header, p.rawContent(), p.length()+1);
         debugs(25, 5, "returning " << header);
         return header;
     }