-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Darwin mdns fix #11992
Darwin mdns fix #11992
Conversation
Also use safe string functions.
Update the MDNS darwin fix.
PR #11992: Size comparison from ef6455d to da0d8ee Full report (33 builds for efr32, esp32, k32w, linux, nrfconnect, p6, qpg, telink)
|
I'm sort of curious. Does the resolve for interface id For some context I'm running
And then launching the
Running
|
Use the existing method to clean up the context's memory.
PR #11992: Size comparison from 1adaf7a to 1a6ce33 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving if we have verified that this fixes the problem and verified that the problem is still there after the memory fixes.
Thank you!
I think the symptoms we are seeing here are slightly different, because the dns-sd command line tool also cannot resolve an address on these computers. I think you may be correct that there is a firewall blocking something to do with bonjour on our network. Not sure how common this will be across companies, but at the very least we know the memory fix alone does not get our devs unblocked. |
PR #11992: Size comparison from aa8a6d0 to 20f5e80 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
fast track: platform specific change |
/rebase |
PR #11992: Size comparison from 3e863a9 to 5677163 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
PR #11992: Size comparison from 53721c0 to 967ab9c Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Co-authored-by: Michael Sandstedt <michael.sandstedt@gmail.com>
PR #11992: Size comparison from 8f73103 to 0a2245b Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
GetAddrInfo hangs if the interface is local only.
Fixes #11891
Change overview
Default to localhost on interface -1.
Testing
Tested by Andy - now get address correctly for devices returned on interface -1.