fix: STAA-9 remediation commit i

livingrockrises · livingrockrises · commit 175136d2c91a · 2025-05-06T23:57:29.000+04:00
diff --git a/test/foundry/mocks/MockValidator.sol b/test/foundry/mocks/MockValidator.sol
@@ -100,6 +100,7 @@ contract MockValidator is ERC7739Validator {
     return owner == address(0) ? smartAccount : owner;
   }
 
+  /// @notice In practise should be designed to decode only necessary parameters and ignore or revert on excessive trailing data
   function onInstall(bytes calldata data) external {
     smartAccountOwners[msg.sender] = address(bytes20(data));
   }

Original file line number	Diff line number	Diff line change
`@@ -100,6 +100,7 @@ contract MockValidator is ERC7739Validator {`
`100`	`100`	`return owner == address(0) ? smartAccount : owner;`
`101`	`101`	`}`
`102`	`102`
	`103`	`+ /// @notice In practise should be designed to decode only necessary parameters and ignore or revert on excessive trailing data`
`103`	`104`	`function onInstall(bytes calldata data) external {`
`104`	`105`	`smartAccountOwners[msg.sender] = address(bytes20(data));`
`105`	`106`	`}`