Early rate$2,400 of senior audit time for $500. Early members keep the rate as it climbs.$2,400 of senior audit time for $500See how →
F-2025-0011·gas-optimization

Redundant allowance check

Acknowledgedstakingnft-boostrewards
TL;DR

stakeTokens duplicates the allowance check that safeTransferFrom already performs internally, costing extra gas for no additional safety.

Severity
INFO
Impact
LOW
Likelihood
LOW
Method
MManual review
CAT.
Complexity
LOW
Exploitability
LOW
02Section · Description

Description

The stakeTokens function performs an explicit allowance check before calling safeTransferFrom:

solidity
uint allowed = stakingToken.allowance(msg.sender, address(this));
require(allowed >= amount, "Allowance too low. Please approve more tokens.");
stakingToken.safeTransferFrom(msg.sender, address(this), amount);

This check is redundant since safeTransferFrom already includes an allowance verification. The duplicate check results in unnecessary gas consumption from an extra external call.

F-2025-0011