[Bug 26815] New: Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Summary: Mini Golf cannot be played a second time
           Product: Wine
           Version: 1.3.18
          Platform: x86
               URL: http://www.myrealgames.com/download-free-games/mini-go
                    lf/
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Starting the first time works but closing and starting up again doesn't

All that happens in the terminal is this

fixme:msxml:domelem_setAttributeNode (0x1319e0)->(0x1319cc 0x32ee14): semi-stub
fixme:msxml:domelem_setAttributeNode (0x131a10)->(0x1319fc 0x32ee14): semi-stub
fixme:msxml:domelem_setAttributeNode (0x1319c0)->(0x131a34 0x32ee30): semi-stub

...........

With Runtime Error! that pops up.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Andrew Millington  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #1 from Austin English  2011-04-18 01:32:54 CDT ---
(In reply to comment #0)
> Starting the first time works but closing and starting up again doesn't
>
> All that happens in the terminal is this
>
> fixme:msxml:domelem_setAttributeNode (0x1319e0)->(0x1319cc 0x32ee14): semi-stub
> fixme:msxml:domelem_setAttributeNode (0x131a10)->(0x1319fc 0x32ee14): semi-stub
> fixme:msxml:domelem_setAttributeNode (0x1319c0)->(0x131a34 0x32ee30): semi-stub
>
> ...........
>
> With Runtime Error! that pops up.

Does 'winetricks -q msxml3' help?

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #2 from Andrew Millington  2011-04-18 02:38:38 CDT ---
Just silences the fixmes that's all.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Andrew Millington  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|-unknown                    |msxml3

--- Comment #3 from Andrew Millington  2011-04-18 05:03:48 CDT ---
'winetricks -q msxml3' on the first run makes future runs possible.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #4 from Austin English  2011-04-18 13:13:25 CDT ---
(In reply to comment #3)
> 'winetricks -q msxml3' on the first run makes future runs possible.

Please attach +msxml,+tid trace on the first run with builtin msxml3.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1

--- Comment #5 from Ken Sharp  2011-04-18 16:28:46 CDT ---
Confirming

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #6 from Ken Sharp  2011-04-18 16:30:03 CDT ---
Created an attachment (id=34180)
 --> (http://bugs.winehq.org/attachment.cgi?id=34180)
Wine 1.3.18 +msxml +tid (first run) [48MB]

This is 48MB uncompressed!

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #7 from Ken Sharp  2011-04-18 16:30:40 CDT ---
Created an attachment (id=34181)
 --> (http://bugs.winehq.org/attachment.cgi?id=34181)
Wine 1.3.18 +msxml +tid (second run) [3MB]

Don't know if this is any use.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #8 from Alexandre Julliard  2011-04-18 17:09:39 CDT ---
The bug is that it corrupts the config/minigolf.cfg.xml file.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #9 from Andrew Millington  2011-07-07 23:35:00 CDT ---
Still present in wine-1.3.23-318-g1a74194
It crashes with a Microsoft Visual C++ Runtime error

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #10 from Nikolay Sivov  2011-11-21 13:39:54 CST ---
Please attach +msxml,+tid with current wine for git, and two xml files from
config/minigolf.cfg.xml. One created/altered with builtin version and another
one when you install native msxml3 with winetricks before you start this game
for a first time (it's ok for to have all this in a single archive).

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #11 from Andrew Millington  2011-11-21 18:28:17 CST ---
Still present in wine-1.3.32
Logs soon to come.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #12 from Andrew Millington  2011-11-21 18:44:41 CST ---
Created attachment 37568
  --> http://bugs.winehq.org/attachment.cgi?id=37568
+msxml,+tid log with the original and altered version of the configuration
file.

Hopefully I haven't messed up this.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #13 from Andrew Millington  2011-11-21 18:46:18 CST ---
The above attachment is using wine-1.3.32

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #14 from Andrew Millington  2011-11-21 19:30:32 CST ---
Created attachment 37569
  --> http://bugs.winehq.org/attachment.cgi?id=37569
wine-1.3.32(+msxml,+tid) with ./winetricks -q msxml3

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #15 from Nikolay Sivov  2011-11-22 13:23:05 CST ---
Created attachment 37583
  --> http://bugs.winehq.org/attachment.cgi?id=37583
patch

Attach +msxml,+tid with this patch over current git. I need a log from a first
run when you still have a valid configuration file.

If it's simple as that it's trivial to fix.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #16 from Andrew Millington  2011-11-23 04:29:33 CST ---
Created attachment 37590
  --> http://bugs.winehq.org/attachment.cgi?id=37590
Latest git with applied patch on first run and has some noise due to a libpng
problem

Sadly it did not work and the config file is still corrupted.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #17 from Nikolay Sivov  2011-11-23 09:12:31 CST ---
(In reply to comment #16)
> Sadly it did not work and the config file is still corrupted.

It wasn't supposed to fix it, the only important output is :

---
0009:trace:msxml:domdoc_save (0x136638)->({VT_UNKNOWN: 0x9b63b8})
0009:trace:msxml:domdoc_save using IStream
---

so in every case it uses IStream that we support. I thought it's something
obvious with saving failure. I'll test myself, thanks for a log.

P.S. please use commonly used compressors for attachments, like bzip2 or
something similar.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

--- Comment #18 from Nikolay Sivov  2012-07-07 01:46:38 CDT ---
The problem here is that we don't output '<?xml ... ?>' node. Application has
its own logic to encode resulting file after it's been saved to stream by
msxml3. If libxml2 writes starting prolog (+some additional tracing) log shows:

---
0009:trace:file:wine_nt_to_unix_file_name
L"\??\C:\PROG~FBU\MYRE~2QS.COM\MINI~UDL\config\MiniGolf.cfg.xml" ->
"/home/nsivov/.wine/dosdevices/c:/Program Files/MyRealGames.com/Mini
Golf/config/minigolf.cfg.xml"
0009:trace:file:CreateFileW returning 0xc
0009:trace:msxml:domdoc_save (0x12a2c0)->({VT_UNKNOWN: 0x9b4388})
0009:trace:msxml:domdoc_stream_save_writecallback 0x9b4388 0x7d884268 4764
2381. data="\xff\xfe<\x00?\x00x\x00m\x00l\x00
\x00v\x00e\x00r\x00s\x00i\x00o\x00n\x00=\x00"\x001\x00.\x000\x00"\x00
\x00e\x00n\x00c\x00o\x00d\x00i\x00n\x00g\x00=\x00"\x00U\x00T\x00F\x00-\x001\x006\x00"\x00?\x00>\x00\n\x00<\x00r\x00o\x00o\x00t\x00>\x00<\x00r\x00e\x00n\x00d\x00e\x00r\x00e\x00r\x00
\x00m\x00o\x00"...
0024:trace:file:WriteFile 0xc 0xc9d968 1 0xc9d948 (nil)
0009:trace:msxml:domdoc_stream_save_writecallback 0x9b4388 0x7d878f18 0 0.
data=""
0024:trace:file:WriteFile "<"
0009:trace:msxml:domdoc_stream_save_writecallback 0x9b4388 0x7d878f18 0 0.
data=""
0009:trace:file:WriteFile 0xc 0x32e60c 2965 0x32e5ec (nil)
0009:trace:file:WriteFile "?xml version="1.0"
encoding="windows-1251"?>\r\n<root>\r\n  <renderer\r\n   
models_lod="0"\r\n    disable="false"/>\r\n  <app\r\n    dt="0.032"\r\n  
 fixed_rand_seed="false"\r\n    rand_seed="0"\r\n   
sleep_after_tick="0"\r\n    physics_dt="0.002"/>\r\n  <screen\r\n   
lx="1024""... 
---

So callback writes it as UTF-16 and after that stream implementation flushes it
to file (in separate thread) with altered encoding set to 'windows-1251'.

Probably with no encoding specified application encoder don't know what to do.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34180|0                           |1
        is obsolete|                            |

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34181|0                           |1
        is obsolete|                            |

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #37583|0                           |1
        is obsolete|                            |

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #37590|0                           |1
        is obsolete|                            |

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |46d08bb6cefa6372ef3f597a130
                   |                            |d14ee62c662dc
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #19 from Nikolay Sivov  2012-07-09 15:12:33 CDT ---
Fixed by 46d08bb6cefa6372ef3f597a130d14ee62c662dc.

[Bug 26815] Mini Golf cannot be played a second time

http://bugs.winehq.org/show_bug.cgi?id=26815

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #20 from Alexandre Julliard  2012-07-17 13:52:36 CDT ---
Closing bugs fixed in 1.5.9.