The if-branch of Basket.changeLicenseFee function ensures that pendingLicenseFee.licenseFee == newLicenseFee which means setting licenseFee = newLicenseFee is equivalent to licenseFee = pendingLicenseFee.licenseFee but the former saves an expensive storage load operation.
GalloDaSballo
commented
2 years ago
Handle
cmichel
Vulnerability details
The
if-branch ofBasket.changeLicenseFeefunction ensures thatpendingLicenseFee.licenseFee == newLicenseFeewhich means settinglicenseFee = newLicenseFeeis equivalent tolicenseFee = pendingLicenseFee.licenseFeebut the former saves an expensive storage load operation.