Inheritable Smart Contract Wallet

Summary In func withdrawInheritedFunds if funds are sent to address with fallback func, that reverts , can break the whole array for sending the funds to addresses

Vulnerability Details Line https://github.com/CodeHawks-Contests/2025-03-inheritable-smart-contract-wallet/blob/9de6350f3b78be35a987e972a1362e26d8d5817d/src/InheritanceManager.sol#L246

(bool success,) = beneficiary.call{value: amountPerBeneficiary}("");
require(success, "something went wrong");

As explained above, the known as `push` method is not the best practice, and could DoS the contract

Impact High

Tools Used Patrick's security course ?

Recommendations use the method push over `Pull`