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

s2: Crash observed... #361

Closed
klauspost opened this issue Apr 22, 2021 · 0 comments · Fixed by #362
Closed

s2: Crash observed... #361

klauspost opened this issue Apr 22, 2021 · 0 comments · Fixed by #362

Comments

@klauspost
Copy link
Owner

runtime.throw(0x7e4e72, 0x5)
	C:/hostedtoolcache/windows/go/1.15.11/x64/src/runtime/panic.go:1116 +0x79 fp=0xc00be4fc70 sp=0xc00be4fc40 pc=0x2c89f9
runtime.sigpanic()
	C:/hostedtoolcache/windows/go/1.15.11/x64/src/runtime/signal_windows.go:249 +0x24f fp=0xc00be4fca0 sp=0xc00be4fc70 pc=0x2def2f
github.com/klauspost/compress/s2.encodeBlockAsm4MB(0xc000244004, 0x200004, 0x200004, 0xc00c148000, 0x200000, 0x200000, 0x1301)
	D:/a/compress/compress/s2/encodeblock_amd64.s:1255 +0xd5 fp=0xc00be5fcc8 sp=0xc00be4fca0 pc=0x743db5
github.com/klauspost/compress/s2.encodeBlock(0xc000244004, 0x200004, 0x200004, 0xc00c148000, 0x200000, 0x200000, 0x17719f7275cc0480)
	D:/a/compress/compress/s2/encode_amd64.go:28 +0x1f9 fp=0xc00be5fd18 sp=0xc00be5fcc8 pc=0x715b39
github.com/klauspost/compress/s2.Encode(0xc000244000, 0x0, 0x200008, 0xc00c148000, 0x200000, 0x200000, 0xc00c106000, 0x20cfa, 0x40007)
	D:/a/compress/compress/s2/encode.go:51 +0x25f fp=0xc00be5fd98 sp=0xc00be5fd18 pc=0x70c5bf
github.com/klauspost/compress/s2.TestEncodeNoiseThenRepeats(0xc0000c5200)
	D:/a/compress/compress/s2/s2_test.go:738 +0x186 fp=0xc00be5fed0 sp=0xc00be5fd98 pc=0x7250c6
klauspost added a commit that referenced this issue Apr 22, 2021
Fixes rare crash on amd64.

Fixes #361
klauspost added a commit that referenced this issue Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant