Minting zero tokens when underlyingToken is not Ether in cashIn()

Since the cost of launching a new pool is minimal, an attacker can maliciously consume VirtualTokens.

LP for v3 pool of underlying tokens with decimals != 18 would have incorrect NFT metadata

Missing `lower<upper` check in `mint_position`

update_emergency_council_7_D_0_C_1_C_58() updates nft manager instead of emergency council

swap_2 implementation will randomly revert due to improper check, root cause for failed test ethers_suite_uniswap_orchestrated_uniswap_two

Users can't remove liquidity while a pool is disabled

Wrong liquidity formula used

No related function to set fee_protocol

Single plot can be occupied by multiple renters

[H-01] Miscalculation in `_farmPlots` function could lead to a user unable to unstake all NFTs

Invalid validation in _farmPlots function allowing a malicious user repeated farming without locked funds

Users can farm on zero-tax land if the landlord locked tokens before the LandManager deployment

Multicall does not work as intended

Invalid validation allows users to unlock early

Missing disapproval check in `LockManager.sol::approveUSDPrice` allows simultaneous approval and disapproval of a price proposal

Lack of Balance Validation