diff options
author | Lars Hjemli | 2007-10-24 21:14:44 +0200 |
---|---|---|
committer | Lars Hjemli | 2007-10-27 09:05:40 +0200 |
commit | 13d2b0b25cf19de03c7cdcf8bb708be1168aeda7 (patch) | |
tree | 11241d302395aaffc2eca097465f5e019a123601 | |
parent | correct typo in CSS (diff) | |
download | cgit-13d2b0b25cf19de03c7cdcf8bb708be1168aeda7.tar.gz cgit-13d2b0b25cf19de03c7cdcf8bb708be1168aeda7.zip |
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) | |||
233 | p = strchr(t, '\n') + 1; | 233 | p = strchr(t, '\n') + 1; |
234 | } | 234 | } |
235 | 235 | ||
236 | while (*p && (*p != '\n')) | ||
237 | p = strchr(p, '\n') + 1; // skip unknown header fields | ||
238 | |||
236 | while (*p == '\n') | 239 | while (*p == '\n') |
237 | p = strchr(p, '\n') + 1; | 240 | p = strchr(p, '\n') + 1; |
238 | 241 | ||
@@ -292,6 +295,9 @@ struct taginfo *cgit_parse_tag(struct tag *tag) | |||
292 | p = strchr(p, '\n') + 1; | 295 | p = strchr(p, '\n') + 1; |
293 | } | 296 | } |
294 | 297 | ||
298 | while (p && *p && (*p != '\n')) | ||
299 | p = strchr(p, '\n') + 1; // skip unknown tag fields | ||
300 | |||
295 | while (p && (*p == '\n')) | 301 | while (p && (*p == '\n')) |
296 | p = strchr(p, '\n') + 1; | 302 | p = strchr(p, '\n') + 1; |
297 | if (p && *p) | 303 | if (p && *p) |