summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2015-03-13 09:41:49 -0400
committerNick Mathewson <nickm@torproject.org>2015-03-13 09:41:49 -0400
commit3a68f2f54ed67176a8ed0b7aa276ef58a54b26db (patch)
tree2384f4434651629918e6c2de50d2308836bca7cc /src
parent833b6d30bec568660e3a9cdeacd1e8c2837f1a6a (diff)
downloadtor-3a68f2f54ed67176a8ed0b7aa276ef58a54b26db.tar.gz
tor-3a68f2f54ed67176a8ed0b7aa276ef58a54b26db.zip
const-ify the new failure vars, and one old one
Diffstat (limited to 'src')
-rw-r--r--src/common/compat_pthreads.c13
-rw-r--r--src/or/control.c2
-rw-r--r--src/or/rendservice.c2
3 files changed, 8 insertions, 9 deletions
diff --git a/src/common/compat_pthreads.c b/src/common/compat_pthreads.c
index 70259a8a53..fdc504690b 100644
--- a/src/common/compat_pthreads.c
+++ b/src/common/compat_pthreads.c
@@ -91,10 +91,9 @@ static pthread_mutexattr_t attr_recursive;
void
tor_mutex_init(tor_mutex_t *mutex)
{
- int err;
if (PREDICT_UNLIKELY(!threads_initialized))
tor_threads_init();
- err = pthread_mutex_init(&mutex->mutex, &attr_recursive);
+ const int err = pthread_mutex_init(&mutex->mutex, &attr_recursive);
if (PREDICT_UNLIKELY(err)) {
log_err(LD_GENERAL, "Error %d creating a mutex.", err);
tor_fragile_assert();
@@ -276,16 +275,16 @@ void
tor_threads_init(void)
{
if (!threads_initialized) {
- int ret;
pthread_mutexattr_init(&attr_recursive);
pthread_mutexattr_settype(&attr_recursive, PTHREAD_MUTEX_RECURSIVE);
- ret = pthread_attr_init(&attr_detached);
- tor_assert(ret == 0);
+ const int ret1 = pthread_attr_init(&attr_detached);
+ tor_assert(ret1 == 0);
#ifndef PTHREAD_CREATE_DETACHED
#define PTHREAD_CREATE_DETACHED 1
#endif
- ret = pthread_attr_setdetachstate(&attr_detached, PTHREAD_CREATE_DETACHED);
- tor_assert(ret == 0);
+ const int ret2 =
+ pthread_attr_setdetachstate(&attr_detached, PTHREAD_CREATE_DETACHED);
+ tor_assert(ret2 == 0);
threads_initialized = 1;
set_main_thread();
}
diff --git a/src/or/control.c b/src/or/control.c
index 819a57f214..34d539bcc4 100644
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -3102,7 +3102,7 @@ handle_control_authchallenge(control_connection_t *conn, uint32_t len,
tor_free(client_nonce);
return -1;
}
- int fail = crypto_rand(server_nonce, SAFECOOKIE_SERVER_NONCE_LEN);
+ const int fail = crypto_rand(server_nonce, SAFECOOKIE_SERVER_NONCE_LEN);
tor_assert(!fail);
/* Now compute and send the server-to-controller response, and the
diff --git a/src/or/rendservice.c b/src/or/rendservice.c
index 77eaea0ed9..eb2748c7d8 100644
--- a/src/or/rendservice.c
+++ b/src/or/rendservice.c
@@ -3320,7 +3320,7 @@ rend_services_introduce(void)
intro = tor_malloc_zero(sizeof(rend_intro_point_t));
intro->extend_info = extend_info_from_node(node, 0);
intro->intro_key = crypto_pk_new();
- int fail = crypto_pk_generate_key(intro->intro_key);
+ const int fail = crypto_pk_generate_key(intro->intro_key);
tor_assert(!fail);
intro->time_published = -1;
intro->time_to_expire = -1;