On Mon, Sep 12, 2016 at 05:01:49PM -0500, Andy Doan wrote:
> This makes it easier to traverse into a section of git repositories.
> 
> Signed-off-by: Andy Doan <andy.d...@linaro.org>
> ---
>  ui-repolist.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ui-repolist.c b/ui-repolist.c
> index 30915df..f6b6b47 100644
> --- a/ui-repolist.c
> +++ b/ui-repolist.c
> @@ -322,7 +322,7 @@ void cgit_print_repolist(void)
>                    strcmp(section, last_section)))) {
>                       htmlf("<tr class='nohover'><td colspan='%d' 
> class='reposection'>",
>                             columns);
> -                     html_txt(section);
> +                     htmlf("<a href='%s'>%s</a>", section, section);

I probably should have spotted this before, but "section" isn't being
properly escaped after this patch.  This needs to be:

        html("<a href='");
        html_attr(section);
        html("'>");
        html_txt(section);
        html("</a>");

(although the first and last strings could be combined with the
surrounding statements).

>                       html("</td></tr>");
>                       last_section = section;
>               }
> -- 
> 2.7.4
_______________________________________________
CGit mailing list
CGit@lists.zx2c4.com
http://lists.zx2c4.com/mailman/listinfo/cgit

Reply via email to