Not checking families in all cases
circuit_find_to_cannibalize() compares keys when deciding if two routers are the same, but it doesn't check families.
This calls maybe for a "is r1 the same router or same family as r2" function, plus a code audit for everywhere we compare routers for equality and/or add them to smartlists.
[Automatically added by flyspray2trac: Operating System: All]