[Bug 24325] New: MSXML Parser corrupts line breaks
http://bugs.winehq.org/show_bug.cgi?id=24325 Summary: MSXML Parser corrupts line breaks Product: Wine Version: 1.1.42 Platform: x86-64 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: msxml3 AssignedTo: wine-bugs(a)winehq.org ReportedBy: chris(a)generalpurposedomain.com Created an attachment (id=30657) --> (http://bugs.winehq.org/attachment.cgi?id=30657) Corrupt xml file If the parser is set to auto indent lines, the \r line breaks are replaced by " ", resulting in an invalid XML file. The \n breaks are written fine, though. I wrote a testcase in delphi, i attached both the testcase and the resulting XML file. If I install winetricks msxml3, the problem disappears, and files are written fine. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 --- Comment #1 from Austin English <austinenglish(a)gmail.com> 2010-09-09 14:34:30 CDT --- Please attach a +msxml trace (with builtin, not native). -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 --- Comment #2 from Christopher Bittner <chris(a)generalpurposedomain.com> 2010-09-09 14:34:56 CDT --- Created an attachment (id=30659) --> (http://bugs.winehq.org/attachment.cgi?id=30659) Testcase written in delphi -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 --- Comment #3 from Christopher Bittner <chris(a)generalpurposedomain.com> 2010-09-09 14:47:01 CDT --- Created an attachment (id=30661) --> (http://bugs.winehq.org/attachment.cgi?id=30661) msxml trace -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Christopher Bittner <chris(a)generalpurposedomain.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #30661|application/octet-stream |text/plain mime type| | -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, testcase -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Andrew Nguyen <arethusa26(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 --- Comment #4 from Andrew Nguyen <arethusa26(a)gmail.com> 2010-09-09 14:57:50 CDT --- I see the problem in wine-1.3.2-157-g2c4b081. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 ocean04(a)suomi24.fi changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ocean04(a)suomi24.fi --- Comment #5 from ocean04(a)suomi24.fi 2011-06-27 06:55:08 CDT --- Still in 1.3.23 -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 --- Comment #6 from ocean04(a)suomi24.fi 2011-07-03 10:58:19 CDT --- Looked this and here is what I found. First and last line of saved XML doesn't contain \r, it was stripped. domdoc.c: domdoc_save ctx = xmlSaveToIO(domdoc_save_writecallback, domdoc_save_closecallback, handle, NULL, XML_SAVE_NO_DECL); If change that to: ctx = xmlSaveToIO(domdoc_save_writecallback, domdoc_save_closecallback, handle, "UTF-8", XML_SAVE_NO_DECL); -> is replaced with Not sure if related, but read answer: http://mail.gnome.org/archives/xml/2011-June/msg00013.html Maybe just strip remaining \r in "node_set_content"? Not exactly same XML as Windows, but \r may not be possible using libxml2.. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #30657|text/xml |text/plain mime type| | -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Kyle Auble <randomidman48(a)yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |randomidman48(a)yahoo.com -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Hans Leidekker <hans(a)meelstraat.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dank(a)kegel.com --- Comment #7 from Hans Leidekker <hans(a)meelstraat.net> 2012-09-28 02:58:00 CDT --- *** Bug 31215 has been marked as a duplicate of this bug. *** -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 --- Comment #8 from Nikolay Sivov <bunglehead(a)gmail.com> 2012-10-08 00:21:57 CDT --- Created attachment 42021 --> http://bugs.winehq.org/attachment.cgi?id=42021 patch Hans, could you try this one to see if it works for .net installer? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 --- Comment #9 from Hans Leidekker <hans(a)meelstraat.net> 2012-10-08 03:04:57 CDT --- (In reply to comment #8)
Created attachment 42021 [details] patch
Hans, could you try this one to see if it works for .net installer?
Yes, that works. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 --- Comment #10 from Nikolay Sivov <bunglehead(a)gmail.com> 2012-11-12 21:49:02 CST --- Workaround is committed as 47d2f3caf56eb6c06b1d8f6d0f07b9815da685e6, please retest. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx(a)gmail.com --- Comment #11 from Bruno Jesus <00cpxxx(a)gmail.com> 2012-12-16 14:16:43 CST --- The test case from comment 2 produces the following output: Windows: Line 1: <?xml version="1.0"?> Line 2: <settings> Line 3: <setting/> Line 4: </settings> Line 5: Line 5 is empty. Wine: Line 1: <settings> Line 2: <setting/> Line 3: </settings> The output is different but the or is not present so I guess it's fixed. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Kyle Auble <randomidman48(a)yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|randomidman48(a)yahoo.com | -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #12 from Bruno Jesus <00cpxxx(a)gmail.com> 2013-03-26 00:20:51 CDT --- Still works in wine 1.5.26, assuming fixed as the testcase works and the other bug closed as duplicate also works. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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.
http://bugs.winehq.org/show_bug.cgi?id=24325 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #13 from Alexandre Julliard <julliard(a)winehq.org> 2013-03-29 13:32:47 CDT --- Closing bugs fixed in 1.5.27. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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