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.

Sign up for GitHub

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

Jump to bottom

ABI MethodById panic #17797

Closed
ququzone opened this issue Oct 1, 2018 · 2 comments
Closed

ABI MethodById panic #17797

ququzone opened this issue Oct 1, 2018 · 2 comments

Comments

@ququzone
Copy link

ququzone commented Oct 1, 2018 •
edited
Loading

when parse #4999272 block, i use erc20 abi get method panic, transaction hash is : 0xbefd2f3a0288731f4db3e24995a1e45c68403a3991e64f00f62b800c22e778c4

m, err := ERC20ABI.MethodById(tx.Data())
@holiman
Copy link
Contributor

holiman commented Oct 1, 2018

Could you provide the stack trace for the panic?

@holiman
Copy link
Contributor

holiman commented Oct 1, 2018

Ah, nvm. There is no data: https://etherscan.io/tx/0xbefd2f3a0288731f4db3e24995a1e45c68403a3991e64f00f62b800c22e778c4 that's why it crashes.

holiman added a commit to holiman/go-ethereum that referenced this issue Oct 1, 2018
@holiman
accounts/abi: fix panic in MethodById lookup. Fixes ethereum#17797
23ca841
@gballet gballet closed this as completed in 96fd50b Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@holiman @ququzone