Summary

In the divideNft function, the modifier onlyNftOwner(nftAddress, tokenId) is applied twice:

Impact

This redundancy leads to the same ownership check being executed twice during function execution, which is unnecessary and inefficient.

The repetitive use of the modifier increases gas consumption for every execution of the function since the same logic is run twice.

Tools Used

Manual review

Recommendations