remove while loop

Author: chronolaw

lib/resty/core/socket.lua

@@ -220,43 +220,57 @@ local function sslhandshake(cosocket, reused_session, server_name, ssl_verify,
         error("no request ctx found", 2)
     end
 
-    while true do
-        if rc == FFI_ERROR then
-            if openssl_error_code[0] ~= 0 then
-                return nil, openssl_error_code[0] .. ": " .. ffi_str(errmsg[0])
-            end
+    if rc == FFI_ERROR then
+        if openssl_error_code[0] ~= 0 then
+            return nil, openssl_error_code[0] .. ": " .. ffi_str(errmsg[0])
+        end
+
+        return nil, ffi_str(errmsg[0])
+    end
 
-            return nil, ffi_str(errmsg[0])
+    if rc == FFI_DONE then
+        return reused_session
+    end
+
+    if rc == FFI_OK then
+        if reused_session == false then
+            return true
         end
 
-        if rc == FFI_DONE then
-            return reused_session
+        rc = C.ngx_http_lua_ffi_socket_tcp_get_sslhandshake_result(r, u,
+                 session_ptr, errmsg, openssl_error_code)
+
+        assert(rc == FFI_OK)
+
+        if session_ptr[0] == nil then
+            return nil
         end
 
-        if rc == FFI_OK then
-            if reused_session == false then
-                return true
-            end
+        return ffi_gc(session_ptr[0], C.ngx_http_lua_ffi_ssl_free_session)
+    end
 
-            rc = C.ngx_http_lua_ffi_socket_tcp_get_sslhandshake_result(r, u,
-                     session_ptr, errmsg, openssl_error_code)
+    assert(rc == FFI_AGAIN)
 
-            assert(rc == FFI_OK)
+    co_yield()
 
-            if session_ptr[0] == nil then
-                return nil
-            end
+    rc = C.ngx_http_lua_ffi_socket_tcp_get_sslhandshake_result(r, u,
+             session_ptr, errmsg, openssl_error_code)
 
-            return ffi_gc(session_ptr[0], C.ngx_http_lua_ffi_ssl_free_session)
+    if rc == FFI_ERROR then
+        if openssl_error_code[0] ~= 0 then
+            return nil, openssl_error_code[0] .. ": " .. ffi_str(errmsg[0])
         end
 
-        assert(rc == FFI_AGAIN)
+        return nil, ffi_str(errmsg[0])
+    end
 
-        co_yield()
+    assert(rc == FFI_OK)
 
-        rc = C.ngx_http_lua_ffi_socket_tcp_get_sslhandshake_result(r, u,
-                 session_ptr, errmsg, openssl_error_code)
+    if session_ptr[0] == nil then
+        return nil
     end
+
+    return ffi_gc(session_ptr[0], C.ngx_http_lua_ffi_ssl_free_session)
 end