[PATCH] wininet: Initialize netconn_stream's content_length

Jacek Caban jacek at codeweavers.com
Wed Nov 23 04:15:00 CST 2011


On 11/23/11 04:17, Andy Clayton wrote:
> On Tue, Nov 22, 2011 at 9:28 AM, Andy Clayton <q3aiml at gmail.com 
> <mailto:q3aiml at gmail.com>> wrote:
>
>     Hi Jacek,
>
>     On Tue, Nov 22, 2011 at 3:59 AM, Jacek Caban
>     <jacek at codeweavers.com <mailto: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.

This patch looks good to me.

Thanks,
     Jacek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20111123/cd7d968a/attachment.html>


More information about the wine-devel mailing list