fix bug 3561, 'HTML <a> tag with no href makes message display incorrectly.'
authorPaul <paul@claws-mail.org>
Tue, 10 Nov 2015 15:20:23 +0000 (15:20 +0000)
committerPaul <paul@claws-mail.org>
Tue, 10 Nov 2015 15:20:23 +0000 (15:20 +0000)
based on patch by Andy Balaam

AUTHORS
src/gtk/authors.h
src/html.c

diff --git a/AUTHORS b/AUTHORS
index b75f349..12ed96a 100644 (file)
--- a/AUTHORS
+++ b/AUTHORS
@@ -307,3 +307,4 @@ contributors (in addition to the above; based on Changelog)
        Mikhail Kurinnoi
        Arthur Huillet
        Blatinox
+       Andy Balaam
index 4651705..d4aa975 100644 (file)
@@ -86,11 +86,12 @@ static char *CONTRIBS_LIST[] = {
 "André Filipe de Assunção e Brito",
 "Sergei Astanin",
 "Ruslan N. Balkin",
+"Andy Balaam",
 "Henri Bauer",
+"Didier Barvaux",
 "Fabio Júnior Beneditto",
 "M. Benkmann",
 "Wilbert Berendsen",
-"Didier Barvaux",
 "Sébastien Bigaret",
 "Laurent Bigonville",
 "Jean-Luc Biord",
index 19cb52c..72df946 100644 (file)
@@ -649,6 +649,7 @@ static SC_HTMLState sc_html_parse_tag(SC_HTMLParser *parser)
                parser->state = SC_HTML_BR;
        } else if (!strcmp(tag->name, "a")) {
                GList *cur;
+               parser->href = NULL;
                for (cur = tag->attr; cur != NULL; cur = cur->next) {
                        if (cur->data && !strcmp(((SC_HTMLAttr *)cur->data)->name, "href")) {
                                g_free(parser->href);
@@ -658,6 +659,8 @@ static SC_HTMLState sc_html_parse_tag(SC_HTMLParser *parser)
                                break;
                        }
                }
+               if (parser->href == NULL)
+                       parser->href = g_strdup("");
                parser->state = SC_HTML_HREF_BEG;
        } else if (!strcmp(tag->name, "/a")) {
                parser->state = SC_HTML_HREF;