Detector: Detect use of inline assembly

[crytic]

Detector: Detect use of inline assembly

[gitcoinbot]

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

---

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

• If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
• Want to chip in? Add your own contribution here.
• Questions? Checkout Gitcoin Help or the Gitcoin Slack
• $47,900.92 more funded OSS Work available on the Gitcoin Issue Explorer

[gitcoinbot]

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.

I would like to start with the most simple detector and this one looks like a good candidate.

Learn more on the Gitcoin Issue Details page.

[cryptomental]

Hi. PR opened: #31

$ slither tests/inline_assembly_contract.sol 
tests/inline_assembly_contract.sol
INFO:Detectors:Old version of Solidity used in tests/inline_assembly_contract.sol: ['0.4.0']
INFO:Detectors:Assembly in tests/inline_assembly_contract.sol, Contract: GetCode, Function: at
INFO:Slither:tests/inline_assembly_contract.sol analyzed (1 contracts), 2 result(s) found

[cryptomental]

This tool is pretty cool. I will have a look at the other detectors as well.

[montyly]

Thank you for your contribution, the PR looks good!

The dev branch was out-of-date with master, so I just updated it, and added minor comments to #31.

Additionally, could you update the travis script with your testcases? (see the Adding-a-new-detector#test-the-detector section)

[cryptomental]

Hi @montyly great! I will rebase asap and update the travis script.

[cryptomental]

@montyly rebase done, I adapted the detector to use IMPACT & CONFIDENCE and added Travis tests. Could please have a look now?

[disconnect3d]

Added in #31

[montyly]

Good job @cryptomental! #31 is merged.

@mkosowsk can you validate the bounty?

Thanks!

[gitcoinbot]

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

---

Work for 50.0 DAI (50.0 USD @ $1.0/DAI) has been submitted by:

1. @cryptomental

@mkosowsk please take a look at the submitted work:

• PR by @cryptomental

---

• Learn more on the Gitcoin Issue Details page
• Want to chip in? Add your own contribution here.
• Questions? Checkout Gitcoin Help or the Gitcoin Slack
• $50,802.87 more funded OSS Work available on the Gitcoin Issue Explorer

[gitcoinbot]

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

---

The funding of 50.0 DAI (50.0 USD @ $1.0/DAI) attached to this issue has been approved & issued to @cryptomental.

• Learn more on the Gitcoin Issue Details page
• Questions? Checkout Gitcoin Help or the Gitcoin Slack
• $50,379.18 more funded OSS Work available on the Gitcoin Issue Explorer