litentry / litentry-graph

A GraphQL Server providing aggregated blockchain identity data
Apache License 2.0
7 stars 0 forks source link

Research the rule of EVM padding code #257

Closed wangminqi closed 2 years ago

wangminqi commented 2 years ago

https://etherscan.io/tx/0x92a0d3e9190cdb9ae986e95a922452bb864835130e7c382265a6c6689d71fa19 The sample shows that in the past version of code, address value can be padded with random value. It may not be limited to address (other fixed length datatype is also potential). We do not know if we manually padding the data, can EVM still recognize.

The decoder needs a rule for handling such type of error. We need some research and test about this topic.

geastwood commented 2 years ago

~If I understand correctly we currently throw an error in this case. However we want to do research to be able to decode in such case?~

NVM, I see the discussion here.

https://litentry.slack.com/archives/C02C0QH27CY/p1655910731536639

bdmason commented 2 years ago

this is the wrong repository for issues, this is where we do EVM work https://github.com/litentry/evm-processor

geastwood commented 2 years ago

FYI: https://github.com/litentry/evm-processor/issues/102