Submission Details
Severity:
Missing event in `setTreatCost` function
Summary
Fuction setTreatCost` changes the price of a selected NFT and certainly changes the state of the contract. In general, if you change a storage variable or move Ether in and out of the contract, you should emit an event.
Vulnerability Details
function setTreatCost(
string memory _treatName,
uint256 _cost
) public onlyOwner {
require(treatList[_treatName].cost > 0, "Treat must cost something.");
treatList[_treatName].cost = _cost;
- missing event
}
Impact
The information about changing price will not be logged.
Recommendations
Add event , f.e event priceChanged(uint256 newPrice, string treatName)
Rewards breakdown
nSLOC
109This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.