FULL PRODUCT VERSION :


A DESCRIPTION OF THE PROBLEM :
This bug is result of 6245596 "fix" .

<img src="http://www/test?a=b&c=d&e=f">
src will be changed to
<img src="http://www/test?a=b&c;=d&e;=f">

because of
String str = '&' + nm + ';';

and image will not be displayed.


My solution would be to add local variable suffix :

String suffix = "";
switch (ch) {
          case '\n':
            ln++;
            ch = readCh();
            lfCount++;
            break;

          case '\r':
            ln++;
            if ((ch = readCh()) == '\n') {
                ch = readCh();
                crlfCount++;
            }
            else {
                crCount++;
            }
            break;

          case ';':
suffix = ";";
            ch = readCh();
            break;
        }

and later

String str = '&' + nm + suffix;



STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
Run source code below and you will see wrong result. (with additional semicolons)


EXPECTED VERSUS ACTUAL BEHAVIOR :
EXPECTED -
<img src="http://www/test?a=b&c=d&e=f">
ACTUAL -
<img src="http://www/test?a=b&c;=d&e;=f">


REPRODUCIBILITY :
This bug can be reproduced always.

---------- BEGIN SOURCE ----------
import java.io.IOException;
import java.io.StringReader;
import java.util.Enumeration;

import javax.swing.text.MutableAttributeSet;
import javax.swing.text.html.HTML;
import javax.swing.text.html.HTMLEditorKit.ParserCallback;
import javax.swing.text.html.parser.ParserDelegator;

/**
 *
 */

/**
 * @author Milos
 *
 */
public class HtmlParserTest extends ParserCallback {
String text = "<img src=\"http://www/test?a=b&c=d&e=f\">";

    public void parse() throws IOException {
     StringReader reader = new StringReader(text);
        ParserDelegator delegator = new ParserDelegator();
        delegator.parse(reader, this, true); // the third parameter is TRUE to ignore charset directive
    }
    
    @Override
    public void handleSimpleTag(HTML.Tag t, MutableAttributeSet a, int pos) {
     StringBuilder sb = new StringBuilder();
sb.append("<" + t);
for (Enumeration<?> en = a.getAttributeNames(); en.hasMoreElements(); ) {
Object name = en.nextElement();
String nameStr = name.toString();
if (!nameStr.equals("_implied_") && !nameStr.equals("style")) {
sb.append(" ").append(nameStr).append("=\"").append(a.getAttribute(name)).append("\"");
}
}
sb.append(">");
System.out.println(sb);
    }
    
    public static void main(String[] args) throws IOException {
HtmlParserTest parser = new HtmlParserTest();
parser.parse();
}
}


---------- END SOURCE ----------