[Bug 37217] New: wine builtin iexplore couldn't load local pages correctly in chinese named folders
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(a)winehq.org Reporter: penghao(a)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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #1 from Hao Peng <penghao(a)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" -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #2 from Hao Peng <penghao(a)linuxdeepin.com> --- Created attachment 49488 --> https://bugs.winehq.org/attachment.cgi?id=49488 log with mshtml,ieframe -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Qian Hong <fracting(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW CC| |fracting(a)gmail.com Ever confirmed|0 |1 --- Comment #3 from Qian Hong <fracting(a)gmail.com> --- Thanks for reporting, confirming :) -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #4 from Hao Peng <penghao(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #5 from Austin English <austinenglish(a)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? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Zhiyi Zhang <yi.gd.cn(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |yi.gd.cn(a)gmail.com --- Comment #6 from Zhiyi Zhang <yi.gd.cn(a)gmail.com> --- Issue still exist in wine-3.0 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #7 from Zhiyi Zhang <yi.gd.cn(a)gmail.com> --- Created attachment 60392 --> https://bugs.winehq.org/attachment.cgi?id=60392 staging patch -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Vijay Kamuju <infyquest(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |infyquest(a)gmail.com --- Comment #8 from Vijay Kamuju <infyquest(a)gmail.com> --- Please send the patch to wine-devel(a)winehq.org for review and inclusion -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #9 from Zhiyi Zhang <yi.gd.cn(a)gmail.com> --- (In reply to Vijay Kamuju from comment #8)
Please send the patch to wine-devel(a)winehq.org for review and inclusion
Yes. I will send it later. It seems there are errors when tests run on Windows -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #10 from Zhiyi Zhang <yi.gd.cn(a)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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 --- Comment #11 from Zhiyi Zhang <yi.gd.cn(a)gmail.com> --- Fixed in ac066d3eee85fd6e3a27ca59b158c5e040d91727 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |ac066d3eee85fd6e3a27ca59b15 | |8c5e040d91727 --- Comment #12 from Nikolay Sivov <bunglehead(a)gmail.com> --- Marking fixed, ac066d3eee85fd6e3a27ca59b158c5e040d91727. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |urlmon -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #13 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 3.2. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37217 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |--- --- Comment #14 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 3.0.x milestone from bugs included in 3.0.1. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs@winehq.org