ArduinoMDNS icon indicating copy to clipboard operation
ArduinoMDNS copied to clipboard

Service query is overflowing

Open Luftzig opened this issue 5 years ago • 3 comments

by sending the length the header instead of the just 4 needed bytes. See #7

Luftzig avatar Jul 24 '20 16:07 Luftzig

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Yoav Luft seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Apr 09 '21 13:04 CLAassistant

This PR fixes a critical bug!

Without it we get weird buffer overruns in the processing logic that cause malformed packets.

@agdl, @marqdevx, does this library have any maintainers?

If not you can add me as one and I will upstream this as well as try to merge the sensible future commits from trippylightning's branch.

gigaj0ule avatar Jun 20 '22 19:06 gigaj0ule

Removing a call to addServiceRecord made this library way more reliable for me @aentinger

zyphlar avatar Apr 27 '24 05:04 zyphlar