Skip to content

Commit

Permalink
Merge pull request #15 from Nexus-2023/audit_fixes
Browse files Browse the repository at this point in the history
removed additional checks
  • Loading branch information
RohitAudit authored Jan 25, 2024
2 parents 30388e8 + bc74767 commit b4a7080
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion contracts/nexus_bridge/NexusBridgeDAO.sol
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ abstract contract NexusBridgeDAO is NexusBaseBridge {
validatorCount -= 1;
}

function redeemRewards(address reward_account,uint256 expectedFee) external onlyDAO validNexusFee(NexusFeePercentage){
function redeemRewards(address reward_account,uint256 expectedFee) external onlyDAO{
if(expectedFee!=NexusFeePercentage) revert IncorrectNexusFee();
uint256 total_rewards = getRewards();
if(total_rewards > VALIDATOR_DEPOSIT) revert WaitingForValidatorExits();
Expand Down
2 changes: 1 addition & 1 deletion contracts/nexus_bridge/nexusLibrary.sol
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ contract NexusLibrary {
slashedAmount;
}

function redeemRewards(address reward_account,uint256 expectedFee) external onlyDAO validNexusFee(getVariable(NEXUS_FEE_PERCENTAGE_SLOT)) {
function redeemRewards(address reward_account,uint256 expectedFee) external onlyDAO {
uint256 NexusFeePercentage = getVariable(NEXUS_FEE_PERCENTAGE_SLOT);
if(expectedFee!=NexusFeePercentage) revert IncorrectNexusFee();
uint256 total_rewards = getRewards();
Expand Down

0 comments on commit b4a7080

Please sign in to comment.