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

[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/[email protected]/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/[email protected]/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
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-wallet venus-wallet component
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants