[PATCH] wininet: Initialize netconn_stream's content_length

Andy Clayton q3aiml at gmail.com
Tue Nov 22 21:17:07 CST 2011


On Tue, Nov 22, 2011 at 9:28 AM, Andy Clayton <q3aiml at gmail.com> wrote:

> Hi Jacek,
>
> On Tue, Nov 22, 2011 at 3:59 AM, Jacek Caban <jacek at codeweavers.com>wrote:
>
>> This doesn't seem right. At the point where we call
>> HTTP_ReceiveRequestData, content_length is supposed to be set based on
>> response header. The bug must be somewhere else.
>>
>
> Thanks for looking over my patch. So it seems then that
> HTTP_HttpSendRequestW should not be calling HTTP_ReceiveRequestData if
> bEndRequest is false (http.c:4941), since there is no way at that point it
> could have read the headers. Is that a better fix?
>
> Thanks again for your help,
>
> Andy
>
>
I've attached a second attempt showing this. Thanks for any thoughts you
can provide.

Andy
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20111122/3433599e/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-wininet-avoid-calling-HTTP_ReceiveRequestData-before.patch
Type: text/x-patch
Size: 7971 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20111122/3433599e/attachment.bin>


More information about the wine-devel mailing list