issues
search
code-423n4
/
2021-10-tempus-findings
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
TempusController's _exitTempusAmmAndRedeem function performs swap without minimum required price
#49
code423n4
closed
2 years ago
1
depositYieldBearing didn't check address != 0
#48
code423n4
opened
2 years ago
2
State variables could be set private
#47
code423n4
closed
2 years ago
2
`internal` functions can be `private`
#46
code423n4
opened
2 years ago
2
`public` functions can be `external`
#45
code423n4
opened
2 years ago
2
Unnecessary array boundaries check when loading an array element twice
#44
code423n4
closed
2 years ago
2
Local variables initialization inside a for loop
#43
code423n4
closed
2 years ago
2
Unnecessary `MLOAD`s in for-each loops
#42
code423n4
closed
2 years ago
1
Unnecessary checked arithmetic in for loops
#41
code423n4
closed
2 years ago
2
Prefix increaments are cheaper than postfix increaments
#40
code423n4
opened
2 years ago
1
Open TODOs
#39
code423n4
opened
2 years ago
1
Use of uint8 for counter in for loop increases gas costs
#38
code423n4
opened
2 years ago
1
getAMMOrderedAmounts and _exitTempusAmmAndRedeem functions use explicit token comparison for ordering instead of relying on Balancer's PoolTokens
#37
code423n4
opened
2 years ago
1
for loop with _TOTAL_TOKENS
#36
code423n4
opened
2 years ago
1
unchecked operations
#35
code423n4
closed
2 years ago
2
Lower than uint256 storage variables
#34
code423n4
closed
2 years ago
1
Unused imports
#33
code423n4
opened
2 years ago
1
Lack of validation for Maturity Date
#32
code423n4
opened
2 years ago
2
Cache array length in for loops can save gas
#31
code423n4
opened
2 years ago
1
Adding unchecked directive can save gas
#30
code423n4
opened
2 years ago
1
Typos
#29
code423n4
opened
2 years ago
1
Long Revert Strings
#28
code423n4
opened
2 years ago
2
Gas: `ERC20OwnerMintableToken.burn` should use caller
#27
code423n4
opened
2 years ago
3
Gas: Don't store cToken twice
#26
code423n4
opened
2 years ago
1
`_setAmplificationData` should clear upper bits of values
#25
code423n4
opened
2 years ago
2
`transferFees` may not be the contract itself
#24
code423n4
opened
2 years ago
2
No `swap` slippage checks
#23
code423n4
opened
2 years ago
2
`exitTempusAMMAndRedeem` redeems to the wrong account
#22
code423n4
closed
2 years ago
2
`exitTempusAMM` can be made to fail
#21
code423n4
opened
2 years ago
2
`depositAndFix` can be made to fail
#20
code423n4
opened
2 years ago
2
Missing parameter validation
#19
code423n4
closed
2 years ago
2
TempusAMM freezing all actions except proportional exit on maturity seems unnecessary
#18
code423n4
opened
3 years ago
2
Inheritance from BaseGeneralPool is unused
#17
code423n4
opened
3 years ago
3
Repeated token transfers on deposits are unnecessary
#16
code423n4
opened
3 years ago
1
Use of `matured` storage variable is unnecessary
#15
code423n4
opened
3 years ago
2
Aave/Compound pools result in liquidity mining returns being lost
#14
code423n4
opened
3 years ago
0
cToken funds are locked if Compound's exchange rate is 0
#13
code423n4
opened
3 years ago
3
Param `initInterestRate` in `TempusPool::constructor` should not be 0
#12
code423n4
opened
3 years ago
2
Make `protocolName` variables in protocol pools constant
#11
code423n4
opened
3 years ago
3
Steal tokens from TempusController
#10
code423n4
opened
3 years ago
4
PermanentlyOwnable does not prevent transferring ownership to a dead address.
#9
code423n4
opened
3 years ago
2
Manipulating updateInterestRate() in Tempus Pools to mint more Principal and Yield Tokens Than They Should
#8
code423n4
opened
3 years ago
3
Scaling factors for token 0/1 might swap in TempusAMM constructor.
#7
code423n4
opened
3 years ago
2
No zero address check for controller in TempusPool
#6
code423n4
opened
3 years ago
2
Improper Access Control
#5
code423n4
opened
3 years ago
2
Named Return Issues
#4
code423n4
opened
3 years ago
2
testing submission form
#3
code423n4
closed
3 years ago
1
Summarize your findings for the bug or vulnerability.
#2
code423n4
closed
3 years ago
0
Summarize your findings for the bug or vulnerability.
#1
code423n4
closed
3 years ago
1