Author: Christian Hesse <mail@eworm.de>
ui-shared: fix resource leak: free allocation from cgit_fileurl Coverity-id: 13918 Signed-off-by: Christian Hesse <mail@eworm.de>
ui-shared.c | 16 +++++++++++-----
diff --git a/ui-shared.c b/ui-shared.c index 89c48979586edaa7dc6d7df1aa133f288638b70e..d9f825853a273309860821ca08421fad3ed2cdd3 100644 --- a/ui-shared.c +++ b/ui-shared.c @@ -746,16 +746,19 @@ html_attr(ctx.cfg.favicon); html("'/>\n"); } if (host && ctx.repo && ctx.qry.head) { + char *fileurl; struct strbuf sb = STRBUF_INIT; strbuf_addf(&sb, "h=%s", ctx.qry.head); html("<link rel='alternate' title='Atom feed' href='"); html(cgit_httpscheme()); html_attr(cgit_hosturl()); - html_attr(cgit_fileurl(ctx.repo->url, "atom", ctx.qry.vpath, - sb.buf)); + fileurl = cgit_fileurl(ctx.repo->url, "atom", ctx.qry.vpath, + sb.buf); + html_attr(fileurl); html("' type='application/atom+xml'/>\n"); strbuf_release(&sb); + free(fileurl); } if (ctx.repo) cgit_add_clone_urls(print_rel_vcs_link); @@ -997,9 +1000,12 @@ cgit_stats_link("stats", NULL, hc("stats"), ctx.qry.head, ctx.qry.vpath); html("</td><td class='form'>"); html("<form class='right' method='get' action='"); - if (ctx.cfg.virtual_root) - html_url_path(cgit_fileurl(ctx.qry.repo, "log", - ctx.qry.vpath, NULL)); + if (ctx.cfg.virtual_root) { + char *fileurl = cgit_fileurl(ctx.qry.repo, "log", + ctx.qry.vpath, NULL); + html_url_path(fileurl); + free(fileurl); + } html("'>\n"); cgit_add_hidden_formfields(1, 0, "log"); html("<select name='qt'>\n");