Core Contracts

Summary

The RAAC protocol documentation stipulates:

"The RAACNFT is an ERC721 token representing real estate assets in the RAAC lending protocol. It implements minting functionality tied to house prices and allows for batch minting of NFTs."

and:

"Key features of the implementation include:

• Implements batch minting functionality"

The RAACNFT contrat indeeds holds a currentBatchSize variable whose value is set to 3, that can be increased with the addNewBatch function:

The problem is that this variable is never used, and no batch minting mechanism is available in the contract.

Impact

The impact of this issue is medium as there is one key feature of the RAACNFT contract is missing in the current implementation.

Tools Used

Manual review

Recommendations

Add a batchMint function to allow batch minting. Also, currentBatchSize is not strictly necessary, batch minting could be variable in size. More documentation is required about the desired behaviour for batch minting.