From 033691212a49d3d32f8a684a493c8c11b8887eee Mon Sep 17 00:00:00 2001 From: teor Date: Fri, 15 Sep 2017 14:02:22 +1000 Subject: Make an assert into a BUG warning in the bridge code If future code asks if there are any running bridges, without checking if bridges are enabled, log a BUG warning rather than crashing. Fixes 23524 on 0.3.0.1-alpha --- src/or/bridges.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'src/or/bridges.c') diff --git a/src/or/bridges.c b/src/or/bridges.c index 812b0ddce8..1eec4e39ec 100644 --- a/src/or/bridges.c +++ b/src/or/bridges.c @@ -836,7 +836,9 @@ learned_bridge_descriptor(routerinfo_t *ri, int from_cache) MOCK_IMPL(int, any_bridge_descriptors_known, (void)) { - tor_assert(get_options()->UseBridges); + if (BUG(!get_options()->UseBridges)) { + return 0; + } if (!bridge_list) return 0; -- cgit v1.2.3-54-g00ecf