Bug 165597
| Summary: | When using contains Korean chraracter then crash `location.hash` value. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Jung Yong Woo <stegano> |
| Component: | Platform | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Major | CC: | achristensen, ap |
| Priority: | P2 | ||
| Version: | Safari 10 | ||
| Hardware: | All | ||
| OS: | All | ||
Jung Yong Woo
Summary:
When using contains Korean chraracter then crash `location.hash` value.
Steps to Reproduce:
1. install Safari
2. launch Safari
3. move to https://apple.com(or another site)
4. Top menu -> for developer -> Show error console
5. below javascript code input to console
location.hash = "한글 테스트" // input Korean character to location.hash;
console.log(location.hash); // you will show "#\" in console, but must be `한글 테스트` or `%ED%95%9C%EA%B8%80%20%ED%85%8C%EC%8A%A4%ED%8A%B8`.
Expected Results:
"#한글 테스트" or "#%ED%95%9C%EA%B8%80%20%ED%85%8C%EC%8A%A4%ED%8A%B8"
Actual Results:
"#\"
Version:
OS X
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Jung Yong Woo
I think it relate to https://bugs.webkit.org/show_bug.cgi?id=15182
Alex Christensen
I can reproduces this for the old URL::parse, but with the new URLParser this problem is fixed. It logs "%ED%95%9C%EA%B8%80% %ED%85%8C%EC%8A%A4%ED%8A%B8" and does not percent encode the space which matches Chrome and a recent change to the URL spec in https://github.com/whatwg/url/pull/169
Could you verify the fix in Safari Tech Preview? https://developer.apple.com/safari/technology-preview/
Jung Yong Woo
Thanks for your reply :)
It works well in Safari Technology Preview.
Alex Christensen
*** This bug has been marked as a duplicate of bug 162660 ***