diff options
author | Kristaps Dzonsons <kristaps@bsd.lv> | 2010-01-07 10:24:43 +0000 |
---|---|---|
committer | Kristaps Dzonsons <kristaps@bsd.lv> | 2010-01-07 10:24:43 +0000 |
commit | ad3053cbf11882fc2478adedbd82aadb0b6f9ca9 (patch) | |
tree | 18b2912353652906bf4d1c4eb4baf77b0faf88ed /man.c | |
parent | 65a5d430e92f93dd778bf5b4aaabe7e52238d532 (diff) | |
download | mandoc-ad3053cbf11882fc2478adedbd82aadb0b6f9ca9.tar.gz |
Check for white-space at the end of free-form text. Lack of check spotted by Jason McIntyre.
Diffstat (limited to 'man.c')
-rw-r--r-- | man.c | 26 |
1 files changed, 23 insertions, 3 deletions
@@ -365,6 +365,7 @@ static int man_ptext(struct man *m, int line, char *buf) { int i, j; + char sv; /* Literal free-form text whitespace is preserved. */ @@ -379,7 +380,11 @@ man_ptext(struct man *m, int line, char *buf) for (i = 0; ' ' == buf[i]; i++) /* Skip leading whitespace. */ ; - if (0 == buf[i]) { + if ('\0' == buf[i]) { + /* Trailing whitespace? */ + if (i && ' ' == buf[i - 1]) + if ( ! man_pwarn(m, line, i - 1, WTSPACE)) + return(0); if ( ! pstring(m, line, 0, &buf[i], 0)) return(0); goto descope; @@ -393,15 +398,30 @@ man_ptext(struct man *m, int line, char *buf) if (i && ' ' == buf[i] && '\\' == buf[i - 1]) continue; - buf[i++] = 0; + sv = buf[i]; + buf[i++] = '\0'; + if ( ! pstring(m, line, j, &buf[j], (size_t)(i - j))) return(0); + /* Trailing whitespace? Check at overwritten byte. */ + + if (' ' == sv && '\0' == buf[i]) + if ( ! man_pwarn(m, line, i - 1, WTSPACE)) + return(0); + for ( ; ' ' == buf[i]; i++) /* Skip trailing whitespace. */ ; j = i; - if (0 == buf[i]) + + /* Trailing whitespace? */ + + if (' ' == buf[i - 1] && '\0' == buf[i]) + if ( ! man_pwarn(m, line, i - 1, WTSPACE)) + return(0); + + if ('\0' == buf[i]) break; } |