dnsdist: Add the TCP socket to the map only if the connection succeeds #4734

Merged
merged 1 commit into from Dec 4, 2016

Projects

None yet

2 participants

@rgacogne
Member
rgacogne commented Dec 4, 2016

Short description

Otherwise we create a value-initialized (to 0) entry in the map and try to use it later, as reported by @philmayers and tracked down by @ahupowerdns in #4733.
Fixes #4733.

Checklist

I have:

@rgacogne rgacogne dnsdist: Add the TCP socket to the map only if the connection succeeds
Otherwise we create a value-initialized (to 0) entry in the map and
try to use it later.
5a99e22
@rgacogne rgacogne added this to the dnsdist-1.1.0 milestone Dec 4, 2016
@ahupowerdns ahupowerdns merged commit de08b9a into PowerDNS:master Dec 4, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rgacogne rgacogne deleted the rgacogne:dnsdist-tcp-ssocket-exception branch Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment