Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Memory leak in ldns_dnssec_zone_new_frm_fp_l() #190

Open
FGasper opened this issue Oct 18, 2022 · 3 comments
Open

Memory leak in ldns_dnssec_zone_new_frm_fp_l() #190

FGasper opened this issue Oct 18, 2022 · 3 comments
Assignees

Comments

@FGasper
Copy link
Contributor

FGasper commented Oct 18, 2022

#186 adds memory-leak detection to the CI run, and it indicates a memory leak in this function.

What appears to be happening is that this duplicated record in 25-ZONEMD:

duplicate.example.	300	IN	TXT	"I must be digested just once"

… is represented by 2 rr structs, only 1 of which gets assigned to the dnssec_zone. The other is never freed.

@Zero3K
Copy link

Zero3K commented Feb 7, 2023

Will this be fixed soon?

@FGasper
Copy link
Contributor Author

FGasper commented Feb 7, 2023

@Zero3K I think any fix would have to be pretty invasive. I could be wrong though.

@wtoorop wtoorop self-assigned this Feb 8, 2023
@wtoorop
Copy link
Member

wtoorop commented Feb 8, 2023

I can have a look at this

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants