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@winehq.org ReportedBy: leeight@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 }
http://bugs.winehq.org/show_bug.cgi?id=19852
Dmitry Timoshkov dmitry@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |1.1.28
--- Comment #1 from Dmitry Timoshkov dmitry@codeweavers.com 2009-08-27 04:21:06 --- Is it possible to add a test case?
http://bugs.winehq.org/show_bug.cgi?id=19852
leeight leeight@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |leeight@gmail.com
--- Comment #2 from leeight leeight@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?
http://bugs.winehq.org/show_bug.cgi?id=19852
Dmitry Timoshkov dmitry@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression CC|leeight@gmail.com |
--- Comment #3 from Dmitry Timoshkov dmitry@codeweavers.com 2009-08-27 07:32:42 --- Please perform the regression test: http://wiki.winehq.org/RegressionTesting
http://bugs.winehq.org/show_bug.cgi?id=19852
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download
http://bugs.winehq.org/show_bug.cgi?id=19852
leeight leeight@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aric@codeweavers.com
--- Comment #4 from leeight leeight@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@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
http://bugs.winehq.org/show_bug.cgi?id=19852
Hans Leidekker hans@meelstraat.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |hans@meelstraat.net
--- Comment #5 from Hans Leidekker hans@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.
http://bugs.winehq.org/show_bug.cgi?id=19852
Henri Verbeet hverbeet@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |b9f2f9d2265616286eb7a627185 | |2cfdcc121ae90
http://bugs.winehq.org/show_bug.cgi?id=19852
Frédéric Delanoy frederic.delanoy@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Abandoned? CC| |frederic.delanoy@gmail.com
--- Comment #6 from Frédéric Delanoy frederic.delanoy@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.
http://bugs.winehq.org/show_bug.cgi?id=19852
Ken Sharp kennybobs@o2.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |ABANDONED
--- Comment #7 from Ken Sharp kennybobs@o2.co.uk 2013-03-28 17:52:22 CDT --- No reply in months. Please reopen if problem persists.
http://bugs.winehq.org/show_bug.cgi?id=19852
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED CC| |nerv@dawncrow.de
--- Comment #8 from André H. nerv@dawncrow.de 2013-04-13 16:15:19 CDT --- closing