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

TestDSBackendParallel panic #4619

Closed
7 of 20 tasks
zl03jsj opened this issue Dec 13, 2021 · 1 comment · Fixed by #4629
Closed
7 of 20 tasks

TestDSBackendParallel panic #4619

zl03jsj opened this issue Dec 13, 2021 · 1 comment · Fixed by #4629
Assignees
Labels
C-bug Category: This is a bug V-venus venus component
Milestone

Comments

@zl03jsj
Copy link
Contributor

zl03jsj commented Dec 13, 2021

Checklist

  • This is not a security-related bug/issue. If it is, please follow please follow the security policy.
  • This is not a question or a support request. If you have any lotus related questions, please ask in the venus forum.
  • This is not a new feature request. If it is, please file a feature request instead.
  • This is not an enhancement request. If it is, please file a improvement suggestion instead.
  • I have searched on the issue tracker and the venus forum, and there is no existing related issue or discussion.
  • I am running the latest release, or the most recent RC(release canadiate) for the upcoming release or the dev branch(master), or have an issue updating to any of these.
  • I did not make any code changes to venus.

Venus component

  • venus daemon - chain sync
  • venus auth - authentication
  • venus gateway - chain service gateway
  • venus miner - mining and block production
  • venus sealer/worker - sealing
  • venus sealer - proving(WindowPoSt)
  • venus market - storage deal
  • venus market - retrieval deal
  • venus market - data transfer
  • venus light-weight client
  • venus JSON-RPC API
  • venus messager - message management (mpool)
  • Other

Venus Version

"Commit": "v1.1.3-rc1 fc255320bfb0140b9254db3d3db234ef349c53a8"

Describe the Bug

TestDSBackendParallel failed and get a panic.
it doesn't happen every time, but you guys should find out why it occurs in this certain testing.

Logging Information

=== RUN   TestDSBackendParallel
panic: runtime error: index out of range [0] with length 0

goroutine 77 [running]:
github.com/awnumar/memguard/core.(*Coffer).View(0xc0000dc6c0, 0x0, 0x0, 0x0)
	/go/pkg/mod/github.com/awnumar/[email protected]/core/coffer.go:112 +0x211
github.com/awnumar/memguard/core.Open(0xc000482018, 0xc, 0x64, 0x7f62dd1cad40)
	/go/pkg/mod/github.com/awnumar/[email protected]/core/enclave.go:101 +0x65
github.com/awnumar/memguard.(*Enclave).Open(0xc00048e000, 0x76fe4e, 0x400, 0x7f62dd1cad40)
	/go/pkg/mod/github.com/awnumar/[email protected]/enclave.go:43 +0x32
github.com/filecoin-project/venus/pkg/crypto.(*KeyInfo).UsePrivateKey(0xc000192000, 0xc0024aad68, 0x0, 0x0)
	/go/src/github.com/filecoin-project/venus/pkg/crypto/keyinfo.go:167 +0x47
github.com/filecoin-project/venus/pkg/crypto.(*KeyInfo).PublicKey(0xc000192000, 0x7f630e6691d8, 0x8, 0x52c57b, 0xc0001e80a0, 0xc0000a0c00)
	/go/src/github.com/filecoin-project/venus/pkg/crypto/keyinfo.go:157 +0x73
github.com/filecoin-project/venus/pkg/crypto.(*KeyInfo).Address(0xc000192000, 0x7f630e6691d8, 0x20, 0xc000488080, 0xc00048e000)
	/go/src/github.com/filecoin-project/venus/pkg/crypto/keyinfo.go:141 +0x2f
github.com/filecoin-project/venus/pkg/wallet.(*DSBackend).putKeyInfo(0xc0001121e0, 0xc000192000, 0xc00048e000, 0x1)
	/go/src/github.com/filecoin-project/venus/pkg/wallet/dsbackend.go:158 +0x50
github.com/filecoin-project/venus/pkg/wallet.(*DSBackend).newSecpAddress(0xc0001121e0, 0x0, 0x0, 0x0, 0x0)
	/go/src/github.com/filecoin-project/venus/pkg/wallet/dsbackend.go:139 +0xcc
github.com/filecoin-project/venus/pkg/wallet.(*DSBackend).NewAddress(0xc0001121e0, 0x1, 0x0, 0x0, 0x0, 0x0)
	/go/src/github.com/filecoin-project/venus/pkg/wallet/dsbackend.go:127 +0x15c
github.com/filecoin-project/venus/pkg/wallet.TestDSBackendParallel.func2(0xc0001121e0, 0xc002443080, 0xc00032e2c0)
	/go/src/github.com/filecoin-project/venus/pkg/wallet/dsbackend_test.go:134 +0x34
created by github.com/filecoin-project/venus/pkg/wallet.TestDSBackendParallel
	/go/src/github.com/filecoin-project/venus/pkg/wallet/dsbackend_test.go:133 +0x1c7
FAIL	github.com/filecoin-project/venus/pkg/wallet	0.741s

Repo Steps

No response

@Fatman13 Fatman13 added the C-bug Category: This is a bug label Dec 14, 2021
@Fatman13 Fatman13 added the V-venus venus component label Dec 20, 2021
@Fatman13 Fatman13 added this to the sprint 29 milestone Dec 20, 2021
@zl03jsj
Copy link
Contributor Author

zl03jsj commented Dec 24, 2021

this problem was fixed in #4629

@zl03jsj zl03jsj closed this as completed Dec 24, 2021
@Fatman13 Fatman13 linked a pull request Dec 24, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug V-venus venus component
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants