Core Contracts

Regnum Aurum Acquisition Corp
HardhatReal World AssetsNFT
77,280 USDC
View results
Submission Details
Severity: high
Invalid

Improper Loan Record Deletion Causing Both Healthy and Unhealthy NFTs To Be Deleted

Summary

The function deletes all NFT loan records using delete user.nftTokenIds, removing tracking information for all loans regardless of their status.

Vulnerability Details

This single line:

  • Erases all NFT loan records

  • Removes tracking capability for active loans

  • Destroys historical data

  • Makes recovery or dispute resolution impossible

```solidity

delete user.nftTokenIds;

```

Impact

  • Loss of loan tracking information

  • Accounting system inconsistencies

  • Impossible to verify loan histories

  • Difficulty in handling disputes

  • Complex recovery scenarios

  • Potential double-liquidation risks

Tools Used

Manual review

Recommendations

There are multiple ways to mitigate this but first if the protocol intend not to keep track of the liquidated NFT, they should delete only the liquidated NFT not all the NFTs owned by the user. If they intend to keep track of liquidated NFTs they should be a loan tracking struct to keep track of all liquidated and non-liquidated NFTs

Updates

Lead Judging Commences

inallhonesty Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement
inallhonesty Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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