Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[venus-wallet] 新的签名策略会导致签名时panic #5523

Closed
11 tasks
Tracked by #5527
diwufeiwen opened this issue Dec 2, 2022 · 0 comments · Fixed by filecoin-project/venus-wallet#115
Closed
11 tasks
Tracked by #5527
Assignees
Labels
C-bug Category: This is a bug V-venus-wallet venus-wallet component

Comments

@diwufeiwen
Copy link
Contributor

链服务模块 / Chain Service Components

  • venus
  • venus-auth
  • venus-gateway
  • venus-messager
  • venus-miner
  • 文档 / docs

订单服务模块 / Deal Service Components

  • venus-market
  • 文档 / docs

算力服务模块 / Storage Power Service Components

  • venus-sector-manager
  • venus-worker
  • 文档 / docs

版本 / Version

./venus-wallet -v
venus remote-wallet version 1.9.0-pre-rc1'+git83b33ee'

描述 / Describe the Bug

见标题

日志 / Logging Information

2022-12-02T09:54:02.045+0800    INFO    wallet_event    walletevent/wallet_client.go:132        connect to server success a3b5c77f-a40f-42b4-9199-5b520d93f472       {"api hub": "/ip4/127.0.0.1/tcp/45132"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0xf06ae1]

goroutine 223 [running]:
github.com/filecoin-project/venus-wallet/storage/wallet.(*SignFilter).CheckSignMsg(0xc000576480?, {0x57?, 0x57?}, {{0x13b6038?, 0x0?}, {0x1347d20?, 0xc000166070?}})
        /root/thelitao/venus-wallet/storage/wallet/sign_filter.go:33 +0x41
github.com/filecoin-project/venus-wallet/storage/wallet.(*wallet).WalletSign(0xc00068f920, {0x161b190, 0xc000652e40}, {{0xc00007c140?, 0x1?}}, {0xc000168ae0, 0x26, 0x27}, {{0xc000288118, 0x7}, ...})
        /root/thelitao/venus-wallet/storage/wallet/wallet.go:166 +0x231
github.com/ipfs-force-community/venus-gateway/walletevent.(*WalletEventClient).walletSign(0xc0001485b0, {0x161b190, 0xc000652e40}, 0xc0005020c0)
        /root/go/pkg/mod/github.com/ipfs-force-community/venus-gateway@v1.8.0/walletevent/wallet_client.go:167 +0x2d8
created by github.com/ipfs-force-community/venus-gateway/walletevent.(*WalletEventClient).listenWalletRequestOnce
        /root/go/pkg/mod/github.com/ipfs-force-community/venus-gateway@v1.8.0/walletevent/wallet_client.go:138 +0x41b

重现步骤 / Repo Steps

venus-wallet run
创建钱包
用此钱包发消息

@diwufeiwen diwufeiwen added the C-bug Category: This is a bug label Dec 2, 2022
@diwufeiwen diwufeiwen changed the title [venus-wallet] 新的签名策略会导致签名是panic [venus-wallet] 新的签名策略会导致签名时panic Dec 2, 2022
@Fatman13 Fatman13 added the V-venus-wallet venus-wallet component label Dec 2, 2022
@diwufeiwen diwufeiwen mentioned this issue Dec 5, 2022
42 tasks
@simlecode simlecode self-assigned this Dec 7, 2022
@simlecode simlecode moved this to In Progress in Venus Project Dec 7, 2022
Repository owner moved this from In Progress to Done in Venus Project Dec 7, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
C-bug Category: This is a bug V-venus-wallet venus-wallet component
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants