[Bug 10911] New: Download manager doesn't work

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

Summary: Download manager doesn't work
           Product: Wine
           Version: CVS/GIT
          Platform: PC
               URL: http://d.trymedia.com/dm/knowad/15d120m10e_i_1f/trygames
                    /JSMCMastermind.exe
        OS/Version: Linux
            Status: UNCONFIRMED
          Keywords: download
          Severity: normal
          Priority: P2
         Component: wine-shdocvw
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Hi, tried to look into bug http://bugs.winehq.org/show_bug.cgi?id=10874, but
actually i was not able to download the demo with current wine. After starting
JSMCMastermind-dm.exe, the download of the demo should actually start, but all
i get is a weird window. See attached screenshot

[Bug 10911] Download manager doesn't work

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

--- Comment #1 from Louis Lenders   2007-12-27 15:03:34 ---
Created an attachment (id=9829)
 --> (http://bugs.winehq.org/attachment.cgi?id=9829)
the window mentioned above

[Bug 10911] Download manager doesn't work

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

Austin English  changed:

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

--- Comment #2 from Austin English   2007-12-27 15:15:33 ---
Confirming, I just saw this a few minutes ago for another demo (Worms of
Armageddon), but was a bit busy and didn't file a bug for it.

[Bug 10911] Download manager doesn't work

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

--- Comment #3 from Louis Lenders   2008-05-28 17:17:45 ---
Still present in current git

[Bug 10911] Download manager doesn't work

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

--- Comment #4 from Austin English   2008-12-07 18:10:14 ---
Doesn't even start for me in 1.1.10.

[Bug 10911] Download manager doesn't work

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|CVS/GIT                     |unspecified

--- Comment #5 from Austin English   2009-01-18 03:47:22 ---
Removing deprecated CVS/GIT version tag. Please retest in current git. If the
bug is still present in today's wine, but was not present in some earlier
version of wine, please update version field to earliest known version of wine
that had the bug. Thanks!

[Bug 10911] Download manager doesn't work

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

Niko Sandschneider  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #6 from Niko Sandschneider   2010-03-06 03:53:51 ---
In wine 1.1.40 JSMCMastermind-dm.exe still hangs right after start. Terminal
output is attached.

[Bug 10911] Download manager doesn't work

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

--- Comment #7 from Niko Sandschneider   2010-03-06 03:55:45 ---
Created an attachment (id=26635)
 --> (http://bugs.winehq.org/attachment.cgi?id=26635)
Terminal output in wine 1.1.40

[Bug 10911] Download manager doesn't work

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

Zubair  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

[Bug 10911] Download manager doesn't work

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

Bruno Jesus  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #8 from Bruno Jesus  2011-11-29 19:32:57 CST ---
Still present in 1.3.33. All I get is an infinite loop of
err:msxml:doparse XML declaration allowed only at the start of the document

winetricks msxml4 does not solve the issue.

[Bug 10911] Download manager doesn't work

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

--- Comment #9 from Nikolay Sivov  2011-11-30 13:05:32 CST ---
(In reply to comment #8)
> Still present in 1.3.33. All I get is an infinite loop of
> err:msxml:doparse XML declaration allowed only at the start of the document

If you trace something like +wininet you'll see response as "\n<?xml..."
probably. I remember we have a bug about that. msxml is failing correctly as
native does (and as specification suggest).
> winetricks msxml4 does not solve the issue.

[Bug 10911] Download manager doesn't work

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

--- Comment #10 from Bruno Jesus  2011-11-30 19:21:39 CST ---
Nikolay, you're completely right. Do you have any hints about what to do next?

trace:msxml:domdoc_loadXML (0x1189df8)->(L"\n<?xml version=\"1.0\"
encoding=\"UTF-16\"?>\n<dm>\n  <files>\n    <file execute="yes"
stamp="yes">\n     
<url>http://cdn.dli.trymedia.com/r/release/knowad/15d120m10e_i_1f/JSMCMastermind....</url>\n
   </file>\n  </files>\n</dm" 0x66e938)
err:msxml:doparse XML declaration allowed only at the start of the document

[Bug 10911] Download manager doesn't work

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

--- Comment #11 from Nikolay Sivov  2011-11-30 20:36:41 CST ---
We need to figure out where this broken stream comes from and why it contains
leading return character. I don't know details unfortunately, but as a start
you probably need +wininet log or something related, it's interesting to test
with native modules as well to be sure.

[Bug 10911] Download manager doesn't work

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

--- Comment #12 from Bruno Jesus  2011-11-30 20:58:06 CST ---
Created attachment 37717
  --> http://bugs.winehq.org/attachment.cgi?id=37717
tcp dump

As you can see in the attached dump file the site clearly sends an extra \n at
the beginning of the stream.

00a0   31 31 65 0d 0a 0a 3c 3f 78 6d 6c 20 76 65 72 73  11e...<?xml vers

I'll retest in windows later.

[Bug 10911] Download manager doesn't work

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

--- Comment #13 from Nikolay Sivov  2011-11-30 21:10:32 CST ---
Maybe we should strip all space chars when header is separated from request
body, or something like that, but yes, it's interesting to see a response
running windows too.

[Bug 10911] Download manager doesn't work

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

--- Comment #14 from Bruno Jesus  2011-11-30 21:16:55 CST ---
Created attachment 37718
  --> http://bugs.winehq.org/attachment.cgi?id=37718
+wininet,+msxml in 1.3.33

Attached a trimmed version of +wininet,+msxml.

[Bug 10911] Download manager doesn't work

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

--- Comment #15 from Bruno Jesus  2011-11-30 21:26:45 CST ---
Windows receives the same TCP data as expected. But the program goes on and
start downloading. If you can produce a small patch to trim everything before
the first "<" I'll be glad to test (I tried myself but didn't find where that
msxml error message is generated).

[Bug 10911] Download manager doesn't work

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

--- Comment #16 from Nikolay Sivov  2011-12-01 10:00:23 CST ---
(In reply to comment #15)
> Windows receives the same TCP data as expected. But the program goes on and
> start downloading.

Ok, that's good.
> If you can produce a small patch to trim everything before
> the first "<" I'll be glad to test (I tried myself but didn't find where that
> msxml error message is generated).

That is not a way to fix that. When msxml3 receive such broken stream it fails
as it should.

So the problem is somewhere in winhttp/wininet I think.

[Bug 10911] Download manager doesn't work

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #17 from [e-mail] 2011-12-16 07:20:39 CST ---
Download starts here if I do:

winetricks wininet ie6 msxml3

Remove any part and it doesn't work..

[Bug 10911] Download manager doesn't work

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

--- Comment #18 from Nikolay Sivov  2011-12-16 07:27:22 CST ---
(In reply to comment #17)
> Download starts here if I do:
>
> winetricks wininet ie6 msxml3
>
> Remove any part and it doesn't work..

Okay, could you attach +msxml,+tid running only 'winetricks wininet ie6'?

[Bug 10911] Download manager doesn't work

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

--- Comment #19 from [e-mail] 2011-12-16 07:41:37 CST ---
Created attachment 37988
  --> http://bugs.winehq.org/attachment.cgi?id=37988
+msxml,+tid after winetricks wininet ie6

[Bug 10911] Download manager doesn't work

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

--- Comment #20 from [e-mail] 2011-12-16 10:59:16 CST ---
This works in Windows / native msxml3

xml:=CreateOleObject('Microsoft.XMLDOM');
xml.LoadXML(#10+'<?xml version="1.0" encoding="UTF-8" ?><test>test</test>');
xml.Save('c:/testing.xml');

Saves 0 bytes with builtin msxml3

test sample: http://netikka.net/dev/xml2.exe

[Bug 10911] Download manager doesn't work

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

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]        |[e-mail]

--- Comment #21 from Nikolay Sivov  2011-12-16 11:27:18 CST ---
(In reply to comment #20)
> This works in Windows / native msxml3
>
> xml:=CreateOleObject('Microsoft.XMLDOM');
> xml.LoadXML(#10+'<?xml version="1.0" encoding="UTF-8" ?><test>test</test>');
> xml.Save('c:/testing.xml');

Yeah, I tested ::load() method only, and it fails on such stream, I'll more
tests.

[Bug 10911] Download manager doesn't work

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

--- Comment #22 from Nikolay Sivov  2011-12-16 17:06:01 CST ---
So your test program does:

---
0009:trace:msxml:DispatchEx_GetIDsOfNames
(0x134a30)->({00000000-0000-0000-0000-000000000000} 0x33eb04 1 1033 0x33eb48)
0009:trace:msxml:DispatchEx_GetDispID (0x134a30)->(L"LoadXML" 0 0x33eb48)
0009:trace:msxml:preprocess_dispex_data (0x134a30)
0009:trace:msxml:DispatchEx_Invoke (0x134a30)->(63
{00000000-0000-0000-0000-000000000000} 0 1 0x33eae0 (nil) 0x33eb10 (nil))
0009:trace:msxml:DispatchEx_InvokeEx (0x134a30)->(3f 0 1 0x33eae0 (nil)
0x33eb10 (nil))
0009:trace:msxml:domdoc_QueryInterface
(0x134a30)->({2933bf96-7b36-11d2-b20e-00c04f983e60} 0x33e400)
0009:trace:msxml:domdoc_AddRef (0x134a30)->(3)
0009:trace:msxml:domdoc_loadXML (0x134a30)->(L"\n<?xml version=\"1.0\"
encoding=\"UTF-8\" ?><test>test</test>" 0x33e0f8)
0009:err:msxml:doparse XML declaration allowed only at the start of the
document
0009:trace:msxml:domdoc_loadXML failed to parse document
---

and after that

---
0009:trace:msxml:DispatchEx_GetIDsOfNames
(0x134a30)->({00000000-0000-0000-0000-000000000000} 0x33eb04 1 1033 0x33eb48)
0009:trace:msxml:DispatchEx_GetDispID (0x134a30)->(L"Save" 0 0x33eb48)
0009:trace:msxml:DispatchEx_Invoke (0x134a30)->(64
{00000000-0000-0000-0000-000000000000} 0 1 0x33eae0 (nil) 0x33eb10 (nil))
0009:trace:msxml:DispatchEx_InvokeEx (0x134a30)->(40 0 1 0x33eae0 (nil)
0x33eb10 (nil))
0009:trace:msxml:domdoc_QueryInterface
(0x134a30)->({2933bf96-7b36-11d2-b20e-00c04f983e60} 0x33e400)
0009:trace:msxml:domdoc_AddRef (0x134a30)->(3)
0009:trace:msxml:domdoc_save (0x134a30)->({VT_BSTR: L"c:/testing.xml"})
---

same sequence in a test (well, with direct calls and not IDispatch ones)
results in empty file running on Windows. That's what I thought is going on,
but really the problem is all about different CLSIDs that are used to create an
instance.

Actually there's two CLSID_DOMDocument from different SDK headers with
different values, we use the one from msxml2.idl where it's just a define for
CLSID_DOMDocument2. But in xmldom.idl it's defined with proper class section
using separate IID value.

I'll think how to resolve this name mess.

[Bug 10911] Download manager doesn't work

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

--- Comment #23 from Nikolay Sivov  2011-12-17 15:34:36 CST ---
Created attachment 38007
  --> http://bugs.winehq.org/attachment.cgi?id=38007
patch

Please try this one for reported msxml issue.

[Bug 10911] Download manager doesn't work

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

--- Comment #24 from [e-mail] 2011-12-18 07:39:15 CST ---
Patch works here (for msxml issue)

[Bug 10911] Download manager doesn't work

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

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]        |[e-mail]

--- Comment #25 from Nikolay Sivov  2011-12-19 13:51:41 CST ---
msxml bug is fixed with e736c3e9837fc818ad5154e14be56fb09f11eef5.

[Bug 10911] Download manager doesn't work

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

--- Comment #26 from Bruno Jesus  2012-05-14 18:20:59 CDT ---
In wine 1.5.4 the program seems to start in the console, the xml is parsed
correctly but there is no progress window opened. Can anyone else test?

[Bug 10911] Download manager doesn't work

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

GyB  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #27 from GyB  2012-07-08 07:21:07 CDT ---
*** Bug 18814 has been marked as a duplicate of this bug. ***

[Bug 10911] Download manager doesn't work

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

GyB  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #28 from GyB  2012-07-08 07:32:41 CDT ---
I tested the download manager with Wine 1.5.8 and Gecko 1.6 installed.

- the application starts in the console, but it doesn't put up any windows
- after 'winetricks ie7' the application window appears, but much of its
contents are missing (empty progress bar, no text at all)
- native msxml3 is needed to make the contents of the window appear. After
several attempts, connecting to the server fails.
- native wininet.dll is needed to make a successful connection, and downloading
the actual game.

[Bug 10911] Download manager doesn't work

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

--- Comment #29 from Bruno Jesus  2013-03-26 00:07:58 CDT ---
Still in wine 1.5.26.

[Bug 10911] Jumpstart Mystery Club download manager doesn't work

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

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |e736c3e9837fc818ad5154e14be
                   |                            |56fb09f11eef5
             Status|NEW                         |RESOLVED
                 CC|                            |[e-mail]
          Component|shdocvw                     |msxml3
            Version|unspecified                 |1.1.40
         Resolution|---                         |FIXED
            Summary|Download manager doesn't    |Jumpstart Mystery Club
                   |work                        |download manager doesn't
                   |                            |work

--- Comment #30 from Anastasius Focht  ---
Hello folks,

don't recycle bugs.

--- quote ---
Nikolay Sivov 2011-12-19 13:51:41 CST

msxml bug is fixed with e736c3e9837fc818ad5154e14be56fb09f11eef5.
--- quote ---

Open a new bug if <insert whatever issue> is still present.

Please stick to _one_ issue and avoid multiple winetricks recipes (ie7, msxml,
wininet).
Also be sure to add _working_ downlink link.

Regards

[Bug 10911] Jumpstart Mystery Club download manager doesn't work

https://bugs.winehq.org/show_bug.cgi?id=10911

Alexandre Julliard  changed:

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

--- Comment #31 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.10.