hats-finance / Safe-0x2909fdefd24a1ced675cb1444918fa766d76bdac

A collection of modules that can be used with the Safe contract
GNU Lesser General Public License v3.0
0 stars 1 forks source link

Fallback lacking `payable` #4

Open hats-bug-reporter[bot] opened 1 week ago

hats-bug-reporter[bot] commented 1 week ago

Github username: -- Twitter username: -- Submission hash (on-chain): 0xa6a3284659a5587ce0cf8e25c0a192477e4f6047474707d2241935f16754f2c3 Severity: low

Description: Description\ If it is not marked payable, the contract will throw an exception if it receives plain ether without data.

Instances (3):

File: P256.sol

154:         address fallbackVerifier = address(uint160(Verifiers.unwrap(verifiers)));

164:         if (!success && fallbackVerifier != address(0)) {

165:             success = verifySignatureAllowMalleability(IP256Verifier(fallbackVerifier), message, r, s, x, y);

File: IP256Verifier.sol

33:     fallback(bytes calldata input) external returns (bytes memory output);
File: ISafe.sol

29:         address fallbackHandler,
0xEricTee commented 1 week ago

Invalid