From 1211f156aa74537c489bf91057a042ea90dd63ae Mon Sep 17 00:00:00 2001
From: Jay Paik <jay@alchemy.com>
Date: Sun, 7 Jan 2024 17:43:17 -0500
Subject: [PATCH] fix: [spearbit-46] remove extra argument from
 _checkCallPermissions()

---
 .../session/permissions/SessionKeyPermissionsPlugin.sol      | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/plugins/session/permissions/SessionKeyPermissionsPlugin.sol b/src/plugins/session/permissions/SessionKeyPermissionsPlugin.sol
index 1350e97c..cb1d8a09 100644
--- a/src/plugins/session/permissions/SessionKeyPermissionsPlugin.sol
+++ b/src/plugins/session/permissions/SessionKeyPermissionsPlugin.sol
@@ -265,8 +265,8 @@ contract SessionKeyPermissionsPlugin is ISessionKeyPermissionsPlugin, SessionKey
             for (uint256 i = 0; i < callsLength;) {
                 Call memory call = calls[i];
                 nativeTokenSpend += call.value;
-                validationSuccess = validationSuccess
-                    && _checkCallPermissions(accessControlType, keyId, call.target, call.value, call.data);
+                validationSuccess =
+                    validationSuccess && _checkCallPermissions(accessControlType, keyId, call.target, call.data);
 
                 unchecked {
                     ++i;
@@ -336,7 +336,6 @@ contract SessionKeyPermissionsPlugin is ISessionKeyPermissionsPlugin, SessionKey
         ContractAccessControlType accessControlType,
         SessionKeyId keyId,
         address target,
-        uint256, /*value*/
         bytes memory callData
     ) internal view returns (bool validationSuccess) {
         validationSuccess = true;