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
net: LookupAddr does not set IsNotFound with cgo resolver #57548
Comments
I think this CL is unrelated because it changes cgoLookupIPCNAME and not cgoLookupAddrPTR which is used by LookupAddr. |
@rudis you are right, my mistake, deleted that comment. I will make a new CL for this. |
Change https://go.dev/cl/460255 mentions this issue: |
Thanks for the quick response. CL looks good to me. |
Fixes golang#57548 Change-Id: I4092624b1e906fbcd520f43aa4bc71c144dea3b6 GitHub-Last-Rev: ab04835 GitHub-Pull-Request: golang#57549 Reviewed-on: https://go-review.googlesource.com/c/go/+/460255 Auto-Submit: Ian Lance Taylor <iant@google.com> Reviewed-by: Bryan Mills <bcmills@google.com> Reviewed-by: Damien Neil <dneil@google.com> Run-TryBot: Ian Lance Taylor <iant@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@google.com>
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
What did you expect to see?
Both go and cgo resolver correctly report
IsNotFound
.What did you see instead?
Only go resolver does so.
There's already a pull request for this but it wasn't merged: #45959
The text was updated successfully, but these errors were encountered: