Skip to content
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

x/crypto: TestDsa failure on Linux ARM #36164

Closed
toothrot opened this issue Dec 16, 2019 · 1 comment
Closed

x/crypto: TestDsa failure on Linux ARM #36164

toothrot opened this issue Dec 16, 2019 · 1 comment
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone

Comments

@toothrot
Copy link
Contributor

What version of Go are you using (go version)?

Tip

Does this issue reproduce with the latest release?

Also failing on release-branch.go1.13 and 1.12

What operating system and processor architecture are you using (go env)?

(ARM buildlet)

What did you do?

https://build.golang.org/log/610cd72794e4757d8207300f9740caed3cb65ee4

What did you expect to see?

OK

What did you see instead?

--- FAIL: TestDsa (0.11s)
panic: runtime error: slice bounds out of range [:-2147483643] [recovered]
	panic: runtime error: slice bounds out of range [:-2147483643]

goroutine 34 [running]:
testing.tRunner.func1(0x5a40a0)
	/workdir/go/src/testing/testing.go:916 +0x534
panic(0x19d9a0, 0x66d7c0)
	/workdir/go/src/runtime/panic.go:967 +0x118
golang.org/x/crypto/cryptobyte.(*String).read(...)
	/workdir/gopath/src/golang.org/x/crypto/cryptobyte/string.go:30
golang.org/x/crypto/cryptobyte.(*String).ReadBytes(...)
	/workdir/gopath/src/golang.org/x/crypto/cryptobyte/string.go:144
golang.org/x/crypto/cryptobyte.(*String).readASN1(0x5c1e88, 0x5c1e94, 0x5c1e5c, 0x191d01, 0x59c101)
	/workdir/gopath/src/golang.org/x/crypto/cryptobyte/asn1.go:743 +0x32c
golang.org/x/crypto/cryptobyte.(*String).ReadAnyASN1(...)
	/workdir/gopath/src/golang.org/x/crypto/cryptobyte/asn1.go:546
golang.org/x/crypto/cryptobyte.(*String).ReadASN1(0x5c1e88, 0x5c1e94, 0x59de30, 0x166ce4)
	/workdir/gopath/src/golang.org/x/crypto/cryptobyte/asn1.go:521 +0x3c
golang.org/x/crypto/internal/wycheproof/internal/dsa.VerifyASN1(0x5f7ee0, 0x664c60, 0x14, 0x20, 0x59de30, 0x33, 0x70, 0x0)
	/workdir/gopath/src/golang.org/x/crypto/internal/wycheproof/internal/dsa/dsa.go:25 +0x80
golang.org/x/crypto/internal/wycheproof.TestDsa(0x5a40a0)
	/workdir/gopath/src/golang.org/x/crypto/internal/wycheproof/dsa_test.go:117 +0x210
testing.tRunner(0x5a40a0, 0x1bac0c)
	/workdir/go/src/testing/testing.go:954 +0xbc
created by testing.(*T).Run
	/workdir/go/src/testing/testing.go:1005 +0x2b0
FAIL	golang.org/x/crypto/internal/wycheproof	20.094s

/cc @katiehockman @FiloSottile

@toothrot toothrot added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Dec 16, 2019
@toothrot toothrot modified the milestones: Backlog, Unreleased Dec 16, 2019
@gopherbot
Copy link
Contributor

Change https://golang.org/cl/212625 mentions this issue: internal/wycheproof: skip failing test on arm architectures

@golang golang locked and limited conversation to collaborators Dec 26, 2020
c-expert-zigbee pushed a commit to c-expert-zigbee/crypto_go that referenced this issue Mar 28, 2022
Fixes golang/go#36164

Change-Id: I4248cb3e25346f6859c473b729811b36ac6cf872
Reviewed-on: https://go-review.googlesource.com/c/crypto/+/212625
Reviewed-by: Filippo Valsorda <[email protected]>
c-expert-zigbee pushed a commit to c-expert-zigbee/crypto_go that referenced this issue Mar 29, 2022
Fixes golang/go#36164

Change-Id: I4248cb3e25346f6859c473b729811b36ac6cf872
Reviewed-on: https://go-review.googlesource.com/c/crypto/+/212625
Reviewed-by: Filippo Valsorda <[email protected]>
c-expert-zigbee pushed a commit to c-expert-zigbee/crypto_go that referenced this issue Mar 29, 2022
Fixes golang/go#36164

Change-Id: I4248cb3e25346f6859c473b729811b36ac6cf872
Reviewed-on: https://go-review.googlesource.com/c/crypto/+/212625
Reviewed-by: Filippo Valsorda <[email protected]>
LewiGoddard pushed a commit to LewiGoddard/crypto that referenced this issue Feb 16, 2023
Fixes golang/go#36164

Change-Id: I4248cb3e25346f6859c473b729811b36ac6cf872
Reviewed-on: https://go-review.googlesource.com/c/crypto/+/212625
Reviewed-by: Filippo Valsorda <[email protected]>
BiiChris pushed a commit to BiiChris/crypto that referenced this issue Sep 15, 2023
Fixes golang/go#36164

Change-Id: I4248cb3e25346f6859c473b729811b36ac6cf872
Reviewed-on: https://go-review.googlesource.com/c/crypto/+/212625
Reviewed-by: Filippo Valsorda <[email protected]>
desdeel2d0m added a commit to desdeel2d0m/crypto that referenced this issue Jul 1, 2024
Fixes golang/go#36164

Change-Id: I4248cb3e25346f6859c473b729811b36ac6cf872
Reviewed-on: https://go-review.googlesource.com/c/crypto/+/212625
Reviewed-by: Filippo Valsorda <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Development

No branches or pull requests

2 participants