diff --git a/modules/proxy/mod_proxy.c b/modules/proxy/mod_proxy.c index c84cd9a59f..94c2954b9c 100644 --- a/modules/proxy/mod_proxy.c +++ b/modules/proxy/mod_proxy.c @@ -985,7 +985,7 @@ static int proxy_handler(request_rec *r) /* Initialise worker if needed, note the shared area must be initialized by the balancer logic */ if (balancer) { - ap_proxy_initialize_worker(worker, r->server, conf->pool); + ap_proxy_initialize_worker(worker, r->server, conf->pool); } if (balancer && balancer->max_attempts_set && !max_attempts) @@ -1357,7 +1357,7 @@ static const char * const char *err; err = ap_check_cmd_context(cmd, NOT_IN_DIRECTORY|NOT_IN_FILES); - if (err) { + if (err) { return err; } @@ -1512,7 +1512,7 @@ static const char * add_pass_reverse(cmd_parms *cmd, void *dconf, const char *f, const char *err; err = ap_check_cmd_context(cmd, NOT_IN_DIRECTORY|NOT_IN_FILES); - if (err) { + if (err) { return err; } @@ -2391,7 +2391,7 @@ static void child_init(apr_pool_t *p, server_rec *s) PROXY_STRNCPY(conf->forward->s->name, "proxy:forward"); PROXY_STRNCPY(conf->forward->s->hostname, "*"); PROXY_STRNCPY(conf->forward->s->scheme, "*"); - conf->forward->hash = conf->forward->s->hash = + conf->forward->hash = conf->forward->s->hash = ap_proxy_hashfunc(conf->forward->s->name, PROXY_HASHFUNC_DEFAULT); /* Do not disable worker in case of errors */ conf->forward->s->status |= PROXY_WORKER_IGNORE_ERRORS; @@ -2404,7 +2404,7 @@ static void child_init(apr_pool_t *p, server_rec *s) PROXY_STRNCPY(reverse->s->name, "proxy:reverse"); PROXY_STRNCPY(reverse->s->hostname, "*"); PROXY_STRNCPY(reverse->s->scheme, "*"); - reverse->hash = reverse->s->hash = + reverse->hash = reverse->s->hash = ap_proxy_hashfunc(reverse->s->name, PROXY_HASHFUNC_DEFAULT); /* Do not disable worker in case of errors */ reverse->s->status |= PROXY_WORKER_IGNORE_ERRORS; diff --git a/modules/proxy/mod_proxy.h b/modules/proxy/mod_proxy.h index d75628605a..0c7bdac96d 100644 --- a/modules/proxy/mod_proxy.h +++ b/modules/proxy/mod_proxy.h @@ -15,7 +15,7 @@ */ #ifndef MOD_PROXY_H -#define MOD_PROXY_H +#define MOD_PROXY_H /** * @file mod_proxy.h @@ -159,7 +159,7 @@ typedef struct { } proxy_status; /* Status display options */ apr_sockaddr_t *source_address; apr_global_mutex_t *mutex; /* global lock (needed??) */ - + int req_set:1; int viaopt_set:1; int recv_buffer_size_set:1; @@ -384,7 +384,7 @@ struct proxy_balancer_method { void *context; /* general purpose storage */ apr_status_t (*reset)(proxy_balancer *balancer, server_rec *s); apr_status_t (*age)(proxy_balancer *balancer, server_rec *s); - apr_status_t (*updatelbstatus)(proxy_balancer *balancer, proxy_worker *elected, server_rec *s); + apr_status_t (*updatelbstatus)(proxy_balancer *balancer, proxy_worker *elected, server_rec *s); }; #define PROXY_THREAD_LOCK(x) apr_thread_mutex_lock((x)->mutex) @@ -395,7 +395,7 @@ struct proxy_balancer_method { /* hooks */ -/* Create a set of PROXY_DECLARE(type), PROXY_DECLARE_NONSTD(type) and +/* Create a set of PROXY_DECLARE(type), PROXY_DECLARE_NONSTD(type) and * PROXY_DECLARE_DATA with appropriate export and import tags for the platform */ #if !defined(WIN32) @@ -423,14 +423,14 @@ struct proxy_balancer_method { #define PROXY_OPTIONAL_HOOK(name,fn,pre,succ,order) \ APR_OPTIONAL_HOOK(proxy,name,fn,pre,succ,order) -APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, scheme_handler, (request_rec *r, - proxy_worker *worker, proxy_server_conf *conf, char *url, +APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, scheme_handler, (request_rec *r, + proxy_worker *worker, proxy_server_conf *conf, char *url, const char *proxyhost, apr_port_t proxyport)) -APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, canon_handler, (request_rec *r, +APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, canon_handler, (request_rec *r, char *url)) APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, create_req, (request_rec *r, request_rec *pr)) -APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, fixups, (request_rec *r)) +APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, fixups, (request_rec *r)) /** * pre request hook. @@ -443,7 +443,7 @@ APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, fixups, (request_rec *r)) APR_DECLARE_EXTERNAL_HOOK(proxy, PROXY, int, pre_request, (proxy_worker **worker, proxy_balancer **balancer, request_rec *r, - proxy_server_conf *conf, char **url)) + proxy_server_conf *conf, char **url)) /** * post request hook. * It is called after request for updating runtime balancer status. @@ -528,7 +528,7 @@ PROXY_DECLARE(proxy_worker *) ap_proxy_get_worker(apr_pool_t *p, const char *url); /** * Define and Allocate space for the worker to proxy configuration - * @param p memory pool to allocate worker from + * @param p memory pool to allocate worker from * @param worker the new worker * @param balancer the balancer that the worker belongs to * @param conf current proxy server configuration @@ -543,7 +543,7 @@ PROXY_DECLARE(char *) ap_proxy_define_worker(apr_pool_t *p, /** * Share a defined proxy worker via shm - * @param worker worker to be shared + * @param worker worker to be shared * @param shm location of shared info * @param i index into shm * @return APR_SUCCESS or error code @@ -583,7 +583,7 @@ PROXY_DECLARE(proxy_balancer *) ap_proxy_get_balancer(apr_pool_t *p, /** * Define and Allocate space for the balancer to proxy configuration - * @param p memory pool to allocate balancer from + * @param p memory pool to allocate balancer from * @param balancer the new balancer * @param conf current proxy server configuration * @param url url containing balancer name @@ -601,7 +601,7 @@ PROXY_DECLARE(char *) ap_proxy_define_balancer(apr_pool_t *p, * @param r current request * @param conf current proxy server configuration * @param url request url that balancer can rewrite. - * @return OK or HTTP_XXX error + * @return OK or HTTP_XXX error * @note It calls balancer pre_request hook if the url starts with balancer:// * The balancer then rewrites the url to particular worker, like http://host:port */ @@ -618,7 +618,7 @@ PROXY_DECLARE(int) ap_proxy_pre_request(proxy_worker **worker, * @param conf current proxy server configuration * @return OK or HTTP_XXX error * @note Whenever the pre_request is called, the post_request has to be - * called too. + * called too. */ PROXY_DECLARE(int) ap_proxy_post_request(proxy_worker *worker, proxy_balancer *balancer, @@ -647,7 +647,7 @@ PROXY_DECLARE(int) ap_proxy_post_request(proxy_worker *worker, * @param server_portstr Via headers server port * @param server_portstr_size size of the server_portstr buffer * @return OK or HTTP_XXX error - */ + */ PROXY_DECLARE(int) ap_proxy_determine_connection(apr_pool_t *p, request_rec *r, proxy_server_conf *conf, proxy_worker *worker, @@ -694,7 +694,7 @@ PROXY_DECLARE(int) ap_proxy_acquire_connection(const char *proxy_function, * @param s current server record * @return OK or HTTP_XXX error * @note The connection will be closed if conn->close_on_release is set - */ + */ PROXY_DECLARE(int) ap_proxy_release_connection(const char *proxy_function, proxy_conn_rec *conn, server_rec *s); @@ -707,7 +707,7 @@ PROXY_DECLARE(int) ap_proxy_release_connection(const char *proxy_function, * @return OK or HTTP_XXX error * @note In case the socket already exists for conn, just check the link * status. - */ + */ PROXY_DECLARE(int) ap_proxy_connect_backend(const char *proxy_function, proxy_conn_rec *conn, proxy_worker *worker, diff --git a/modules/proxy/mod_proxy_balancer.c b/modules/proxy/mod_proxy_balancer.c index f367795420..7990529dd4 100644 --- a/modules/proxy/mod_proxy_balancer.c +++ b/modules/proxy/mod_proxy_balancer.c @@ -35,18 +35,18 @@ static char balancer_nonce[APR_UUID_FORMATTED_LENGTH + 1]; * Register our mutex type before the config is read so we * can adjust the mutex settings using the Mutex directive. */ -static int balancer_pre_config(apr_pool_t *pconf, apr_pool_t *plog, +static int balancer_pre_config(apr_pool_t *pconf, apr_pool_t *plog, apr_pool_t *ptemp) { apr_status_t rv; - + rv = ap_mutex_register(pconf, balancer_mutex_type, NULL, APR_LOCK_DEFAULT, 0); if (rv != APR_SUCCESS) { return rv; } - + return OK; } @@ -207,7 +207,7 @@ static proxy_worker *find_route_worker(proxy_balancer *balancer, int i; int checking_standby; int checked_standby; - + proxy_worker **workers; checking_standby = checked_standby = 0; @@ -475,7 +475,7 @@ static int proxy_balancer_pre_request(proxy_worker **worker, /* Step 3: force recovery */ force_recovery(*balancer, r->server); - + /* Step 3.5: Update member list for the balancer */ /* TODO: Implement as provider! */ /* proxy_update_members(balancer, r, conf); */ @@ -715,8 +715,7 @@ static int balancer_post_config(apr_pool_t *pconf, apr_pool_t *plog, * the process. */ apr_uuid_get(&uuid); apr_uuid_format(balancer_nonce, &uuid); - - + /* * Get worker slotmem setup */ @@ -736,14 +735,14 @@ static int balancer_post_config(apr_pool_t *pconf, apr_pool_t *plog, proxy_balancer *balancer; sconf = s->module_config; conf = (proxy_server_conf *)ap_get_module_config(sconf, &proxy_module); - + /* Initialize shared scoreboard data */ balancer = (proxy_balancer *)conf->balancers->elts; for (i = 0; i < conf->balancers->nelts; i++, balancer++) { proxy_worker **workers; proxy_worker *worker; ap_slotmem_instance_t *new = NULL; - + balancer->max_workers = balancer->workers->nelts + balancer->growth; balancer->sname = ap_md5(pconf, (const unsigned char *)balancer->name); @@ -756,10 +755,10 @@ static int balancer_post_config(apr_pool_t *pconf, apr_pool_t *plog, balancer->sname); return HTTP_INTERNAL_SERVER_ERROR; } - + apr_pool_cleanup_register(pconf, (void *)s, lock_remove, apr_pool_cleanup_null); - + ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, s, "Doing create: %s (%s), %d, %d", balancer->name, balancer->sname, (int)sizeof(proxy_worker_shared), @@ -782,7 +781,7 @@ static int balancer_post_config(apr_pool_t *pconf, apr_pool_t *plog, if ((rv = storage->grab(balancer->slot, &index)) != APR_SUCCESS) { ap_log_error(APLOG_MARK, APLOG_EMERG, rv, s, "slotmem_grab failed"); return !OK; - + } if ((rv = storage->dptr(balancer->slot, index, (void *)&shm)) != APR_SUCCESS) { ap_log_error(APLOG_MARK, APLOG_EMERG, rv, s, "slotmem_dptr failed"); @@ -796,7 +795,7 @@ static int balancer_post_config(apr_pool_t *pconf, apr_pool_t *plog, } s = s->next; } - + return OK; } @@ -849,11 +848,11 @@ static int balancer_handler(request_rec *r) return HTTP_BAD_REQUEST; } } - + /* Check that the supplied nonce matches this server's nonce; * otherwise ignore all parameters, to prevent a CSRF attack. */ if (*balancer_nonce && - ((name = apr_table_get(params, "nonce")) == NULL + ((name = apr_table_get(params, "nonce")) == NULL || strcmp(balancer_nonce, name) != 0)) { apr_table_clear(params); } @@ -985,7 +984,7 @@ static int balancer_handler(request_rec *r) ap_rvputs(r, "