[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [tor/master] Fix an enum comparison that was blowing up jenkins.



commit baee2feddd4504e0158795f729f151cdf9a2ca1b
Author: George Kadianakis <desnacked@xxxxxxxxxx>
Date:   Thu May 21 13:39:49 2020 +0300

    Fix an enum comparison that was blowing up jenkins.
    
    The warning was:
        11:23:10 ../tor/src/feature/hs/hs_service.c: In function 'log_cant_upload_desc':
        11:23:10 ../tor/src/feature/hs/hs_service.c:3118:3: error: comparison of unsigned expression < 0 is always false [-Werror=type-limits]
    See #34254 for more info.
    
    I guess this means that gcc assigned an unsigned type to the
    `log_desc_upload_reason_t` enum and it warned if we compared it against 0...
    
    For now I think it's simpler to remove that check instead of turning the enum
    to a signed type, or trying to hack it some other way.
    
    From what it seems, enum is up to the compiler on whether it's signed/unsigned:
         https://stackoverflow.com/questions/159034/are-c-enums-signed-or-unsigned
---
 src/feature/hs/hs_service.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/feature/hs/hs_service.c b/src/feature/hs/hs_service.c
index 5b8f66183..bff0d11e4 100644
--- a/src/feature/hs/hs_service.c
+++ b/src/feature/hs/hs_service.c
@@ -3115,7 +3115,7 @@ log_cant_upload_desc(const hs_service_t *service,
    * control that value in the code flow but will be apparent during
    * development if a reason is added but LOG_DESC_UPLOAD_REASON_NUM_ is not
    * updated. */
-  if (BUG(reason > LOG_DESC_UPLOAD_REASON_MAX || reason < 0)) {
+  if (BUG(reason > LOG_DESC_UPLOAD_REASON_MAX)) {
     return;
   }
 



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits