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>

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
@@ -228,16 +228,19 @@ struct commitinfo *cgit_parse_commit(struct commit *commit)
                 ret->committer = substr(p, t);
                 p = t;
                 t = strchr(t, '>') + 1;
                 ret->committer_email = substr(p, t);
                 ret->committer_date = atol(++t);
                 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;
 
         t = strchr(p, '\n');
         if (t) {
                 if (*t == '\0')
                         ret->subject = strdup("** empty **");
                 else
@@ -287,15 +290,18 @@ struct taginfo *cgit_parse_tag(struct tag *tag)
                         p = t;
                         t = strchr(t, '>') + 1;
                         ret->tagger_email = substr(p, t);
                         ret->tagger_date = atol(++t);
                 }
                 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)
                 ret->msg = xstrdup(p);
         free(data);
         return ret;
 }