crytic / tealer

Static Analyzer for Teal
GNU Affero General Public License v3.0
61 stars 14 forks source link

Update MissingFeeCheck detector to use transaction field analysis #134

Closed S3v3ru5 closed 1 year ago

S3v3ru5 commented 1 year ago

The fee detector just checks for the following pattern and reports paths if an execution path does not have this pattern of instructions.

txn Fee
int <...>
[ == | != | < | <= | ..]

This PR updates the detector to use possible fee values calculated by transaction field analysis. Fee detector reports paths if it is not checked against some non-integer or is checked against a Large fee value.

S3v3ru5 commented 1 year ago

Builds on PR #132