Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Commit

Permalink
fix for signature validation to properly check null address
Browse files Browse the repository at this point in the history
  • Loading branch information
Andrew Redden committed Dec 10, 2022
1 parent 9cfa4c3 commit e447ab5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/VaultImplementation.sol
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,8 @@ abstract contract VaultImplementation is
params.lienRequest.s
);
if (
recovered != owner() && recovered != s.delegate && recovered != address(0)
(recovered != owner() && recovered != s.delegate) ||
recovered == address(0)
) {
revert IVaultImplementation.InvalidRequest(
InvalidRequestReason.INVALID_SIGNATURE
Expand Down

0 comments on commit e447ab5

Please sign in to comment.