Skip to content

Commit

Permalink
Fixes issue apache#8329 crash in NextHopConsistentHash where when the…
Browse files Browse the repository at this point in the history 
… only

host in a strategy is unavailable due to DNS lookup failures.

- refactored NextHopConsistentHash::findNextHop() to fix the crash
  and simplify it.
- fixed the unit tests in test_NextHopConsistentHash.cc so that test
  failure checks using strcmp() do not crash with a nullptr hostname
  result.
  • Loading branch information
@jrushford
jrushford committed Sep 29, 2021
1 parent f4274a8 commit 308b4bc
Show file tree
Hide file tree
Showing 3 changed files with 223 additions and 203 deletions.
Loading

0 comments on commit 308b4bc

Please sign in to comment.