patch 9.1.1021: string might be used without a trailing NUL

John Marriott · chrisbra · commit 70dfc374ec72 · 2025-01-16T18:58:20.000+01:00
Problem:  string might be used without a trailing NUL (after v9.1.0997)
Solution: Make sure that the buffer is NUL terminated

closes: #16457

Signed-off-by: John Marriott &lt;basilisk@internode.on.net&gt;
Signed-off-by: Christian Brabandt &lt;cb@256bit.org&gt;
diff --git a/src/drawscreen.c b/src/drawscreen.c
@@ -479,7 +479,10 @@ win_redr_status(win_T *wp, int ignore_pum UNUSED)
 		    || bufIsChanged(wp->w_buffer)
 		    || wp->w_buffer->b_p_ro)
 		&& plen < MAXPATHL - 1)
-	    *(p + plen++) = ' ';
+	{
+	    *(p + plen++) = ' ';	// replace NUL with space
+	    *(p + plen) = NUL;		// NUL terminate the string
+	}
 	if (bt_help(wp->w_buffer))
 	    plen += vim_snprintf((char *)p + plen, MAXPATHL - plen, "%s", _("[Help]"));
 #ifdef FEAT_QUICKFIX
diff --git a/src/version.c b/src/version.c
@@ -704,6 +704,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    1021,
 /**/
     1020,
 /**/
