search

Consensys / mythx-developer-support

Support resources and bug reporting for Mythril Tool Developers
3 stars 1 forks source link

Unused variables are not signaled as warning. #18

Closed caleteeter closed 5 years ago

caleteeter commented 5 years ago

When analyzing a contract, unused variables should be shown as a warning at minimum.

rocky commented 5 years ago

If this is the case, right now this would be a weakness in the linter/static analyzer used on the MythX side. It should be able to detect this directly from solc's AST.

@thec00n care to comment?

caleteeter commented 5 years ago

I can repo pretty easily, can share if you need.

rocky commented 5 years ago

Yes, please do. Thanks.

nbanmp commented 5 years ago

As this is not an issue directly related to truffle-security itself, but instead with the api behind it, I have transferred the issue to the mythx-developer-support repository.

muellerberndt commented 5 years ago

Unused variables are now reported.