troglobit / mdnsd

Jeremie Miller's original mdnsd
BSD 3-Clause "New" or "Revised" License
57 stars 35 forks source link

Fix fuzzer findings #29

Closed wolframroesler closed 4 years ago

wolframroesler commented 4 years ago

I ran the AFL fuzz tester on mDNS and found several crashes and hangs, mostly because packet parser errors were ignored and the resulting half-baked message object was processed anyway. With this PR, internal parser errors are forwarded to the caller, and the packet is skipped.

troglobit commented 4 years ago

Fantastic, thank you so much for this! <3