State variable changed in `Escrow::_depositIntoEscrow` even if collectionType neither ERC721 nor ERC1155.
Summary
Escrow::_depositIntoEscrowthis function is intended to deposit in escrow and it should allow only ERC721 and ERC1155 collectionType, but with any collectionType it changes this mapping.
Vulnerability Details
_escrow this mapping also changes when any collectionType intended to deposit into escrow, but it should only allows ERC721 and ERC1155 collectionType.
Impact
Unnecessary state variable changed
Tools Used
Manual review
Recommendations
Check whether the collectionType should be ERC721 or ERC1155 in Escrow::_depositIntoEscrow function
Lead Judging Commences
invalid-tokens-with-other-standards-will-be-treated-like-ERC1155
No impact on the bridge or the users. If any NFT protocol use another standard with that bridge, they have to know how the bridge works and also that it doesn’t work with other standards at the moment.
Prize pool breakdown
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.