crytic / crytic/slither

Improper usage of ecrecover

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

Description

### Describe the desired feature

Any use of `ecrecover()` outside of generally accepted libraries should be flagged as a finding deserving deeper investigation.

If we want this to be a more sophisticated detector, figure out a way for it to check statements 310-313 from the ethereum yellow paper:
![image](https://github.com/crytic/slither/assets/1222451/991fd75e-5d1c-4b81-919a-72a17a4a57c8)

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.