issues
search
code-423n4
/
2022-01-livepeer-findings
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
LIP-73 contracts employ unsafe practice of unlocked pragma
#245
code423n4
closed
2 years ago
1
Gas: Use `msg.sender` directly instead of caching it in a variable
#244
code423n4
opened
2 years ago
1
Gas: use `msg.sender` instead of OpenZeppelin's `_msgSender()` when GSN capabilities aren't used
#243
code423n4
opened
2 years ago
1
Anyone can migrate address(0)
#242
code423n4
closed
2 years ago
1
fundDepositAndReserveFor function does not exist in protocol
#241
code423n4
closed
2 years ago
4
L1Migrator have no payable receive function to receive ETH from BridgeMinter
#240
code423n4
closed
2 years ago
1
L2Migrator Allows to Deposit Ether And Ether Will be Locked in the contract
#239
code423n4
closed
2 years ago
2
Fund loss when insufficient call value to cover fee
#238
code423n4
opened
2 years ago
3
Changing function visibility from public to external can save gas
#237
code423n4
closed
2 years ago
2
`migrateETH` will not work
#236
code423n4
closed
2 years ago
1
Anyone can freeze fund in BridgeMinter
#235
code423n4
closed
2 years ago
1
Migrate old balance on setToken
#234
code423n4
opened
2 years ago
1
Gas Optimization: Use cached value
#233
code423n4
closed
2 years ago
1
recoverSigner returns an empty address when the signature is empty
#232
code423n4
closed
2 years ago
1
Gas Optimization: Use unchecked for safe math
#231
code423n4
closed
2 years ago
1
Gas Optimization: Use external keyword instead of public for some functions
#230
code423n4
closed
2 years ago
2
Gas Optimization: `> 0` is less efficient than `!= 0` for uint
#229
code423n4
closed
2 years ago
1
Gas Optimization: Struct layout
#228
code423n4
closed
2 years ago
2
Gas Optimization: Upgrade to Solidity ^0.8.4
#227
code423n4
closed
2 years ago
1
Gas Optimization: Long revert string
#226
code423n4
closed
2 years ago
1
Use abi.encodePacked for gas optimization
#225
code423n4
opened
2 years ago
0
Low-level calls don't check the success return value
#224
code423n4
closed
2 years ago
2
L2Migrator calls wrong function on bondingManager
#223
code423n4
closed
2 years ago
2
Repeated math operations
#222
code423n4
closed
2 years ago
1
Assign to 0 when _amount = l2SupplyFromL1
#221
code423n4
closed
2 years ago
1
Assigned operations to constant variables
#220
code423n4
closed
2 years ago
1
setController 2-step process
#219
code423n4
closed
2 years ago
1
Function should be declared as external to save gas
#218
code423n4
closed
2 years ago
2
Missing zero address in constructor L1LPTDataCache.sol#L20-L27
#217
code423n4
closed
2 years ago
1
For uint `> 0` can be replaced with ` != 0` for gas optimization
#216
code423n4
closed
2 years ago
1
Don't assign default values
#215
code423n4
opened
2 years ago
1
Loops can be implemented more efficiently
#214
code423n4
closed
2 years ago
1
Immutable variables
#213
code423n4
closed
2 years ago
2
Lack of zero checks
#212
code423n4
closed
2 years ago
1
Unsafe ERC20 transfer
#211
code423n4
closed
2 years ago
1
Cache length of array outside for-loop
#210
code423n4
closed
2 years ago
1
Use `immutable` for `AccessControl`'s role declarations
#209
code423n4
closed
2 years ago
1
Don't make unnecessary library calls
#208
code423n4
closed
2 years ago
1
Don't use deprecated library functions
#207
code423n4
opened
2 years ago
3
wrong check
#206
code423n4
closed
2 years ago
1
[WP-H5] `L1Migrator.sol#migrateETH()` dose not send `bridgeMinter`'s ETH to L2 causing ETH get frozen in the contract
#205
code423n4
opened
2 years ago
2
Unnecessary checked arithmetic in for loops
#204
code423n4
opened
2 years ago
1
`L1LPTGateway`, `L2LPTGateway` should start off paused after deployed
#203
code423n4
opened
2 years ago
1
[WP-M4] Unable to use `L2GatewayRouter` to withdraw LPT from L2 to L1, as `L2LPTGateway` does not implement `L2GatewayRouter` expected method
#202
code423n4
opened
2 years ago
1
Revert strings are too long
#201
code423n4
closed
2 years ago
1
The initialize function does not check for non-zero address and emit event
#200
code423n4
opened
2 years ago
3
l1CirculatingSupply may not give correct value after a mass withdrawal from L2
#199
code423n4
closed
2 years ago
5
[WP-H3] `L1Migrator.sol#migrateETH()` Improper implementation of `L1Migrator` causing `migrateETH()` always reverts, can lead to ETH in `BridgeMinter` getting stuck in the contract
#198
code423n4
opened
2 years ago
3
Avoid unnecessary arithmetic operations and storage reads can save gas
#197
code423n4
closed
2 years ago
1
Remove redundant `_setRoleAdmin()` can save gas
#196
code423n4
opened
2 years ago
1
Next