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 30, 2021
1 parent 44d1be6 commit 16e21c7
Show file tree
Hide file tree
Showing 3 changed files with 212 additions and 202 deletions.
Loading

0 comments on commit 16e21c7

Please sign in to comment.