sherlock-audit / 2024-08-cork-protocol-judging

2 stars 2 forks source link

Mammoth Laurel Nightingale - [L-1] Empty Function Body - Consider commenting why #283

Closed sherlock-admin4 closed 2 months ago

sherlock-admin4 commented 2 months ago

Mammoth Laurel Nightingale

Low/Info

[L-1] Empty Function Body - Consider commenting why

Summary

Vulnerability Detail

Impact

Code Snippet

Instances (7):

File: core/ModuleCore.sol

31:     {}
File: core/assets/Asset.sol

79:     {}
File: core/assets/AssetFactory.sol

195:     function _authorizeUpgrade(address newImplementation) internal override onlyOwner notDelegated {}
File: core/flash-swaps/FlashSwapRouter.sol

41:     function _authorizeUpgrade(address newImplementation) internal override onlyOwner notDelegated {}
File: dummy/DummyERC20WithMetadata.sol

13:     constructor(string memory name, string memory symbol) ERC20(name, symbol) ERC20Permit(name) {}
File: dummy/DummyWETH.sol

15:     constructor() ERC20("Dummy Wrapped ETH", "DWETH") {}
File: libraries/VaultLib.sol

150:         } catch {}

Tool used

Manual Review

Recommendation