http://bugs.winehq.org/show_bug.cgi?id=11791
Summary: BiblePro installer crashes Product: Wine Version: CVS/GIT Platform: Other URL: http://bibleocean.com/default.aspx?UC=DOWNLOAD OS/Version: other Status: UNCONFIRMED Keywords: download, Installer, regression Severity: normal Priority: P2 Component: wininet AssignedTo: wine-bugs@winehq.org ReportedBy: xerox_xerox2000@yahoo.co.uk
Was about to try another .net app, but it crashes right away here. As stated in bug http://bugs.winehq.org/show_bug.cgi?id=10316, the installer got further before this regression. Must be one of those wininet patches that went in lately. I'll attach the crash
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #1 from Louis Lenders xerox_xerox2000@yahoo.co.uk 2008-03-01 00:49:06 --- Created an attachment (id=11054) --> (http://bugs.winehq.org/attachment.cgi?id=11054) crash log.
Odly it crashes in libX11. With native wininet all is fine again.
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #2 from Louis Lenders xerox_xerox2000@yahoo.co.uk 2008-03-01 02:39:41 --- 728e5fa55966987c3a2fdc96d8890987eb7b7e9d is first bad commit commit 728e5fa55966987c3a2fdc96d8890987eb7b7e9d Author: Misha Koshelev mk144210@bcm.edu Date: Thu Sep 20 20:59:40 2007 -0500
wininet: Track child handles, free all child handles on WININET_FreeHandle as native.
:040000 040000 dc8758a2c047971ae8c29339ac4aaac9f2b9e060 749aa9f8ee4ebb8fbecbdf4b7ea4fc95e89ee6f8 M dlls
hmm, looks like misha isn't subscribed to bugzilla, couldn't add him in the cc-field ...
http://bugs.winehq.org/show_bug.cgi?id=11791
Misha Koshelev mk144210@bcm.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mk144210@bcm.edu
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #3 from Austin English austinenglish@gmail.com 2008-03-02 22:05:23 ---
hmm, looks like misha isn't subscribed to bugzilla, couldn't add him in the cc-field ...
I sent him an e-mail about it, haven't heard back yet.
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #4 from Misha Koshelev mk144210@bcm.edu 2008-03-03 00:08:34 --- Hi sorry I am taking a look at it as we speak. I have a new 64 bit system and am getting wine git working on it. Right now it looks like I am getting a differ error but I need to double check something first. If I keep getting the different error I will post that here, otherwise I will begin to work on getting a fix to the regression.
Misha
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #5 from Misha Koshelev mk144210@bcm.edu 2008-03-04 21:05:42 --- Ok I can reproduce it will work on fix...
Misha
http://bugs.winehq.org/show_bug.cgi?id=11791
Lei Zhang thestig@google.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever Confirmed|0 |1
--- Comment #6 from Lei Zhang thestig@google.com 2008-03-04 21:08:12 --- confirmed by comment 5
http://bugs.winehq.org/show_bug.cgi?id=11791
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |drraph@gmail.com
--- Comment #7 from Austin English austinenglish@gmail.com 2008-03-30 17:10:09 --- *** Bug 12280 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #8 from Raphael drraph@gmail.com 2008-04-11 04:31:35 --- The bug which I reported as bug 12280 has a working fix there. Is that all that is needed for this bug too?
Raphael
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #9 from Misha Koshelev mk144210@bcm.edu 2008-04-11 07:55:51 --- btw, sorry I have been quite busy with work/school.
That patch fixed bug 9479. If you revert it I believe you'll reopen that bug and plus that patch per my testing was correct so the problem either is in actuality somewhere else (but exposed by that patch) or in a specific subcase for that patch which is not being handled. So I don't recommend simply reverting it.
Thanks Misha
http://bugs.winehq.org/show_bug.cgi?id=11791
Reinier Lamers tux_rocker@reinier.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tux_rocker@reinier.de
http://bugs.winehq.org/show_bug.cgi?id=11791
Hans Leidekker hans@it.vu.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |hans@it.vu.nl
--- Comment #10 from Hans Leidekker hans@it.vu.nl 2008-05-25 09:48:32 --- Mischas patch has probably just uncovered another bug in wininet. InternetOpenUrl is a convenience wrapper that will basically open a connection, send a request and return the request handle. When the request handle is closed we send a callback for the parent connection handle too which the app does not expect because it never created it.
We're already suppressing the INTERNET_STATUS_HANDLE_CREATED callback for the connection handle when called through InternetOpenUrl, we should do the same for the INTERNET_STATUS_HANDLE_CLOSING callback.
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #11 from Hans Leidekker hans@it.vu.nl 2008-05-25 10:02:11 --- Created an attachment (id=13330) --> (http://bugs.winehq.org/attachment.cgi?id=13330) wininet: Don't send a handle closing callback for session handles created with InternetOpenUrl.
This patch fixes the crash.
http://bugs.winehq.org/show_bug.cgi?id=11791
Dan Kegel dank@kegel.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dank@kegel.com
--- Comment #12 from Dan Kegel dank@kegel.com 2008-05-26 11:31:09 --- *** Bug 12333 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #13 from Louis Lenders xerox_xerox2000@yahoo.co.uk 2008-05-29 01:58:02 --- Hi Hans, is there a reason that you haven't send the patch to wine-patches?
Though technically spoken this might not be a regression, it affects quite some applications that used to work fine, and it would be nice if this could be fixed before wine-1.0
http://bugs.winehq.org/show_bug.cgi?id=11791
--- Comment #14 from Hans Leidekker hans@it.vu.nl 2008-05-29 03:54:47 ---
Hi Hans, is there a reason that you haven't send the patch to wine-patches?
This is one of a handful of fixes for wininet that I'm currently testing with various http clients to gain confidence that they won't cause any regressions or uncover more wininet bugs. I'll probably send them in this weekend.
http://bugs.winehq.org/show_bug.cgi?id=11791
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch
http://bugs.winehq.org/show_bug.cgi?id=11791
Hans Leidekker hans@it.vu.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #15 from Hans Leidekker hans@it.vu.nl 2008-06-02 08:25:42 --- Fixed by 9d55252d41f2c6b578c76716c1d113588d700c0e
http://bugs.winehq.org/show_bug.cgi?id=11791
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #16 from Alexandre Julliard julliard@winehq.org 2008-06-06 10:39:52 --- Closing bugs fixed in 1.0-rc4.
http://bugs.winehq.org/show_bug.cgi?id=11791
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|CVS/GIT |unspecified