crytic / crytic/slither

Yul calldata offset and length support

Open
#1,056 2 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
6.4k
Forks
1.1k
PR merge metrics
No merged PRs in 30d

Description

### Describe the desired feature

In solidity 0.7.5, `.offset` and `.length` were added to calldata variables of dynamic arrays in yul as described in https://blog.soliditylang.org/2020/11/18/solidity-0.7.5-release-announcement/.

It appears that currently slither is missing support for those cause code that makes use of that to fail inside the Yul parser. Support should be similar to the existing support of `.slot` and `.offset` for storage variables.

I'm planning on submitting a PR with a proposed fix, but I wanted to open an issue as well in case their are issues with the PR

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.