https://bugs.winehq.org/show_bug.cgi?id=37217
Bug ID: 37217 Summary: wine builtin iexplore couldn't load local pages correctly in chinese named folders Product: Wine Version: 1.7.25 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: penghao@linuxdeepin.com
Created attachment 49487 --> https://bugs.winehq.org/attachment.cgi?id=49487 test files (has Chinese characters in the path name)
this bug was found in a finanical securities software. program downloaded some html/css/img contents from the internet, and show these contents in a pop window with builtin mshtml. But actually the pop window's mshtml just load the html file (like index.html), the img/css/js resources were not loaded. I tested builtin iexplore and found the same problem. finally I found there are some Chinese characters in the path. if I remove them or change them to English characters, It will be all fine.
Please see the test files in attached test.tar.gz. After uncompress it, use native ie to open index.html, for example: wine iexplore "file:///c:/测试用/index.html"
if css/js load success, the background color of the window will be black, text will be white, and you will see "you should see me!" which writen by js. if failed, you will just see black "Hello World" in a white background.
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #1 from Hao Peng penghao@linuxdeepin.com --- inmportant part found in log: trace:ieframe:navigate_url navigating to L"file:///c:\\6d4b\8bd5\7528\index.html" trace:mshtml:get_uri_string ret "file:///c:/%4B%D5%28/index.html" trace:mshtml:get_uri_string ret "file:///c:/K%D5(/css/base.css" trace:mshtml:get_uri_string ret "file:///c:/K%D5(/js/test.js" trace:ieframe:DocObjectService_FireNavigateComplete2 got URL L"file://c:\\6d4b\8bd5\7528\index.html"
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #2 from Hao Peng penghao@linuxdeepin.com --- Created attachment 49488 --> https://bugs.winehq.org/attachment.cgi?id=49488 log with mshtml,ieframe
https://bugs.winehq.org/show_bug.cgi?id=37217
Qian Hong fracting@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW CC| |fracting@gmail.com Ever confirmed|0 |1
--- Comment #3 from Qian Hong fracting@gmail.com --- Thanks for reporting, confirming :)
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #4 from Hao Peng penghao@linuxdeepin.com --- we found a bug in urlmon/url.c: static inline void pct_encode_val(WCHAR val, WCHAR *dest)
this function convert a unsafe character into a safe string starting with %, but it won't consider the condition of cjk characters in unicode, which will encode into three %xx part.like: 中 %E4%B8%AD
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #5 from Austin English austinenglish@gmail.com --- This is your friendly reminder that there has been no bug activity for over a year. Is this still an issue in current (1.7.51 or newer) wine?
https://bugs.winehq.org/show_bug.cgi?id=37217
Zhiyi Zhang yi.gd.cn@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |yi.gd.cn@gmail.com
--- Comment #6 from Zhiyi Zhang yi.gd.cn@gmail.com --- Issue still exist in wine-3.0
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #7 from Zhiyi Zhang yi.gd.cn@gmail.com --- Created attachment 60392 --> https://bugs.winehq.org/attachment.cgi?id=60392 staging patch
https://bugs.winehq.org/show_bug.cgi?id=37217
Vijay Kamuju infyquest@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |infyquest@gmail.com
--- Comment #8 from Vijay Kamuju infyquest@gmail.com --- Please send the patch to wine-devel@winehq.org for review and inclusion
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #9 from Zhiyi Zhang yi.gd.cn@gmail.com --- (In reply to Vijay Kamuju from comment #8)
Please send the patch to wine-devel@winehq.org for review and inclusion
Yes. I will send it later. It seems there are errors when tests run on Windows
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #10 from Zhiyi Zhang yi.gd.cn@gmail.com --- (In reply to Hao Peng from comment #4)
we found a bug in urlmon/url.c: static inline void pct_encode_val(WCHAR val, WCHAR *dest)
this function convert a unsafe character into a safe string starting with %, but it won't consider the condition of cjk characters in unicode, which will encode into three %xx part.like: 中 %E4%B8%AD
Thanks for the analysis. It's misleading however. I found the culprit ultimately lies in percent-encoding Unicode characters. Unicode characters should not be encoded, as they are on Windows.
https://bugs.winehq.org/show_bug.cgi?id=37217
--- Comment #11 from Zhiyi Zhang yi.gd.cn@gmail.com --- Fixed in ac066d3eee85fd6e3a27ca59b158c5e040d91727
https://bugs.winehq.org/show_bug.cgi?id=37217
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |ac066d3eee85fd6e3a27ca59b15 | |8c5e040d91727
--- Comment #12 from Nikolay Sivov bunglehead@gmail.com --- Marking fixed, ac066d3eee85fd6e3a27ca59b158c5e040d91727.
https://bugs.winehq.org/show_bug.cgi?id=37217
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |urlmon
https://bugs.winehq.org/show_bug.cgi?id=37217
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #13 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 3.2.
https://bugs.winehq.org/show_bug.cgi?id=37217
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x
https://bugs.winehq.org/show_bug.cgi?id=37217
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |---
--- Comment #14 from Michael Stefaniuc mstefani@winehq.org --- Removing the 3.0.x milestone from bugs included in 3.0.1.