Abort in test_bridges.c

Running test.exe bridges/clear_bridge_list causes this assertion:

  bridges/clear_bridge_list: May 23 17:12:53.714 [err] tor_assertion_failed_():
  Bug: container.h:70: smartlist_get: Assertion sl->num_used > idx failed;

AFAICS, since sweep_bridge_list() caused all entries in bridgelist to get deleted, an index of zero is illegal. Don't ask me why.

And since I compiled all test/*.c sources with -DDEBUG_SMARTLIST, this will trigger this abort(). I fail to see why this isn't done in the officially. \

PS1. I'm on Win-10, tor.exe + libs was built with MSVC 2017. \ PS2. Build tor.exe from master at 23 May 2018.

Trac:
Username: gvanem