Summary

In SpiceAuction::setAuctionConfig, it checks if current auction is active and if so reverts because, in accordance with the docs, it cannot set config for ongoing auctions. However, setAuctionConfig sets the configuration for the next epoch so it should not revert.

Vulnerability Details

When setAuctionConfig is invoked, the configuration for the next epoch (i.e. auctionConfigs[_currentEpochId + 1] is set). There is no reason to revert this function if the current auction is active.

Impact

Reverts a function call impeding setting the next epoch auction configuration when there is no need for it.

Tools Used

Manual review

Recommendations

No need to revert if current auction is active.