public
functions in Auction
can be external
#12
Labels
bug
Warden finding
G (Gas Optimization)
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
Handle
pants
Vulnerability details
The
public
functionsAuction.startAuction()
,Auction.killAuction()
,Auction.initialize()
,Auction.bondForRebalance()
,Auction.settleAuction()
andAuction.addBounty()
are never called byAuction
. Therefore, their visibility can be reduced toexternal
.Impact
external
functions are cheaper thanpublic
functions.Proof of Concept
https://gus-tavo-guim.medium.com/public-vs-external-functions-in-solidity-b46bcf0ba3ac
Tool Used
Manual code review.
Recommended Mitigation Steps
Define these functions as
public
.The text was updated successfully, but these errors were encountered: