[Bug 19852] New: AsyncHttpEndRequestProc was called twice
http://bugs.winehq.org/show_bug.cgi?id=19852 Summary: AsyncHttpEndRequestProc was called twice Product: Wine Version: unspecified Platform: PC OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: wininet AssignedTo: wine-bugs(a)winehq.org ReportedBy: leeight(a)gmail.com AsyncHttpEndRequestProc -> HTTP_HttpEndRequestW -> HTTP_GetResponseHeaders was called twice, which cause my application crashed. the attachment is my debug log, and the temporary solution is change AsyncHttpEndRequestProc like this static void AsyncHttpEndRequestProc(WORKREQUEST *work) { struct WORKREQ_HTTPENDREQUESTW const *req = &work->u.HttpEndRequestW; http_request_t *lpwhr = (http_request_t*)work->hdr; TRACE("%p\n", lpwhr); if(lpwhr->lpszRawHeaders == NULL){ // add this line FIXME("[%p]call HTTP_GetResponseHeaders", (lpwhr)); HTTP_HttpEndRequestW(lpwhr, req->dwFlags, req->dwContext); } // add this line } -- 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=19852 Dmitry Timoshkov <dmitry(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |1.1.28 --- Comment #1 from Dmitry Timoshkov <dmitry(a)codeweavers.com> 2009-08-27 04:21:06 --- Is it possible to add a test case? -- 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=19852 leeight <leeight(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |leeight(a)gmail.com --- Comment #2 from leeight <leeight(a)gmail.com> 2009-08-27 05:47:57 --- you can download Baidu HI http://im.baidu.com/download/down.php?a=im and use this account: username:wine_test password:helloworld you'll find you fail login, but if you are using wine 1.1.24-, it's ok this problems occurs since wine 1.1.25+ (In reply to comment #1)
Is it possible to add a test case?
-- 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=19852 Dmitry Timoshkov <dmitry(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression CC|leeight(a)gmail.com | --- Comment #3 from Dmitry Timoshkov <dmitry(a)codeweavers.com> 2009-08-27 07:32:42 --- Please perform the regression test: http://wiki.winehq.org/RegressionTesting -- 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=19852 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download -- 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=19852 leeight <leeight(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aric(a)codeweavers.com --- Comment #4 from leeight <leeight(a)gmail.com> 2009-08-30 01:33:19 --- after regression test, i found this problem was caused by this patch $ git bisect good b9f2f9d2265616286eb7a6271852cfdcc121ae90 is first bad commit commit b9f2f9d2265616286eb7a6271852cfdcc121ae90 Author: Aric Stewart <aric(a)codeweavers.com> Date: Tue Jun 23 14:30:38 2009 +0900 wininet: Improve HTTP status 100 handling. :040000 040000 a071c537a20a37299e6c1da7eab498c749cac58f 6204ea489fac6ceffc4f01464a4b64d24b9bb36f M dlls and "git show b9f2f9d2265616286eb7a6271852cfdcc121ae90 | patch -p1 -R" is a good solution. hope you can merge it in the trunk, thanks a lot (In reply to comment #3)
Please perform the regression test: http://wiki.winehq.org/RegressionTesting
-- 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=19852 Hans Leidekker <hans(a)meelstraat.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hans(a)meelstraat.net --- Comment #5 from Hans Leidekker <hans(a)meelstraat.net> 2010-06-07 03:11:06 --- I can't reproduce the problem, it crashes before wininet is used. Please attach a +wininet trace if you can reproduce it. -- 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=19852 Henri Verbeet <hverbeet(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |b9f2f9d2265616286eb7a627185 | |2cfdcc121ae90 -- 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=19852 Frédéric Delanoy <frederic.delanoy(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Abandoned? CC| |frederic.delanoy(a)gmail.com --- Comment #6 from Frédéric Delanoy <frederic.delanoy(a)gmail.com> 2012-08-01 04:37:51 CDT --- (In reply to comment #5)
I can't reproduce the problem, it crashes before wininet is used. Please attach a +wininet trace if you can reproduce it.
Pl(In reply to comment #4)
after regression test, i found this problem was caused by this patch
Please retry with wine 1.5.10 or later and check if the problem persists. -- 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=19852 Ken Sharp <kennybobs(a)o2.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |ABANDONED --- Comment #7 from Ken Sharp <kennybobs(a)o2.co.uk> 2013-03-28 17:52:22 CDT --- No reply in months. Please reopen if problem persists. -- 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=19852 André H. <nerv(a)dawncrow.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED CC| |nerv(a)dawncrow.de --- Comment #8 from André H. <nerv(a)dawncrow.de> 2013-04-13 16:15:19 CDT --- closing -- 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