Re: [8/9] winhttp: Don't try to read data when the server response is HTTP_STATUS_NO_CONTENT or HTTP_STATUS_NOT_MODIFIED.

[Bruno Jesus]

On Tue, Dec 9, 2014 at 9:50 AM, Hans Leidekker <hans(a)codeweavers.com> wrote:
> Tests based on a patch by Bruno Jesus.
> ---
>  dlls/winhttp/request.c       | 15 ++++++-----
>  dlls/winhttp/tests/winhttp.c | 62 +++++++++++++++++++++++++++++++++++++++++++-
>  2 files changed, 69 insertions(+), 8 deletions(-)

Hans, I can no longer find the bug that this was about. Can you please
give the bug URL? Do you think your current patch set fixed it?

Best regards,
Bruno

[8/9] winhttp: Don't try to read data when the server response is HTTP_STATUS_NO_CONTENT or HTTP_STATUS_NOT_MODIFIED.

[Hans Leidekker]

On Tue, 2014-12-09 at 23:47 -0200, Bruno Jesus wrote:
> On Tue, Dec 9, 2014 at 9:50 AM, Hans Leidekker <hans(a)codeweavers.com> wrote:
> > Tests based on a patch by Bruno Jesus.
> > ---
> >  dlls/winhttp/request.c       | 15 ++++++-----
> >  dlls/winhttp/tests/winhttp.c | 62 +++++++++++++++++++++++++++++++++++++++++++-
> >  2 files changed, 69 insertions(+), 8 deletions(-)
>
> Hans, I can no longer find the bug that this was about. Can you please
> give the bug URL? Do you think your current patch set fixed it?

This patch passes the test case you attached to https://bugs.winehq.org/show_bug.cgi?id=36937
I don't know if it fixes the application mentioned in that bug.