search

code-423n4 / 2022-06-putty-findings

5 stars 0 forks source link

Gas Optimizations #401

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

[G-06] Remove unnecesary requires

On PuttyV2.sol#L293 and PuttyV2.sol#L598 there is no need to check if address is a contract, safeTransferFrom will take care, also this check does not help because it does not confirm that it is an ERC20

JeeberC4 commented 2 years ago

Warden submitted multiple Gas Optimizations. Will not be judged.

eugenioclrc commented 2 years ago

Please discar this submit, and judge only https://github.com/code-423n4/2022-06-putty-findings/issues/260