aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar Lars Hjemli <hjemli@gmail.com>2007-10-25 04:14:44 (JST)
committerGravatar Lars Hjemli <hjemli@gmail.com>2007-10-27 16:05:40 (JST)
commit13d2b0b25cf19de03c7cdcf8bb708be1168aeda7 (patch)
tree11241d302395aaffc2eca097465f5e019a123601
parentfd39489f4f49f9efe45c881d3f5f503c0068148a (diff)
downloadcgit-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.c6
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)