search

uber-archive / statsrelay

A consistent-hashing relay for statsd and carbon metrics
Other
101 stars 28 forks source link

Deref bugs #42

Closed artms closed 7 years ago

eklitzke commented 9 years ago

A few things:

  1. I'm pretty sure we never actually pass NULL lists to these functions
  2. Coverity can already check if there are places where list can be NULL
  3. Coverity seems to actually complaining if you have extraneous NULL checks (although I could be misinterpreting this)
  4. These would immediately cause segfaults if it was a real bug, and I haven't seen any production segfaults related to this