summaryrefslogtreecommitdiff
path: root/trailer.c
diff options
context:
space:
mode:
authorJonathan Tan <jonathantanmy@google.com>2016-10-21 17:55:02 (GMT)
committerJunio C Hamano <gitster@pobox.com>2016-10-21 18:48:35 (GMT)
commitc463a6b28023dd3ad7ad4542147e20c27dbc83d6 (patch)
tree86aef84aedfa56c587b2480081f3b442ce3b2e3d /trailer.c
parent146245063e286d5a38b146bd5a38da958bd3957b (diff)
downloadgit-c463a6b28023dd3ad7ad4542147e20c27dbc83d6.zip
git-c463a6b28023dd3ad7ad4542147e20c27dbc83d6.tar.gz
git-c463a6b28023dd3ad7ad4542147e20c27dbc83d6.tar.bz2
trailer: forbid leading whitespace in trailers
Currently, interpret-trailers allows leading whitespace in trailer lines. This leads to false positives, especially for quoted lines or bullet lists. Forbid leading whitespace in trailers. Signed-off-by: Jonathan Tan <jonathantanmy@google.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'trailer.c')
-rw-r--r--trailer.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/trailer.c b/trailer.c
index 4c5b931..84105ac 100644
--- a/trailer.c
+++ b/trailer.c
@@ -775,7 +775,7 @@ static int find_trailer_start(struct strbuf **lines, int count)
}
separator_pos = find_separator(lines[start]->buf);
- if (separator_pos >= 1) {
+ if (separator_pos >= 1 && !isspace(lines[start]->buf[0])) {
struct list_head *pos;
trailer_lines++;