Incorrect protocol fee is taken when changing NFTs

Royalty recipients will not get fair share of royalties

A temporary issue shows in the staking functionality which leads to the users receiving less minted tokens.

Missing derivative limit and deposit availability checks will revert the whole `stake()` function

Users can lose their staking rewards.

Protocol fees can be withdrawn multiple times in `Erc20Quest`

Buyer on secondary NFT market can lose fund if they buy a NFT that is already used to claim the reward

Users may not claim Erc1155 rewards when the Quest has ended

User may loose rewards if the receipt is minted after quest end time

The variable `totalUSDborrowed` is wrongly calculated in the function openLoan in Vault_Synths. Users who want to increase their loans will receive less isoUSD.

Malicious user can burn the approved NFTs to the depositer contract and successfully steal owner's AMM tokens.

`CHANGE_COLLATERAL_DELAY` contains a wrong number.

Users won't be able to repay their overduе borrows, duo to a simple mistake made in the function `updateFrozenInfo`.

Two malicious users can drain a big amount of rewards up to 48 weeks, for the little lock time of 10 mins.

Malicious user will prevent anyone from getting shares on their deposited amount of LINK to the vault

frxETHMinter: Non-conforming ERC20 tokens not recoverable

transferfrom can lead to permanently loosing the NFT token.

Usage of deprecated transfer() can result in revert.

Unsafe usage of ERC20 transfer and transferFrom

NFT of NFT collection or NFT drop collection can be locked when calling _mint or mintCountTo function to mint it to a contract that does not support ERC721 protocol