const-ify the new failure vars, and one old one

diff --git a/src/common/compat_pthreads.c b/src/common/compat_pthreads.c
index 70259a8a53c2a10aa81e9cca3433bfd11de056d3..fdc504690b66c1bee2b1adf2568ca561ced1f6d8 100644 (file)
--- 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 819a57f21451690c98e9cc4f443962b6844b16f5..34d539bcc45e48921c674de721820e8cf2f4ccc4 100644 (file)
--- 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 77eaea0ed9dc74ff9fd2dd8d6d62925e234407f1..eb2748c7d81d24f22d54fb11879cecaa182ce25d 100644 (file)
--- 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;