[tor-commits] [tor/maint-0.4.4] Fix Windows build.

ahf at torproject.org ahf at torproject.org
Fri Feb 5 17:18:31 UTC 2021


commit af9107aff2a76d390426079b2d267269e7215027
Author: Alexander Færøy <ahf at torproject.org>
Date:   Fri Feb 5 17:12:52 2021 +0000

    Fix Windows build.
    
    While trying to resolve our CI issues, the Windows build broke with an
    unused function error:
    
       src/test/test_switch_id.c:37:1: error: ‘unprivileged_port_range_start’
       defined but not used [-Werror=unused-function]
    
    We solve this by moving the `#if !defined(_WIN32)` test above the
    `unprivileged_port_range_start()` function defintion such that it is
    included in its body.
    
    This is an unreviewed commit.
    
    See: tor#40275
---
 src/test/test_switch_id.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/test/test_switch_id.c b/src/test/test_switch_id.c
index a911269c99..f97af55d17 100644
--- a/src/test/test_switch_id.c
+++ b/src/test/test_switch_id.c
@@ -31,6 +31,8 @@ static const struct {
   { NULL, 0 }
 };
 
+#if !defined(_WIN32)
+
 /* Returns the first port that we think we can bind to without special
  * permissions. Usually this function returns 1024. */
 static uint16_t
@@ -67,8 +69,6 @@ unprivileged_port_range_start(void)
   return result;
 }
 
-#if !defined(_WIN32)
-
 #define PORT_TEST_RANGE_START 600
 #define PORT_TEST_RANGE_END   1024
 





More information about the tor-commits mailing list