tor

commit 438aba6c0b70c5becc8ba2c17fad98156901f64b
parent 84e825598a8845b34ac0dff2a6d797725c174b19
Author: George Kadianakis <desnacked@riseup.net>
Date:   Wed,  7 Oct 2020 13:42:57 +0300

Merge branch 'maint-0.4.4' into master

Diffstat:
A
changes/bug40129
 | 
4
++++
M
src/feature/dircache/dircache_stub.c
 | 
2
++
M
src/feature/relay/router.c
 | 
5
+++++

3 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/changes/bug40129 b/changes/bug40129
@@ -0,0 +1,4 @@
+  o Minor bugfixes (compilation):
+    - Fix compiler warnings that would occur when building with
+      "--enable-all-bugs-are-fatal" and "--disable-module-relay"
+      at the same time. Fixes bug 40129; bugfix on 0.4.4.1-alpha.
diff --git a/src/feature/dircache/dircache_stub.c b/src/feature/dircache/dircache_stub.c
@@ -14,6 +14,7 @@
 #include "feature/dircache/dirserv.h"
 #include "feature/dircommon/dir_connection_st.h"
 
+DISABLE_GCC_WARNING("-Wmissing-noreturn")
 int
 directory_handle_command(dir_connection_t *conn)
 {
@@ -29,6 +30,7 @@ connection_dirserv_flushed_some(dir_connection_t *conn)
   tor_assert_nonfatal_unreached_once();
   return -1;
 }
+ENABLE_GCC_WARNING("-Wmissing-noreturn")
 
 void
 dir_conn_clear_spool(dir_connection_t *conn)
diff --git a/src/feature/relay/router.c b/src/feature/relay/router.c
@@ -2607,6 +2607,10 @@ check_descriptor_bandwidth_changed(time_t now)
   }
 }
 
+// This function can be "noreturn" if relay mode is disabled and
+// ALL_BUGS_ARE_FATAL is set.
+DISABLE_GCC_WARNING("-Wmissing-noreturn")
+
 /** Note at log level severity that our best guess of address has changed from
  * <b>prev</b> to <b>cur</b>. */
 void
@@ -2636,6 +2640,7 @@ log_addr_has_changed(int severity,
              "Guessed our IP address as %s (source: %s).",
              addrbuf_cur, source);
 }
+ENABLE_GCC_WARNING("-Wmissing-noreturn")
 
 /** Check whether our own address has changed versus the one we have in our
  * current descriptor.