Skip to content

Commit 1a6df5d

Browse files
committed
fix: validate input when granting/revoking drip permission
1 parent 6ef2faa commit 1a6df5d

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

contracts/reservoir/L1Reservoir.sol

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,8 @@ contract L1Reservoir is L1ReservoirV2Storage, Reservoir {
170170
* @param _dripper Address that will be an allowed dripper
171171
*/
172172
function grantDripPermission(address _dripper) external onlyGovernor {
173+
require(_dripper != address(0), "INVALID_ADDRESS");
174+
require(!allowedDrippers[_dripper], "ALREADY_A_DRIPPER");
173175
allowedDrippers[_dripper] = true;
174176
emit AllowedDripperAdded(_dripper);
175177
}
@@ -179,6 +181,8 @@ contract L1Reservoir is L1ReservoirV2Storage, Reservoir {
179181
* @param _dripper Address that will not be an allowed dripper anymore
180182
*/
181183
function revokeDripPermission(address _dripper) external onlyGovernor {
184+
require(_dripper != address(0), "INVALID_ADDRESS");
185+
require(allowedDrippers[_dripper], "NOT_A_DRIPPER");
182186
allowedDrippers[_dripper] = false;
183187
emit AllowedDripperRevoked(_dripper);
184188
}

0 commit comments

Comments
 (0)