[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lwip-devel] [patch #9555] Add MDNS probing

From: Jens Nielsen
Subject: [lwip-devel] [patch #9555] Add MDNS probing
Date: Tue, 3 Jul 2018 06:16:07 -0400 (EDT)
User-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3396.99 Safari/537.36

Follow-up Comment #19, patch #9555 (project lwip):

Yes, sorry I meant to go through the code and remaining todos and do some
tests but I think it looks good. I'll open a new patch if I find something
useful to share.

The only minor comment from my side is that the two
4506db4331ad1426e5589701d7b193a074aabfee should be unnecessary since there is
an LWIP_ERROR to return an error if len > MDNS_LABEL_MAXLEN? Or otherwise the
string null termination on the line below also needs LWIP_MIN

Anyway that's not important. Thanks a lot for the help Simon!


Reply to this item at:


  Message sent via Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]