diff options
| author | Alex Morcos <[email protected]> | 2017-01-10 16:53:10 -0500 |
|---|---|---|
| committer | Alex Morcos <[email protected]> | 2017-01-10 16:53:10 -0500 |
| commit | 116419e58dddef8fe3ff9806a1d8ceebe64ae3e6 (patch) | |
| tree | 9e20acef8fdad52ff5671eb80574756196840b35 /src/init.cpp | |
| parent | Merge #8811: rpc: Add support for JSON-RPC named arguments (diff) | |
| download | discoin-116419e58dddef8fe3ff9806a1d8ceebe64ae3e6.tar.xz discoin-116419e58dddef8fe3ff9806a1d8ceebe64ae3e6.zip | |
Don't overwrite validation state with corruption check
AcceptToMemoryPool has several classes of return false statements.
- return state.Invalid or state.DoS directly itself
- return false and set fMissingInputs (state is valid)
- return false and state is set by failed CheckTransaction
- return false and state is set by failed CheckInputs.
This commit patches the last case where the state variable was reused for additional calls to CheckInputs to identify witness stripping as cause of validation failure. After this commit, it should be the case that if !fMissingInputs, state is always Invalid if AcceptToMemoryPool returns false.
Diffstat (limited to 'src/init.cpp')
0 files changed, 0 insertions, 0 deletions