Skip to content

Issues: code-423n4/2021-06-gro-findings

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

86 Open 40 Closed
86 Open 40 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

Use of deprecated Chainlink function latestAnswer 2 (Med Risk) bug Something isn't working disagree with severity
#126 opened Jul 10, 2021 by code423n4
2
Use of tx.origin for authentication 1 (Low Risk) bug Something isn't working
#122 opened Jul 10, 2021 by code423n4
2
More accurate calculation of return USD of withdrawSingleByLiquidity 1 (Low Risk) bug Something isn't working sponsor confirmed
#121 opened Jul 10, 2021 by code423n4
Add a proper revert message in _withdrawSingle 0 (Non-critical) bug Something isn't working disagree with severity sponsor acknowledged
#120 opened Jul 10, 2021 by code423n4
2
Unlocked pragma used in multiple contracts 0 (Non-critical) bug Something isn't working sponsor disputed
#117 opened Jul 10, 2021 by code423n4
2
function withdrawToAdapter should be inluded in the interface and return withdrawal amount bug Something isn't working G (Gas Optimization) sponsor disputed
#116 opened Jul 7, 2021 by code423n4
2
BaseVaultAdaptor assumes sharePrice is always in underlying decimals 2 (Med Risk) bug Something isn't working sponsor confirmed
#114 opened Jul 7, 2021 by code423n4
strategiesLength should not be allowed to exceed MAX_STRATS 0 (Non-critical) bug Something isn't working disagree with severity sponsor disputed
#110 opened Jul 7, 2021 by code423n4
2
Rational actors will just set themselves as referral 1 (Low Risk) bug Something isn't working disagree with severity sponsor disputed
#108 opened Jul 7, 2021 by code423n4
2
Early user can break minting 2 (Med Risk) bug Something isn't working disagree with severity sponsor acknowledged
#107 opened Jul 7, 2021 by code423n4
2
Usage of deprecated ChainLink API in Buoy3Pool 2 (Med Risk) bug Something isn't working sponsor confirmed
#106 opened Jul 7, 2021 by code423n4
Buoy3Pool._updateRatios unsafe math 1 (Low Risk) bug Something isn't working duplicate This issue or pull request already exists
#105 opened Jul 7, 2021 by code423n4
2
Buoy3Pool.safetyCheck is not precise and has some assumptions 3 (High Risk) bug Something isn't working disagree with severity sponsor confirmed
#104 opened Jul 7, 2021 by code423n4
3
Allocaiton.calcProtocolExposureDelta gas optimization bug Something isn't working G (Gas Optimization) sponsor confirmed
#102 opened Jul 7, 2021 by code423n4
Exposure.sortVaultsByDelta does not work for N_COINS != 3 0 (Non-critical) bug Something isn't working disagree with severity sponsor disputed
#101 opened Jul 7, 2021 by code423n4
2
Insurance.getVaultDeltaForDeposit returns wrong investDelta 1 (Low Risk) bug Something isn't working duplicate This issue or pull request already exists
#98 opened Jul 7, 2021 by code423n4
3
Wrong min amount check in withdrawByStablecoin 0 (Non-critical) bug Something isn't working disagree with severity sponsor confirmed
#97 opened Jul 7, 2021 by code423n4
2
Hardcoded 99 as deadcoin 0 (Non-critical) bug Something isn't working disagree with severity sponsor acknowledged
#96 opened Jul 7, 2021 by code423n4
2
Loss of precision 0 (Non-critical) bug Something isn't working sponsor disputed
#95 opened Jul 7, 2021 by code423n4
2
RebasingGToken emits same events on transfer bug Something isn't working G (Gas Optimization) sponsor acknowledged
#91 opened Jul 7, 2021 by code423n4
Missing parameter validation 1 (Low Risk) bug Something isn't working sponsor acknowledged
#90 opened Jul 7, 2021 by code423n4
1
event LogTransfer is only emitted in function transfer 0 (Non-critical) bug Something isn't working sponsor acknowledged
#88 opened Jul 7, 2021 by code423n4
burnAll should check that factor > 0 and amount > 0 0 (Non-critical) bug Something isn't working sponsor disputed
#87 opened Jul 7, 2021 by code423n4
2
totalAssets > withdrawUsd should be inclusive 1 (Low Risk) bug Something isn't working sponsor acknowledged
#86 opened Jul 7, 2021 by code423n4
2
Two SafeApprove calls when it could be just one bug Something isn't working G (Gas Optimization) sponsor acknowledged
#84 opened Jul 7, 2021 by code423n4
1
ProTip! Adding no:label will show everything without a label.