httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From taka...@apache.org
Subject svn commit: r1601604 - /httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c
Date Tue, 10 Jun 2014 11:21:23 GMT
Author: takashi
Date: Tue Jun 10 11:21:22 2014
New Revision: 1601604

URL: http://svn.apache.org/r1601604
Log:
* apr_status_t vs HTTP status code fix
* use APR_STATUS_IS_ENOTIMPL rather than direct comparison

Modified:
    httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c

Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c?rev=1601604&r1=1601603&r2=1601604&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c Tue Jun 10 11:21:22 2014
@@ -36,7 +36,7 @@ typedef struct ws_baton_t {
     char *scheme;               /* required to release the proxy connection */
 } ws_baton_t;
 
-static int proxy_wstunnel_transfer(request_rec *r, conn_rec *c_i, conn_rec *c_o,
+static apr_status_t proxy_wstunnel_transfer(request_rec *r, conn_rec *c_i, conn_rec *c_o,
                                      apr_bucket_brigade *bb, char *name);
 static void proxy_wstunnel_callback(void *b);
 
@@ -259,10 +259,10 @@ static int proxy_wstunnel_canon(request_
 }
 
 
-static int proxy_wstunnel_transfer(request_rec *r, conn_rec *c_i, conn_rec *c_o,
+static apr_status_t proxy_wstunnel_transfer(request_rec *r, conn_rec *c_i, conn_rec *c_o,
                                      apr_bucket_brigade *bb, char *name)
 {
-    int rv;
+    apr_status_t rv;
 #ifdef DEBUGGING
     apr_off_t len;
 #endif
@@ -432,20 +432,20 @@ static int proxy_wstunnel_request(apr_po
         if (status == SUSPENDED) {
             sockets[0] = baton->client_soc;
             sockets[1] = baton->server_soc;
-            status = ap_mpm_register_socket_callback_timeout(sockets, baton->subpool,
1, 
+            rv = ap_mpm_register_socket_callback_timeout(sockets, baton->subpool, 1, 
                          proxy_wstunnel_callback, 
                          proxy_wstunnel_cancel_callback, 
                          baton, 
                          dconf->idle_timeout);
-            if (status == APR_SUCCESS) { 
+            if (rv == APR_SUCCESS) { 
                 return SUSPENDED;
             }
-            else if (status == APR_ENOTIMPL) { 
+            else if (APR_STATUS_IS_ENOTIMPL(rv)) { 
                 ap_log_rerror(APLOG_MARK, APLOG_TRACE1, 0, r, APLOGNO(02544) "No async support");
                 status = proxy_wstunnel_pump(baton, dconf->idle_timeout, 0); /* force
no async */
             }
             else { 
-                ap_log_rerror(APLOG_MARK, APLOG_ERR, status, r,
+                ap_log_rerror(APLOG_MARK, APLOG_ERR, rv, r,
                               APLOGNO(02543) "error creating websockets tunnel");
                 return HTTP_INTERNAL_SERVER_ERROR;
             }



Mime
View raw message