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

Invalid validation allows users to unlock early

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

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

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

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

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

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

Lack of Balance Validation