summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2018-10-15 10:48:35 -0400
committerNick Mathewson <nickm@torproject.org>2018-10-15 10:48:35 -0400
commit23ce9a60fbd93481f855c30e3e550acefc6052a4 (patch)
treea72d1f846e086d052fd455ffa1c135aad578d03c
parent4983322cc71d56d3b9225704b85a7c4a933a40a1 (diff)
parent3462f8ed6430bdb1528a7b63aaf2281489eb04d1 (diff)
downloadtor-23ce9a60fbd93481f855c30e3e550acefc6052a4.tar.gz
tor-23ce9a60fbd93481f855c30e3e550acefc6052a4.zip
Merge branch 'maint-0.3.4'
-rw-r--r--src/feature/hs/hs_common.c6
-rw-r--r--src/lib/log/util_bug.h7
2 files changed, 9 insertions, 4 deletions
diff --git a/src/feature/hs/hs_common.c b/src/feature/hs/hs_common.c
index 4bad4ae6e6..8dbd9485ea 100644
--- a/src/feature/hs/hs_common.c
+++ b/src/feature/hs/hs_common.c
@@ -530,7 +530,7 @@ rend_data_get_address(const rend_data_t *rend_data)
return TO_REND_DATA_V2(rend_data)->onion_address;
default:
/* We should always have a supported version. */
- tor_assert(0);
+ tor_assert_unreached();
}
}
@@ -553,7 +553,7 @@ rend_data_get_desc_id(const rend_data_t *rend_data, uint8_t replica,
return TO_REND_DATA_V2(rend_data)->descriptor_id[replica];
default:
/* We should always have a supported version. */
- tor_assert(0);
+ tor_assert_unreached();
}
}
@@ -576,7 +576,7 @@ rend_data_get_pk_digest(const rend_data_t *rend_data, size_t *len_out)
}
default:
/* We should always have a supported version. */
- tor_assert(0);
+ tor_assert_unreached();
}
}
diff --git a/src/lib/log/util_bug.h b/src/lib/log/util_bug.h
index 0786b05d77..964bc66840 100644
--- a/src/lib/log/util_bug.h
+++ b/src/lib/log/util_bug.h
@@ -103,7 +103,12 @@
} STMT_END
#endif /* defined(TOR_UNIT_TESTS) && defined(DISABLE_ASSERTS_IN_UNIT_TESTS) */
-#define tor_assert_unreached() tor_assert(0)
+#define tor_assert_unreached() \
+ STMT_BEGIN { \
+ tor_assertion_failed_(SHORT_FILE__, __LINE__, __func__, \
+ "line should be unreached"); \
+ abort(); \
+ } STMT_END
/* Non-fatal bug assertions. The "unreached" variants mean "this line should
* never be reached." The "once" variants mean "Don't log a warning more than