-
Notifications
You must be signed in to change notification settings - Fork 3k
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
nsapi - Fix missing NSAPI_UNSPEC check in gethostbyname for ip literals #3232
nsapi - Fix missing NSAPI_UNSPEC check in gethostbyname for ip literals #3232
Conversation
cc @yogpan01 |
+1 , @SeppoTakalo Has already verified this fix with 6LoWPAN interface. |
Just a heads up, I'm working on adding a set of dns tests to avoid this regression in the future. Will add to this pr. |
+1 |
- test_dns_query - test_dns_query_pref - test_dns_literal - test_dns_literal_pref
10cb37c
to
b04a8da
Compare
Added gethostbyname tests |
Result: FAILUREYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 1032 Test failed! |
/morph test |
Result: ABORTEDYour command has finished executing! Here's what you wrote!
|
/morph test-nightly |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
Outputmbed Build Number: 1037 All builds and test passed! |
There is a missing check for NSAPI_UNSPEC in gethostbyname, this causes all ip literals to be rejected by default, which is definitely not the correct behaviour. May have been missed since the unspec and ip literal check prs were introduced in parallel.
cc @c1728p9, @kjbracey-arm