NLnetLabs / ldns

LDNS is a DNS library that facilitates DNS tool programming
https://nlnetlabs.nl/ldns
BSD 3-Clause "New" or "Revised" License
292 stars 97 forks source link

Issue 147 tabs between cstrings #148

Closed FGasper closed 2 years ago

wtoorop commented 2 years ago

This looks like a really good bugfix, not only for 1.8.1 but also for a bug that was already in ldns, because I guess there existed already an issue with multiple character whitespace before quoted rdata fields containing tabs, right? Later today I'd like to have another good look before merging, but it looks really good and I'm very happy you found the issue, analyzed it so thoroughly and then also provided such an extensive fix including a tpkg test! Thanks again @FGasper !