[Bug 57794] New: WinHTTP implementation assumes HTTP response has a status text

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=57794

Bug ID: 57794
           Summary: WinHTTP implementation assumes HTTP response has a
                    status text
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: winhttp
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

When the HTTP server only provides a status code, e.g. 'HTTP/1.0 200', Wine's
WinHttpReceiveResponse() fails with ERROR_WINHTTP_INVALID_SERVER_RESPONSE.

The exact problem seems to be here:
https://github.com/wine-mirror/wine/blob/55a3a7ee77844258fb480c3c11747893fc6...

This is incongruent with Microsoft's WinHTTP implementation and also generally
a weird requirement that doesn't match real-world usage.

[Bug 57794] WinHTTP implementation assumes HTTP response has a status text

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=57794

--- Comment #1 from Hans Leidekker  ---
Created attachment 78005
  --> https://bugs.winehq.org/attachment.cgi?id=78005
patch

Can you try this patch?

[Bug 57794] WinHTTP implementation assumes HTTP response has a status text

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=57794

--- Comment #2 from [e-mail] ---
I'm not in a position to easily test this. I reported this issue on behalf of
one of my users and worked around it by adding a status text to all responses.

[Bug 57794] WinHTTP implementation assumes HTTP response has a status text

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=57794

Hans Leidekker  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |faff6524b3d72b3ac06e38f9beb
                   |                            |6cb0e459bd6f2
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #3 from Hans Leidekker  ---
Should be fixed with faff6524b3d72b3ac06e38f9beb6cb0e459bd6f2. Thanks for the
bug report!

[Bug 57794] WinHTTP implementation assumes HTTP response has a status text

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=57794

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Alexandre Julliard  ---
Closing bugs fixed in 10.1.