diff options
| author | Lars Hjemli <hjemli@gmail.com> | 2007-10-24 21:14:44 +0200 |
|---|---|---|
| committer | Lars Hjemli <hjemli@gmail.com> | 2007-10-27 09:05:40 +0200 |
| commit | 13d2b0b25cf19de03c7cdcf8bb708be1168aeda7 (patch) | |
| tree | 11241d302395aaffc2eca097465f5e019a123601 | |
| parent | fd39489f4f49f9efe45c881d3f5f503c0068148a (diff) | |
Skip unknown header fields when parsing tags and commits
Both the commit- and tagparser failed to handle unexpected header fields. This adds futureproofing by simply skipping any header we don't know/care about. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| -rw-r--r-- | parsing.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/parsing.c b/parsing.c index 2c05c09..ddbafd5 100644 --- a/parsing.c +++ b/parsing.c @@ -233,6 +233,9 @@ struct commitinfo *cgit_parse_commit(struct commit *commit) p = strchr(t, '\n') + 1; } + while (*p && (*p != '\n')) + p = strchr(p, '\n') + 1; // skip unknown header fields + while (*p == '\n') p = strchr(p, '\n') + 1; @@ -292,6 +295,9 @@ struct taginfo *cgit_parse_tag(struct tag *tag) p = strchr(p, '\n') + 1; } + while (p && *p && (*p != '\n')) + p = strchr(p, '\n') + 1; // skip unknown tag fields + while (p && (*p == '\n')) p = strchr(p, '\n') + 1; if (p && *p) |