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

Detector: Detect the use of low level calls #27

Closed
montyly opened this issue Oct 17, 2018 · 10 comments
Closed

Detector: Detect the use of low level calls #27

montyly opened this issue Oct 17, 2018 · 10 comments
Labels
help wanted Extra attention is needed new detector

Comments

@montyly
Copy link
Member

montyly commented Oct 17, 2018

Low-level calls are more prone to errors

  • Severity: Informational

Hint: Use slithIR to detect if an operation is a LowLevelCall

@montyly montyly added help wanted Extra attention is needed new detector labels Oct 17, 2018
@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 100.0 DAI (100.0 USD @ $1.0/DAI) attached to it as part of the Ethereum Community Fund via ECF Web 3.0 Infrastructure Fund fund.

@cryptomental
Copy link
Contributor

In progress.

cryptomental added a commit to cryptomental/slither that referenced this issue Oct 18, 2018
@cryptomental
Copy link
Contributor

PR opened: #36

cryptomental added a commit to cryptomental/slither that referenced this issue Oct 18, 2018
@montyly
Copy link
Member Author

montyly commented Oct 18, 2018

Thanks for your contribution!
The PR looks good, the detector just needs a separate testcase

cryptomental added a commit to cryptomental/slither that referenced this issue Oct 18, 2018
@montyly
Copy link
Member Author

montyly commented Oct 19, 2018

PR #36 merged, thanks again @cryptomental !

@mkosowsk can you also validate the bounty? thx!

@gitcoinbot
Copy link

gitcoinbot commented Oct 19, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 7 months, 1 week from now.
Please review their action plans below:

1) cryptomental has been approved to start work.

Hi. I would like to pick up this bounty as well together with variable names detector. Estimated time for the pull request: Friday EOD at the latest.

Learn more on the Gitcoin Issue Details page.

@mkosowsk
Copy link

@cryptomental please submit your work via the the Gitcoin Issue Details page and I will send over bounty ASAP. Thanks! 👍

@cryptomental
Copy link
Contributor

cryptomental commented Oct 19, 2018

Thanks @mkosowsk ! the issue can be closed now. Also #26 can be pushed as the pull request was already merged.

@mkosowsk
Copy link

Thanks @cryptomental! But don't thank me, thank the Ethereum Community Fund 👍

See you on the next bounty!

@cryptomental
Copy link
Contributor

ECF rocks! See you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed new detector
Projects
None yet
Development

No branches or pull requests

5 participants