Submission Details
Severity:
Misleading documentation: deposit function lacks documented functionality
Summary
The deposit() function's documentation claims it allows users to sign up other users, but this functionality is not implemented in the code. This discrepancy between documentation and implementation could lead to confusion for developers and integrators.
Vulnerability Details
The deposit() function comment vs implementation mismatch:
ChristmasDinner.sol#L108
/**
* ...
* Allows a user to sign-up other users.
* ..
*/
Impact
Misleading documentation
Potential integration errors
Developer confusion
Tools Used
Manual code review
Recommendations
Either implement the documented functionality or correct the documentation to match the implementation.
Updates
Lead Judging Commences
0xtimefliez about 1 year ago
Submission Judgement Published Assigned finding tags:
Deposit function lacks functionality
Rewards breakdown
nSLOC
129This 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.