rec: Fix a memory leak in DNSSEC validation #3804

Merged
merged 1 commit into from May 2, 2016

Projects

None yet

2 participants

@rgacogne
Member
rgacogne commented May 2, 2016

DNSCryptoKeyEngine::makeFromPublicKeyString() returns a naked
pointer to a new object.

@rgacogne rgacogne rec: Fix a memory leak in DNSSEC validation
`DNSCryptoKeyEngine::makeFromPublicKeyString()` returns a naked
pointer to a new object.
b275b91
@rgacogne rgacogne added this to the rec-4-alpha3 milestone May 2, 2016
@ahupowerdns ahupowerdns merged commit e7fe5d0 into PowerDNS:master May 2, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ahupowerdns
Member

(small tip, not too relevant here, but scoped_ptr or unique_ptr would save a few cycles potentially)

@rgacogne rgacogne deleted the rgacogne:rec-leak-validate branch May 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment