Spec error on function: Factory:approveTemplate
#296
Labels
1 (Low Risk)
Assets are not at risk. State handling, function incorrect as to spec, issues with comments
bug
Something isn't working
resolved
Finding has been patched by sponsor (sponsor pls link to PR containing fix)
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
Handle
Dravee
Vulnerability details
The spec doesn't match with the comments in the code here:
Code: https://github.com/code-423n4/2022-01-insure/blob/main/contracts/Factory.sol#L90-L91
Spec: https://insuredao.gitbook.io/developers/market/factory#approvetemplate
Here, the spec doesn't mention
_isOpen
and seem to confuse the_approval
description with what_isOpen
should be.Tools Used
VS Code
Recommended Mitigation Steps
My guess is that the spec should be corrected
The text was updated successfully, but these errors were encountered: