[Dillo-dev] Check for unterminated quoted strings

6 Sep 2003

      I found this tag on pages at www.reichelt.de:

<A HREF="Info.html?P1mqhKwQAQsAAHVmTEY+Suche+0+50+LM%20324%20D%20SMD+lm324" onmouseover="return overlib('<TABLE><TR><TD><font face=\'Verdana, Arial, Helvetica, sans-serif\' size=1 color=\'#FFFFFF\'></font></TD></TR></TABLE>',CAPTION,'LM 324 D SMD');" onMouseout="return nd();">

It looks like correct html to me or am I wrong?
But dillo does not render it correctly. It is rendered like this:
<A HREF=... onmouseover="...TABLE"><TR><TD>...
So the check for unterminated quoted strings confuses the table handling. 
With a small change to html.c, it will be rendered correctly:
                   /* Skip over quoted string */
                   buf_index++;
                   buf_index += strcspn(buf + buf_index,
-                                       (ch == '"') ? "\">" : "'>");
+                                       (ch == '"') ? "\"" : "'");
                   if ( buf[buf_index] == '>' ) {
                      /* Unterminated string value? Let's look ahead and test:
                       * (<: unterminated, closing-quote: terminated) */

I don't know why that check is there. Does HTML allow quoted strings to be not
terminated? I guess there is a better solution than mine.

Greetings
Andreas Kemnade