fix signature length issue

Author: amiecorso

src/EIP7702Proxy.sol

@@ -90,12 +90,14 @@ contract EIP7702Proxy is Proxy {
                 }
             }
 
-            // Otherwise try ecrecover
-            address recovered = ECDSA.recover(hash, signature);
-            if (recovered == address(this)) {
-                assembly {
-                    mstore(0, ERC1271_MAGIC_VALUE)
-                    return(0, 32)
+            // Only try ECDSA if signature is the right length (65 bytes)
+            if (signature.length == 65) {
+                address recovered = ECDSA.recover(hash, signature);
+                if (recovered == address(this)) {
+                    assembly {
+                        mstore(0, ERC1271_MAGIC_VALUE)
+                        return(0, 32)
+                    }
                 }
             }
 

test/EIP7702Proxy/isValidSignature.t.sol

@@ -102,7 +102,11 @@ contract IsValidSignatureTest is EIP7702ProxyBase {
     function testEmptySignature() public {
         bytes memory emptySignature = "";
 
-        vm.expectRevert();
         bytes4 result = wallet.isValidSignature(testHash, emptySignature);
+        assertEq(
+            result,
+            ERC1271_FAIL_VALUE,
+            "Should reject empty signature"
+        );
     }
 }