Unchecked return value of eth transfer #310
Labels
bug
Something isn't working
duplicate
This issue or pull request already exists
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
Lines of code
https://github.com/PartyDAO/party-contracts-c4/blob/main/contracts/party/PartyGovernance.sol#L783-L796
Vulnerability details
Impact
PartyGovernance.targetAddress() makes a low-level call to send eth to a target address, however does not check if transaction is successful.
Proof of Concept
https://github.com/PartyDAO/party-contracts-c4/blob/main/contracts/party/PartyGovernance.sol#L783-L796
Tools Used
Manual review
Recommended Mitigation Steps
Add a require check for success
The text was updated successfully, but these errors were encountered: