- reinstate paranoia check in wrapf().

1 files changed, 3 insertions, 2 deletions

diff --git a/coreutils/stty.c b/coreutils/stty.c
index 3ce8bf3..4e665bc 100644
--- a/coreutils/stty.c
+++ b/coreutils/stty.c
@@ -462,8 +462,9 @@ static void wrapf(const char *message, ...)
 	va_start(args, message);
 	buflen = vsnprintf(buf, sizeof(buf), message, args);
 	va_end(args);
-	/* buflen = strlen(buf); cheaper not to pull in strlen */
-	if (!buflen /*|| buflen >= sizeof(buf)*/) return;
+	/* We seem to be called only with suitable lengths, but check if
+	   somebody failed to adhere to this assumption just to be sure.  */
+	if (!buflen || buflen >= sizeof(buf)) return;
 
 	if (current_col > 0) {
 		current_col++;