david415 / HoneyBadger

Quantum Insert detector/recorder
GNU General Public License v3.0
305 stars 39 forks source link

fix connection flow hash collision #81

Closed david415 closed 8 years ago

david415 commented 8 years ago

i noticed honeybadger panic`ed when some code asserted that the connection would be processing either it's server or client flow... however i wrote the connection pool using a bad "tcp ip flow" hash function that allowed a collision to a similar flow of a different connection. terrible... this here:

https://github.com/david415/HoneyBadger/blob/master/types/flow.go#L74-L84

david415 commented 8 years ago

fixed here: https://github.com/david415/HoneyBadger/commit/39454c65229da7e4d7c67c718a0a5092e7f0f9ad