@@ -1328,8 +1328,6 @@
static void
man_load_stats(struct management *man)
{
- extern counter_type link_read_bytes_global;
- extern counter_type link_write_bytes_global;
int nclients = 0;
if (man->persist.callback.n_clients)
@@ -2080,12 +2080,12 @@
"may accept clients which do not present a certificate");
}
- const unsigned int tls_version_max =
+ const unsigned int tls_ver_max =
(options->ssl_flags >> SSLF_TLS_VERSION_MAX_SHIFT) & SSLF_TLS_VERSION_MAX_MASK;
- const unsigned int tls_version_min =
+ const unsigned int tls_ver_min =
(options->ssl_flags >> SSLF_TLS_VERSION_MIN_SHIFT) & SSLF_TLS_VERSION_MIN_MASK;
- if (tls_version_max > 0 && tls_version_max < tls_version_min)
+ if (tls_ver_max > 0 && tls_ver_max < tls_ver_min)
{
msg(M_USAGE, "--tls-version-min bigger than --tls-version-max");
}
@@ -3032,16 +3032,16 @@
{
struct dns_domain **entry = &dns->search_domains;
ALLOC_OBJ_CLEAR_GC(*entry, struct dns_domain, &dns->gc);
- struct dns_domain *new = *entry;
- new->name = dhcp->domain;
- entry = &new->next;
+ struct dns_domain *domain = *entry;
+ domain->name = dhcp->domain;
+ entry = &domain->next;
for (unsigned int i = 0; i < dhcp->domain_search_list_len; ++i)
{
ALLOC_OBJ_CLEAR_GC(*entry, struct dns_domain, &dns->gc);
- struct dns_domain *new = *entry;
- new->name = dhcp->domain_search_list[i];
- entry = &new->next;
+ struct dns_domain *search_domain = *entry;
+ search_domain->name = dhcp->domain_search_list[i];
+ entry = &search_domain->next;
}
struct dns_server *server = dns_server_get(&dns->servers, 0, &dns->gc);
@@ -3134,7 +3134,6 @@
static void
options_postprocess_mutate(struct options *o, struct env_set *es)
{
- int i;
/*
* Process helper-type options which map to other, more complex
* sequences of options.
@@ -3167,7 +3166,7 @@
* Convert remotes into connection list
*/
const struct remote_list *rl = o->remote_list;
- for (i = 0; i < rl->len; ++i)
+ for (int i = 0; i < rl->len; ++i)
{
const struct remote_entry *re = rl->array[i];
struct connection_entry ce = o->ce;
@@ -3189,14 +3188,14 @@
}
ASSERT(o->connection_list);
- for (i = 0; i < o->connection_list->len; ++i)
+ for (int i = 0; i < o->connection_list->len; ++i)
{
options_postprocess_mutate_ce(o, o->connection_list->array[i]);
}
if (o->ce.local_list)
{
- for (i = 0; i < o->ce.local_list->len; i++)
+ for (int i = 0; i < o->ce.local_list->len; i++)
{
options_postprocess_mutate_le(&o->ce, o->ce.local_list->array[i], o->mode);
}
@@ -3224,7 +3223,7 @@
}
/* use the same listen list for every outgoing connection */
- for (i = 0; i < o->connection_list->len; ++i)
+ for (int i = 0; i < o->connection_list->len; ++i)
{
o->connection_list->array[i]->local_list = o->ce.local_list;
}
@@ -813,9 +813,9 @@
* Get host's IP address
*/
struct addrinfo *ai;
- int status = openvpn_getaddrinfo(GETADDR_RESOLVE | GETADDR_FATAL, host, port,
- 0, NULL, AF_UNSPEC, &ai);
- ASSERT(status == 0);
+ int ga_status = openvpn_getaddrinfo(GETADDR_RESOLVE | GETADDR_FATAL, host, port,
+ 0, NULL, AF_UNSPEC, &ai);
+ ASSERT(ga_status == 0);
ASSERT(sizeof(hostaddr.addr) >= ai->ai_addrlen);
memcpy(&hostaddr.addr.sa, ai->ai_addr, ai->ai_addrlen);
freeaddrinfo(ai);
@@ -863,8 +863,8 @@
set_cloexec(fd[0]);
/* wait for background child process to initialize */
- int status = recv_control(fd[0]);
- if (status == RESPONSE_INIT_SUCCEEDED)
+ int recv_status = recv_control(fd[0]);
+ if (recv_status == RESPONSE_INIT_SUCCEEDED)
{
/* note that this will cause possible EAGAIN when writing to
* control socket if proxy process is backlogged */
@@ -875,7 +875,7 @@
}
else
{
- msg(M_ERR, "PORT SHARE: unexpected init recv_control status=%d", status);
+ msg(M_ERR, "PORT SHARE: unexpected init recv_control status=%d", recv_status);
}
}
else
@@ -776,24 +776,24 @@
}
/* Sanity check: load client key (as "client") */
- struct key_ctx_bi test_client_key;
- struct buffer test_wrapped_client_key;
+ struct key_ctx_bi check_client_key;
+ struct buffer check_wrapped_client_key;
struct key2 keydata;
msg(D_GENKEY, "Testing client-side key loading...");
- tls_crypt_v2_init_client_key(&test_client_key, &keydata, &test_wrapped_client_key, client_file,
+ tls_crypt_v2_init_client_key(&check_client_key, &keydata, &check_wrapped_client_key, client_file,
client_inline);
- free_key_ctx_bi(&test_client_key);
+ free_key_ctx_bi(&check_client_key);
/* Sanity check: unwrap and load client key (as "server") */
- struct buffer test_metadata = alloc_buf_gc(TLS_CRYPT_V2_MAX_METADATA_LEN, &gc);
- struct key2 test_client_key2 = { 0 };
+ struct buffer check_metadata = alloc_buf_gc(TLS_CRYPT_V2_MAX_METADATA_LEN, &gc);
+ struct key2 check_client_key2 = { 0 };
free_key_ctx(&server_key);
tls_crypt_v2_init_server_key(&server_key, false, server_key_file, server_key_inline);
msg(D_GENKEY, "Testing server-side key loading...");
- ASSERT(tls_crypt_v2_unwrap_client_key(&test_client_key2, &test_metadata,
- test_wrapped_client_key, &server_key));
- secure_memzero(&test_client_key2, sizeof(test_client_key2));
- free_buf(&test_wrapped_client_key);
+ ASSERT(tls_crypt_v2_unwrap_client_key(&check_client_key2, &check_metadata,
+ check_wrapped_client_key, &server_key));
+ secure_memzero(&check_client_key2, sizeof(check_client_key2));
+ free_buf(&check_wrapped_client_key);
cleanup:
secure_memzero(&client_key, sizeof(client_key));
@@ -541,19 +541,19 @@
static DWORD
InterfaceLuid(const char *iface_name, PNET_LUID luid)
{
- NETIO_STATUS status;
+ NETIO_STATUS convert_status;
LPWSTR wide_name = utf8to16(iface_name);
if (wide_name)
{
- status = ConvertInterfaceAliasToLuid(wide_name, luid);
+ convert_status = ConvertInterfaceAliasToLuid(wide_name, luid);
free(wide_name);
}
else
{
- status = ERROR_OUTOFMEMORY;
+ convert_status = ERROR_OUTOFMEMORY;
}
- return status;
+ return convert_status;
}
static BOOL
@@ -1183,8 +1183,8 @@
goto out;
}
- SERVICE_STATUS status;
- if (ControlService(dnssvc, SERVICE_CONTROL_PARAMCHANGE, &status) == 0)
+ SERVICE_STATUS control_status;
+ if (ControlService(dnssvc, SERVICE_CONTROL_PARAMCHANGE, &control_status) == 0)
{
MsgToEventLog(M_ERR, L"%S: ControlService call failed (%lu)", __func__, GetLastError());
goto out;
@@ -3763,12 +3763,12 @@
static DWORD WINAPI
ServiceCtrlInteractive(DWORD ctrl_code, DWORD event, LPVOID data, LPVOID ctx)
{
- SERVICE_STATUS *status = ctx;
+ SERVICE_STATUS *svc_status = ctx;
switch (ctrl_code)
{
case SERVICE_CONTROL_STOP:
- status->dwCurrentState = SERVICE_STOP_PENDING;
- ReportStatusToSCMgr(service, status);
+ svc_status->dwCurrentState = SERVICE_STOP_PENDING;
+ ReportStatusToSCMgr(service, svc_status);
if (exit_event)
{
SetEvent(exit_event);
@@ -55,8 +55,8 @@
{
struct gc_arena gc = gc_new();
struct buffer out_buf = alloc_buf_gc(512, &gc);
- struct buffer clear_buf = alloc_buf_gc(512, &gc);
- buf_clear(&clear_buf);
+ struct buffer clean_buf = alloc_buf_gc(512, &gc);
+ buf_clear(&clean_buf);
bool error = false;
#define LONGDOMAIN "a-reaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaally-long-domain"
@@ -79,7 +79,7 @@
struct buffer small_buf = alloc_buf_gc(sizeof(output_1) - 1, &gc);
buf_clear(&small_buf);
write_dhcp_search_str(&small_buf, DHCP_DOMAIN_SEARCH, search_list, 2, &error);
- assert_memory_equal(BPTR(&small_buf), BPTR(&clear_buf), buf_forward_capacity_total(&small_buf));
+ assert_memory_equal(BPTR(&small_buf), BPTR(&clean_buf), buf_forward_capacity_total(&small_buf));
assert_true(error);
error = false;
@@ -103,13 +103,13 @@
buf_clear(&out_buf);
write_dhcp_search_str(&out_buf, DHCP_DOMAIN_SEARCH, search_list + 5, 1, &error);
- assert_memory_equal(BPTR(&out_buf), BPTR(&clear_buf), buf_forward_capacity_total(&clear_buf));
+ assert_memory_equal(BPTR(&out_buf), BPTR(&clean_buf), buf_forward_capacity_total(&clean_buf));
assert_true(error);
error = false;
buf_clear(&out_buf);
write_dhcp_search_str(&out_buf, DHCP_DOMAIN_SEARCH, search_list, 3, &error);
- assert_memory_equal(BPTR(&out_buf), BPTR(&clear_buf), buf_forward_capacity_total(&clear_buf));
+ assert_memory_equal(BPTR(&out_buf), BPTR(&clean_buf), buf_forward_capacity_total(&clean_buf));
assert_true(error);
error = false;
@@ -344,8 +344,8 @@
/* Message 1: first batch of routes, continuation 2 (more coming) */
struct buffer buf1 = alloc_buf(512);
- const char *msg1 = "PUSH_UPDATE, route 10.1.0.0 255.255.0.0, route 10.2.0.0 255.255.0.0, route 10.3.0.0 255.255.0.0,push-continuation 2";
- buf_write(&buf1, msg1, strlen(msg1));
+ const char *cont_msg1 = "PUSH_UPDATE, route 10.1.0.0 255.255.0.0, route 10.2.0.0 255.255.0.0, route 10.3.0.0 255.255.0.0,push-continuation 2";
+ buf_write(&buf1, cont_msg1, strlen(cont_msg1));
assert_int_equal(process_incoming_push_msg(c, &buf1, c->options.pull, pull_permission_mask(c),
&option_types_found),
@@ -354,8 +354,8 @@
/* Message 2: more routes, continuation 2 (more coming) */
struct buffer buf2 = alloc_buf(512);
- const char *msg2 = "PUSH_UPDATE, route 10.4.0.0 255.255.0.0, route 10.5.0.0 255.255.0.0, route 10.6.0.0 255.255.0.0,push-continuation 2";
- buf_write(&buf2, msg2, strlen(msg2));
+ const char *cont_msg2 = "PUSH_UPDATE, route 10.4.0.0 255.255.0.0, route 10.5.0.0 255.255.0.0, route 10.6.0.0 255.255.0.0,push-continuation 2";
+ buf_write(&buf2, cont_msg2, strlen(cont_msg2));
assert_int_equal(process_incoming_push_msg(c, &buf2, c->options.pull, pull_permission_mask(c),
&option_types_found),
@@ -364,8 +364,8 @@
/* Message 3: final batch of routes, continuation 1 (last message) */
struct buffer buf3 = alloc_buf(512);
- const char *msg3 = "PUSH_UPDATE, route 10.7.0.0 255.255.0.0, route 10.8.0.0 255.255.0.0, route 10.9.0.0 255.255.0.0,push-continuation 1";
- buf_write(&buf3, msg3, strlen(msg3));
+ const char *cont_msg3 = "PUSH_UPDATE, route 10.7.0.0 255.255.0.0, route 10.8.0.0 255.255.0.0, route 10.9.0.0 255.255.0.0,push-continuation 1";
+ buf_write(&buf3, cont_msg3, strlen(cont_msg3));
assert_int_equal(process_incoming_push_msg(c, &buf3, c->options.pull, pull_permission_mask(c),
&option_types_found),