X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/253f3a96f00470bfc69ec3c5978bd152b708c255..4a78cbcd4fa986d9c1e7bd0a5a4bdb619faeb7cb:/sbr/fmt_scan.c

diff --git a/sbr/fmt_scan.c b/sbr/fmt_scan.c
index ab5940ab..9b84861c 100644
--- a/sbr/fmt_scan.c
+++ b/sbr/fmt_scan.c
@@ -175,11 +175,11 @@ cptrimmed(char **dest, char **ep, char *str, unsigned int wid, char fill,
 	     * widths, so use the column width to bump the end pointer when
 	     * appropriate.
 	     */
-	    if (char_len > 1 && epmax - *ep >= char_len - w) {
+	    if (w >= 0  &&  char_len > 1 && epmax - *ep >= char_len - w) {
 		*ep += char_len - w;
 	    }
 
-	    if (cp + w > *ep)
+	    if (w >= 0  &&  cp + w > *ep)
 		break;
 
 	    end -= char_len;
@@ -278,7 +278,7 @@ cpstripped (char **dest, char **end, char *max, char *str)
 	 * by the number of "extra" bytes in this character.  That's the
 	 * character length (char_len) minus the column width (w).
 	 */
-	if (char_len > 1  &&  max - *end >= char_len - w) {
+	if (w >= 0  &&  char_len > 1  &&  max - *end >= char_len - w) {
 	    *end += char_len - w;
 	}
 
@@ -467,29 +467,31 @@ fmt_scan (struct format *format, char *scanl, size_t max, int width, int *dat,
 		       scanl + max - 1);
 	    break;
 	case FT_STRLIT:
-	    sp = str;
-	    while ((c = *sp++) && cp < ep)
-		*cp++ = c;
+	    if (str) {
+		sp = str;
+		while ((c = *sp++) && cp < ep)
+		    *cp++ = c;
+	    }
 	    break;
-	case FT_STRLITZ: {
-	    size_t len = strlen (str);
-
-	    /* Don't want to emit part of an escape sequence.  So if
-	       there isn't enough room in the buffer for the entire
-	       string, skip it completely. */
-	    if (cp - scanl + len + 1 < max) {
-		for (sp = str; *sp; *cp++ = *sp++) continue;
-
-		/* This string doesn't count against the width.	 So
-		   increase ep the same amount as cp, only if the
-		   scan buffer will always be large enough. */
-		if (ep - scanl + len + 1 < max) {
-		    ep += len;
+	case FT_STRLITZ:
+	    if (str) {
+		size_t len = strlen (str);
+
+		/* Don't want to emit part of an escape sequence.  So if
+		   there isn't enough room in the buffer for the entire
+		   string, skip it completely. */
+		if (cp - scanl + len + 1 < max) {
+		    for (sp = str; *sp; *cp++ = *sp++) continue;
+
+		    /* This string doesn't count against the width.
+		       So increase ep the same amount as cp, only if the
+		       scan buffer will always be large enough. */
+		    if (ep - scanl + len + 1 < max) {
+			ep += len;
+		    }
 		}
 	    }
-
 	    break;
-	}
 	case FT_STRFW:
 	    adios (NULL, "internal error (FT_STRFW)");
 
@@ -1077,7 +1079,7 @@ fmt_scan (struct format *format, char *scanl, size_t max, int width, int *dat,
 	       there isn't enough room in the buffer for the entire
 	       string, skip it completely.  Need room for null
 	       terminator, and maybe trailing newline (added below). */
-	    if (cp - scanl + strlen (str) + 1 < max) {
+	    if (str && (cp - scanl + strlen (str) + 1 < max)) {
 		for (sp = str; *sp; *cp++ = *sp++) continue;
 	    }
 	    if (callbacks && callbacks->trace_func)