diff options
author | Lars Hjemli <hjemli@gmail.com> | 2007-10-25 04:14:44 (JST) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-10-27 16:05:40 (JST) |
commit | 13d2b0b25cf19de03c7cdcf8bb708be1168aeda7 (patch) | |
tree | 11241d302395aaffc2eca097465f5e019a123601 | |
parent | fd39489f4f49f9efe45c881d3f5f503c0068148a (diff) | |
download | cgit-13d2b0b25cf19de03c7cdcf8bb708be1168aeda7.zip cgit-13d2b0b25cf19de03c7cdcf8bb708be1168aeda7.tar.gz |
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
@@ -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) |