-
Notifications
You must be signed in to change notification settings - Fork 18k
syscall: Environ
uses an invalid unsafe.Pointer
conversion on Windows
#58714
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
Comments
(CC @golang/windows) |
(Pulled out from #55167 (comment).) |
Found new dashboard test flakes for:
2023-02-15 21:45 windows-amd64-race tools@d87a1002 go@965e9ba0 x/tools/gopls/internal/regtest/misc.TestCallHierarchy_Issue49125 (log)
2023-02-16 18:08 windows-amd64-race tools@e5c9e631 go@7b398b1f x/tools/gopls/internal/regtest/misc.TestGovulncheckInfo (log)
2023-02-16 19:20 windows-amd64-race tools@e5b99481 go@7b398b1f x/tools/gopls/internal/regtest/misc.TestCallHierarchy_Issue49125 (log)
2023-02-22 21:51 windows-amd64-race tools@3ec30bd9 go@ac556f35 x/tools/gopls/internal/regtest/misc.TestCallHierarchy_Issue49125 (log)
2023-02-23 13:00 windows-amd64-race tools@4906a71e go@ac556f35 x/tools/gopls/internal/regtest/misc.TestRunGovulncheckStd (log)
|
Change https://go.dev/cl/471335 mentions this issue: |
Found new dashboard test flakes for:
2023-02-27 20:01 windows-amd64-race tools@b0fcf2a0 go@1362737f x/tools/gopls/internal/regtest/misc.TestRunGovulncheckStd (log)
|
@gopherbot, please backport to Go 1.19 and 1.20. These failures also occur on at least |
Backport issue(s) opened: #58773 (for 1.19), #58774 (for 1.20). Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://go.dev/wiki/MinorReleases. |
Change https://go.dev/cl/471599 mentions this issue: |
Found new dashboard test flakes for:
2023-02-28 15:06 windows-amd64-race tools@902ee93c go@aef8a8cd x/tools/gopls/internal/regtest/misc.TestRunGovulncheckStd (log)
|
aef8a8cd is also |
Found new dashboard test flakes for:
2023-02-28 19:36 windows-amd64-race tools@120642f1 go@aef8a8cd x/tools/gopls/internal/regtest/misc.TestRunGovulncheckStd (log)
|
…n on Windows Updates #58714 Fixes #58774 Change-Id: Ifa5c059ed5e358ed98aee7e83b95dd1806b535f7 Reviewed-on: https://go-review.googlesource.com/c/go/+/471335 Reviewed-by: Than McIntosh <thanm@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org> Run-TryBot: Cuong Manh Le <cuong.manhle.vn@gmail.com> Reviewed-by: Bryan Mills <bcmills@google.com> (cherry picked from commit de8c999) Reviewed-on: https://go-review.googlesource.com/c/go/+/471599 Reviewed-by: Cuong Manh Le <cuong.manhle.vn@gmail.com> Auto-Submit: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> Run-TryBot: Bryan Mills <bcmills@google.com>
In triage: @bcmills Is this effectively done? Should this issue be closed? |
Yes, this is done; |
…n on Windows Updates golang#58714 Fixes golang#58774 Change-Id: Ifa5c059ed5e358ed98aee7e83b95dd1806b535f7 Reviewed-on: https://go-review.googlesource.com/c/go/+/471335 Reviewed-by: Than McIntosh <thanm@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org> Run-TryBot: Cuong Manh Le <cuong.manhle.vn@gmail.com> Reviewed-by: Bryan Mills <bcmills@google.com> (cherry picked from commit de8c999) Reviewed-on: https://go-review.googlesource.com/c/go/+/471599 Reviewed-by: Cuong Manh Le <cuong.manhle.vn@gmail.com> Auto-Submit: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> Run-TryBot: Bryan Mills <bcmills@google.com>
Found new dashboard test flakes for:
2023-02-25 00:08 windows-amd64-race tools@f98fce20 go@1362737f x/tools/gopls/internal/regtest/misc.TestRunGovulncheckStd (log)
|
Looks like x/tools/... uses old version of x/sys/windows package that contains this bug. Reopening this issue so we remember to update x/tools to the later version. Alex |
The pointer conversion on this line is invalid: it refers to a null-terminated string by a hard-coded array size.
The text was updated successfully, but these errors were encountered: