Fix invalid length check in SignatureChecker (#5973)

Amxx · web-flow · commit f7da70c26e7b · 2025-10-07T09:18:29.000+02:00
diff --git a/contracts/utils/cryptography/SignatureChecker.sol b/contracts/utils/cryptography/SignatureChecker.sol
@@ -83,7 +83,7 @@ library SignatureChecker {
             mcopy(add(ptr, 0x44), signature, add(length, 0x20))
 
             let success := staticcall(gas(), signer, ptr, add(length, 0x64), 0x00, 0x20)
-            result := and(success, and(gt(returndatasize(), 0x19), eq(mload(0x00), selector)))
+            result := and(success, and(gt(returndatasize(), 0x1f), eq(mload(0x00), selector)))
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ library SignatureChecker {`
`83`	`83`	`mcopy(add(ptr, 0x44), signature, add(length, 0x20))`
`84`	`84`
`85`	`85`	`let success := staticcall(gas(), signer, ptr, add(length, 0x64), 0x00, 0x20)`
`86`		`- result := and(success, and(gt(returndatasize(), 0x19), eq(mload(0x00), selector)))`
	`86`	`+ result := and(success, and(gt(returndatasize(), 0x1f), eq(mload(0x00), selector)))`
`87`	`87`	`}`
`88`	`88`	`}`
`89`	`89`