Backed Protocol contest - 0v3rf10w's results

QA1:

consistently use safeTransfer NFTLoanFacilitator.sol L#88

        IERC721(collateralContractAddress).transferFrom(msg.sender, address(this), collateralTokenId);

change this kind of similar to NFTLoanFacilitator.sol L#262,242

        IERC721(loan.collateralContractAddress).safeTransferFrom(
            address(this),
            sendCollateralTo,
            loan.collateralTokenId
        );

QA2:

NFTLoanFacilitator.sol L#165

            // will underflow if amount < previousAmount
            uint256 amountIncrease = amount - previousLoanAmount;

(this don't make sense if underflowed or just add if it's underflowing then it's todo or add a check or can use unchecked{} )

NFTLoanFacilitator.sol L#198 (this too don't make sense if above gets underflowed, also add != being an unsigned integer)

Gas1:

shorten revert strings or custom errors to save gas rather than very long strings

Gas2:

uint256 preferred instead of uint types to save gas

Gas3:

NFTLoanFacilitator.sol L#321 use !=0 insted of > for unsigned integers