issues
search
Cyfrin
/
2023-07-foundry-defi-stablecoin
38
stars
33
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
Setting the constructor to `payable`
#1054
codehawks-bot
closed
1 year ago
1
Multiple address `mappings` can be combined into a single `mapping`
#1053
codehawks-bot
opened
1 year ago
0
ERC20 Token Standard
#1052
codehawks-bot
closed
1 year ago
1
Unnecessary "<0" validation on uint256 value
#1051
codehawks-bot
opened
1 year ago
0
Incorrect price calculation in `getUsdValue()` due to decimal assumption
#1050
codehawks-bot
opened
1 year ago
0
Missing Validation on chainlink data
#1049
codehawks-bot
opened
1 year ago
0
staleCheckLatestRoundData will use the wrong price if Chainlink returns price outside min/max range
#1048
codehawks-bot
opened
1 year ago
0
Imcompatiable with non-standard tokens like USDT
#1047
codehawks-bot
closed
1 year ago
1
staleCheckLatestRoundData() doesn't check Arbitrum l2 chainlink feed is active
#1046
codehawks-bot
opened
1 year ago
0
No checks to see whether Arbitrum sequencer is down.
#1045
codehawks-bot
opened
1 year ago
0
Use `Ownable2Step's` transfer function rather than `Ownable's` for transfers of ownership
#1044
codehawks-bot
closed
1 year ago
1
There is no check for a stale price feed
#1043
codehawks-bot
closed
1 year ago
1
In `getTokenAmountFromUsd`, `price` is treated as in 8 decimals that will force to return incorrect amount usd amount for given token leading to loss of funds
#1042
codehawks-bot
opened
1 year ago
1
Pragma isn't specified correctly which can lead to nonfunction/damaged contract when deployed on Arbitrum
#1041
codehawks-bot
opened
1 year ago
0
Consistently check account balance before and after transfers for Fee-On-Transfer discrepancies
#1040
codehawks-bot
opened
1 year ago
0
No check if `price>0` in `latestRoundData()`
#1039
codehawks-bot
opened
1 year ago
0
health-factor is way more sensitive causing liquidation of user on tiny price change
#1038
codehawks-bot
closed
1 year ago
1
Oracle data feed is insufficiently validated
#1037
codehawks-bot
opened
1 year ago
0
`latestRoundData()` has no check for round completeness
#1036
codehawks-bot
opened
1 year ago
0
Did not Approve to zero first before changing the allowance
#1035
codehawks-bot
opened
1 year ago
0
`DSCEngine` should deploy its own `DecentralizedStableCoin`
#1034
codehawks-bot
opened
1 year ago
1
Tokens with missing return values in transfers won't work as expected as collateral
#1033
codehawks-bot
closed
1 year ago
1
Chainlink L2 Sequencer Uptime Feed and Grace Period
#1032
codehawks-bot
opened
1 year ago
0
Hardcoded Timeout Value in OracleLib.sol Leads to Inaccurate Timeout Settings for Different Price Feeds
#1031
codehawks-bot
opened
1 year ago
4
Missing Division By 0 Check
#1030
codehawks-bot
opened
1 year ago
0
Protocol is Incompatible with rebasing tokens and tokens with airdrops
#1029
codehawks-bot
opened
1 year ago
0
Theft of collateral tokens with fewer than 18 decimals
#1028
codehawks-bot
opened
1 year ago
1
No Maximum Supply Limit
#1027
codehawks-bot
closed
1 year ago
1
Missing override Keyword
#1026
codehawks-bot
closed
1 year ago
1
Constructor can be payable
#1025
codehawks-bot
closed
1 year ago
1
Improper validation of Chainlink data
#1024
codehawks-bot
opened
1 year ago
0
Protocol is incompatible with fee-on-transfer tokens
#1023
codehawks-bot
opened
1 year ago
0
Numbers are truncated during calculation, reducing precision
#1022
codehawks-bot
opened
1 year ago
0
The failure of Optimism/Arbitrum sequencer can lead users to be liquidated
#1021
codehawks-bot
opened
1 year ago
0
function with onlyOwner modifier can be payable
#1020
codehawks-bot
closed
1 year ago
1
Use assembly to check zero address
#1019
codehawks-bot
opened
1 year ago
0
x + y is more efficient that +=
#1018
codehawks-bot
opened
1 year ago
0
Chainlink's `latestRoundData` might return stale or incorrect results
#1017
codehawks-bot
opened
1 year ago
0
increment can be unchecked
#1016
codehawks-bot
opened
1 year ago
0
Constants should be modifiable to account for changing market conditions and different collateral tokens
#1015
codehawks-bot
closed
1 year ago
1
Max debt amount available for liquidation isn’t calculated resulting in borrowers incurring unnecessary losses
#1014
codehawks-bot
closed
1 year ago
1
For Loop Local Variable Assignment and Unchecked
#1013
codehawks-bot
opened
1 year ago
0
Liquidations will fail for tokens with decimals() != 18
#1012
codehawks-bot
opened
1 year ago
0
There is no way to liquidate a user when they use protocol with tokens that have less than 18 decimals.
#1011
codehawks-bot
opened
1 year ago
0
Need to check tokenAddresses and priceFeedAddresses values in constructor
#1010
codehawks-bot
opened
1 year ago
0
User's healthFactor calculation is wrong if user deposit with a token that don't have 18 decimals
#1009
codehawks-bot
opened
1 year ago
0
Zero-Address Checks absent in Constructor
#1008
codehawks-bot
opened
1 year ago
0
Update to Latest Solidity Version 0.8.21
#1007
codehawks-bot
closed
1 year ago
1
Unnecessary Health Factor Check in `burnDsc` Function
#1006
codehawks-bot
opened
1 year ago
0
Known vulnerabilities of ERC-20 token, contract `DecentralizedStableCoin.sol`
#1005
codehawks-bot
closed
1 year ago
1
Previous
Next