Fei Protocol contest - 0x1f8b's results

Handle

0x1f8b

Vulnerability details

Impact

Pay less gas.

Proof of Concept

In the line https://github.com/code-423n4/2021-11-fei/blob/add34324513b863f58e4ef7b3cd0c12d776dbb7f/contracts/PegExchanger.sol#L38 is compared a boolean value with true, in order to get a bool result, use the boolean value returned by isEnabled it's cheaper than add a extra comparation.

Tools Used

Manual review

Recommended Mitigation Steps

require(isEnabled(), "Proposals are not both passed");