Respond to audit results

[sisyphusSmiling]

Issue To Be Solved

Respond to audit findings

Findings

Preview Give feedback

1. FLB-1: Onboarding Assets May Prevent the Addition of Token Handlers
2. FLB-2: Local Variable Updated Instead of Storage Variable in BridgePermissions.sol
3. FLB-3: Incorrect Conditional Assignment
4. FLB-4: FlowEVMBridgeNFTEscrow.initializeEscrow() Does Not Panic upon Storage Collision
5. FLB-5: FlowEVMBridgeUtils.borrowCOA() Returns a Reference with Unnecessary Entitlement
6. FLB-6: View Functions Can Run Out of Gas for Lists with Too Many Items
7. FLB-7: Missing Input Validations
8. FLB-8: Supporting Non-Standard Tokens May Lead to Unexpected Behavior
9. FLB-9: Unlocked Pragma
10. FLB-10: Ownership Can Be Renounced
11. FLB-11: Risks Related to Type Verifications
12. FLB-12: Partial Usage of ERC165
13. FLB-13: Events Emitted by the Contracts Could Be Improved
14. FLB-14: Mismatch Between Code and Documentation for Handling Failure of Evm Calls
15. FLB-15: Usage of Hardcoded Gas Amounts for EVM Calls
16. FLB-16: Wherever Necessary, Pre and Post-Conditions Could Be Added to the Transactions
17. FLB-17: Unique Hardcoded Name Used for the NFTCollectionDisplay Metadata of NFTs
18. FLB-18: Unclear Purpose of Using Escrows for Evm-Native Assets
19. FLB-19: TokenMinter May Not Accurately Track Token Total Supply
20. FLB-20: Onboarded Tokens Cannot Be Removed or Blocked
21. FLB-21: Importance of Having Sufficiently High Onboarding and Storage Fees
22. FLB-22: Unclear Serialization Behavior Should Be Clarified