5 files changed, 15 insertions, 8 deletions

diff --git a/cgit.h b/cgit.h
index f06a941..02cc26a 100644
--- a/cgit.h
+++ b/cgit.h
@@ -62,12 +62,14 @@ extern int   cgit_query_ofs;
 extern int htmlfd;
 
 extern void cgit_global_config_cb(const char *name, const char *value);
 extern void cgit_repo_config_cb(const char *name, const char *value);
 extern void cgit_querystring_cb(const char *name, const char *value);
 
+extern void *cgit_free_commitinfo(struct commitinfo *info);
+
 extern char *fmt(const char *format,...);
 
 extern void html(const char *txt);
 extern void htmlf(const char *format,...);
 extern void html_txt(char *txt);
 extern void html_attr(char *txt);
diff --git a/shared.c b/shared.c
index 4574120..b576df8 100644
--- a/shared.c
+++ b/shared.c
@@ -85,6 +85,16 @@ void cgit_querystring_cb(const char *name, const char *value)
                 cgit_query_has_sha1 = 1;
         } else if (!strcmp(name, "ofs")) {
                 cgit_query_ofs = atoi(value);
         }
 }
 
+void *cgit_free_commitinfo(struct commitinfo *info)
+{
+        free(info->author);
+        free(info->author_email);
+        free(info->committer);
+        free(info->committer_email);
+        free(info->subject);
+        free(info);
+        return NULL;
+}
diff --git a/ui-commit.c b/ui-commit.c
index b441e14..f654208 100644
--- a/ui-commit.c
+++ b/ui-commit.c
@@ -58,11 +58,8 @@ void cgit_print_commit(const char *hex)
         html("<div class='commit-subject'>");
         html_txt(info->subject);
         html("</div>");
         html("<div class='commit-msg'>");
         html_txt(info->msg);
         html("</div>");
-        free(info->author);
-        free(info->committer);
-        free(info->subject);
-        free(info);
+        cgit_free_commitinfo(info);
 }
diff --git a/ui-log.c b/ui-log.c
index c52af79..b7da49f 100644
--- a/ui-log.c
+++ b/ui-log.c
@@ -25,16 +25,13 @@ void print_commit(struct commit *commit)
         html_link_open(url, NULL, NULL);
         html_txt(info->subject);
         html_link_close();
         html("</td><td>");
         html_txt(info->author);
         html("</td></tr>\n");
-        free(info->author);
-        free(info->committer);
-        free(info->subject);
-        free(info);
+        cgit_free_commitinfo(info);
 }
 
 
 void cgit_print_log(const char *tip, int ofs, int cnt)
 {
         struct rev_info rev;
diff --git a/ui-summary.c b/ui-summary.c
index 8ff3642..5ddeee3 100644
--- a/ui-summary.c
+++ b/ui-summary.c
@@ -32,12 +32,13 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1,
                 html_link_open(url, NULL, NULL);
                 html_txt(info->subject);
                 html_link_close();
                 html("</td><td>");
                 html_txt(info->author);
                 html("</td></tr>\n");
+                cgit_free_commitinfo(info);
         } else {
                 html("<tr><td>");
                 html_txt(buf);
                 html("</td><td>");
                 htmlf("*** bad ref %s", sha1_to_hex(sha1));
                 html("</td></tr>\n");