QA Report

[code423n4]

Withdrawer.sol

• L26 - The withdraw() function requires an approve from the WETH contract, it sounds strange that a withdraw() function requires an approve, therefore it would be easier to understand for a user with another name.

BeefyVaultStorage.sol

• L5 - IERC20 is imported but not used.

StakingLPVaultHelpers.sol

• L4 - Withdrawer is imported but not used.

[obatirou]

Withdrawer.sol - L26 (disputed)

Disagree, we want to withdraw from the WETH contract, and a lot of withdraw functions need an approval.

[obatirou]

L4 - Withdrawer is imported but not used (duplicate)

Duplicate https://github.com/code-423n4/2022-06-nested-findings/issues/40#issuecomment-1167044373

L5 - IERC20 is imported but not used (duplicate)

Duplicate https://github.com/code-423n4/2022-06-nested-findings/issues/40#issuecomment-1167044373