Omniscia rain protocol Audit
ITier Manual Review Findings
ITier Manual Review Findings
ITR-01M: Report Encoding Discrepancy
Type | Severity | Location |
---|---|---|
Standard Conformity | ITier.sol:L37 |
Description:
The linked documentation line goes against what the TierReport
implementation indicates and would cause it to significantly malfunction as it expects 0xFFFFFFFF
to be yielded for a tier that has never been held or is unknown.
Example:
36/// - If the historical block information is not available the report MAY37/// return `0x00000000` for all held tiers.
Recommendation:
We advise the documentation to be updated so as to not mislead implementors of the tier system. While the MAY
RFC 2119 terminology is appreciated, it can still prove misleading given that it leads to the exact opposite effect of the TierReport
implementation.
Alleviation:
After discussion with the Rain Protocol team, we concluded that the original text was correct albeit slightly ambiguous and as such they proceeded with updating the documentation extensively and explicitly to avoid any similar ambiguities in the future. Given that the original text was correct, we consider this exhibit nullified.