OrderStatus never return 0 #521
Labels
bug
Something isn't working
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
sponsor disputed
Sponsor cannot duplicate the issue, or otherwise disagrees this is an issue
Lines of code
https://github.com/code-423n4/2022-07-golom/blob/e5efa8f9d6dda92a90b8b2c4902320acf0c26816/contracts/core/GolomTrader.sol#L177-L180
Vulnerability details
Impact
When the signature is invalid, the
require
statement is false, the execution stops, and the transaction gets rolled back. The function never returns 0.Proof of Concept
Recommended Mitigation Steps
remove the
require
statement.The text was updated successfully, but these errors were encountered: