Beginner FriendlyFoundryNFT
100 EXP
View results
Submission Details
Severity: high
Invalid

In `Dussehra::killRavana` , it sends the eth to an arbitrary address(user)

Summary

function killRavana() public RamIsSelected {
if (block.timestamp < 1728691069) {
revert Dussehra__MahuratIsNotStart();
}
if (block.timestamp > 1728777669) {
revert Dussehra__MahuratIsFinished();
}
IsRavanKilled = true;
uint256 totalAmountByThePeople = WantToBeLikeRam.length * entranceFee;
totalAmountGivenToRam = (totalAmountByThePeople * 50) / 100;
--> (bool success, ) = organiser.call{value: totalAmountGivenToRam}("");
require(success, "Failed to send money to organiser");
}

Impact

Some exploiter can send the reward to himself rather than it going to the correct user

Tools Used

Manual review , slither

Recommendations

  1. Make the address of the user whose reward it is rather than having any arbitrary one recieving it.

Updates

Lead Judging Commences

bube Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.