40,000 USDC
View results
Submission Details
Severity: gas
Valid

[INFO] `Escrow` and `IEscrow` miss full natspec documentation

Summary

The current implementation of Escrow and IEscrow are partially supporting the Solidity NatSpec Format.

Vulnerability Details

These are the following natspec documentation that is missing

  • Escrow should add natspec at contract level to explain the contract scope

  • Escrow should add natspec documentation for the constructor

  • Escrow should add natspec documentation (@inheritdoc IEscrow) for the following functions: getPrice, getTokenContract, getBuyer, getSeller, getArbiter, getArbiterFee and getState

  • IEscrow should add natspec at interface level to explain the interface scope

  • IEscrow should add full-natspec support to the function confirmReceipt

  • IEscrow should add full-natspec support to the function initiateDispute

  • IEscrow should add full-natspec support to the function resolveDispute

  • IEscrow should add full-natspec support to the function getPrice

  • IEscrow should add full-natspec support to the function getTokenContract

  • IEscrow should add full-natspec support to the function getBuyer

  • IEscrow should add full-natspec support to the function getSeller

  • IEscrow should add full-natspec support to the function getArbiter

  • IEscrow should add full-natspec support to the function getArbiterFee

  • IEscrow should add full-natspec support to the function getState

Impact

Better UX and DX

Tools Used

Manual

Recommendations

The client should consider adding the full natspec documentation support to the Escrow and IEscrow contracts.

Support

FAQs

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