Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OxTushar - Griefing Attack Through Fake Orders and Lack of Ownership Validation in Buy Orders #1018

Open
sherlock-admin2 opened this issue Nov 25, 2024 · 0 comments

Comments

@sherlock-admin2
Copy link

sherlock-admin2 commented Nov 25, 2024

OxTushar

Medium

Griefing Attack Through Fake Orders and Lack of Ownership Validation in Buy Orders

Summary

The DebitaBorrowOffer-Factory.sol allows attackers to create fake orders with arbitrary token addresses, including those that may not exist. Additionally, there are no checks to ensure that a buy order for the same NFT is created by the same owner. This opens the system to abuse, griefing, and operational inefficiencies.

https://github.com/sherlock-audit/2024-11-debita-finance-v3/blob/main/Debita-V3-Contracts/contracts/DebitaBorrowOffer-Factory.sol#L75

https://github.com/sherlock-audit/2024-11-debita-finance-v3/blob/main/Debita-V3-Contracts/contracts/DebitaBorrowOffer-Implementation.sol#L82

Root Cause

There is no check in the borrow order contracts to verify if the collateral is a vaild collateral or not This allows attackers to create orders with invalid or non-existent token addresses.
Multiple buy orders can be created for the same NFT without validating if the buyer already has a claim or right to it.

Internal pre-conditions

No response

External pre-conditions

No response

Attack Path

No response

Impact

Attackers can create multiple fake orders with non-existent tokens, bloating the system and creating confusion.
These orders waste storage and computational resources, affecting legitimate users.
Multiple buy orders for the same NFT can lead to collisions or overwrites, making the auctioning process unreliable.
The attacker can fill the system with orders that will never resolve, effectively locking out legitimate users from efficiently participating.

PoC

No response

Mitigation

No response

@sherlock-admin3 sherlock-admin3 changed the title Sunny Pewter Kookaburra - Griefing Attack Through Fake Orders and Lack of Ownership Validation in Buy Orders OxTushar - Griefing Attack Through Fake Orders and Lack of Ownership Validation in Buy Orders Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant