Inheritable Smart Contract Wallet

In func. removeBeneficiary beneficiary index is not permanently deleting

In Solidity, when we delete an element at a specific index, it doesn’t actually delete. Instead, it just sets that index value = address(0)

This can cause unexpected behavior, as if we remove one element from array of four, but still it'll count four.

Manual Review While Reading Code

Instead of using remove() use pop()