Hans Leidekker : winhttp: Check the return value of send_request.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 97cd0291b004a2fe455fbf95758d45ada10a67ac
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=97cd0291b004a2fe455fbf9575...

Author: Hans Leidekker <hans(a)codeweavers.com>
Date:   Tue Dec  9 12:49:28 2014 +0100

winhttp: Check the return value of send_request.

---

 dlls/winhttp/request.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/dlls/winhttp/request.c b/dlls/winhttp/request.c
index 04e71d9..62ecfee 100644
--- a/dlls/winhttp/request.c
+++ b/dlls/winhttp/request.c
@@ -2314,22 +2314,17 @@ static BOOL receive_response( request_t *request, BOOL async )
             if (!(ret = handle_redirect( request, status ))) break;
 
             /* recurse synchronously */
-            send_request( request, NULL, 0, request->optional, request->optional_len, 0, 0, FALSE );
-            continue;
+            if ((ret = send_request( request, NULL, 0, request->optional, request->optional_len, 0, 0, FALSE ))) continue;
         }
         else if (status == HTTP_STATUS_DENIED || status == HTTP_STATUS_PROXY_AUTH_REQ)
         {
             if (request->hdr.disable_flags & WINHTTP_DISABLE_AUTHENTICATION) break;
 
             drain_content( request );
-            if (!handle_authorization( request, status ))
-            {
-                ret = TRUE;
-                break;
-            }
+            if (!handle_authorization( request, status )) break;
+
             /* recurse synchronously */
-            send_request( request, NULL, 0, request->optional, request->optional_len, 0, 0, FALSE );
-            continue;
+            if ((ret = send_request( request, NULL, 0, request->optional, request->optional_len, 0, 0, FALSE ))) continue;
         }
         break;
     }