THelpTopic: prevent out-of-bounds accesses in TParagraph::text

Valgrind reports the 'text' field of TParagraph being read one byte beyond the allocated memory in functions like scan() and isBlank(), for no obvious reason. The implementation is probably assuming strings are null-terminated when they are not. I guess the easiest workaround is to force a null terminator.

Author: magiblot

source/tvision/helpbase.cpp

@@ -103,15 +103,16 @@ void THelpTopic::readParagraphs( ipstream& s )
 
     s >> i;
     pp = &paragraphs;
-    while ( i > 0)
+    while (i > 0)
     {
         s >> size;
         *pp = new TParagraph;
-        (*pp)->text = new char[size];
+        (*pp)->text = new char[size + 1];
         (*pp)->size = (ushort) size;
-    s >> temp;
+        s >> temp;
         (*pp)->wrap = Boolean(temp);
         s.readBytes((*pp)->text, (*pp)->size);
+        (*pp)->text[(*pp)->size] = '\0';
         pp = &((*pp)->next);
         --i;
     }