cc smarter than bb

diff --git a/src/lib/server/trunk.c b/src/lib/server/trunk.c
index f4a6231d4f05e4e7d9393a843961e7ece9e62423..39d524d986c6419c5bb082b310e37d95e5700ec3 100644 (file)
--- a/src/lib/server/trunk.c
+++ b/src/lib/server/trunk.c
@@ -677,6 +677,7 @@ do { \
 
 /** Reorder the connections in the active heap
  *
+ * fr_heap_extract will also error out if heap_id is bad - no need for assert
  */
 #define CONN_REORDER(_tconn) \
 do { \
@@ -684,7 +685,7 @@ do { \
        if ((fr_heap_num_elements((_tconn)->pub.trunk->active) == 1)) break; \
        if (!fr_cond_assert((_tconn)->pub.state == FR_TRUNK_CONN_ACTIVE)) break; \
        _ret = fr_heap_extract((_tconn)->pub.trunk->active, (_tconn)); \
-       if (!fr_cond_assert_msg(_ret == 0, "Failed extracting conn from active heap: %s", fr_strerror())) break; \      /* Will also error out if heap_id is bad - no need for assert */
+       if (!fr_cond_assert_msg(_ret == 0, "Failed extracting conn from active heap: %s", fr_strerror())) break; \
        fr_heap_insert((_tconn)->pub.trunk->active, (_tconn)); \
 } while (0)