Skip to content

Commit

Permalink
Fix bug causing multisignature failure on IsStandard check
Browse files Browse the repository at this point in the history
An off by one bug caused by inheritance from Bitcoin caused
multisignature transactions that were fully signed to be rejected
due to the number of stack items. The off-by-one bug has been fixed.
  • Loading branch information
cjepson authored and jcvernaleo committed Feb 8, 2016
1 parent 5076a00 commit cb112a8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion txscript/standard.go
Original file line number Diff line number Diff line change
Expand Up @@ -455,7 +455,7 @@ func expectedInputs(pops []parsedOpcode, class ScriptClass,
// the original bitcoind bug where OP_CHECKMULTISIG pops an
// additional item from the stack, add an extra expected input
// for the extra push that is required to compensate.
return asSmallInt(pops[0].opcode) + 1
return asSmallInt(pops[0].opcode)

case NullDataTy:
fallthrough
Expand Down

0 comments on commit cb112a8

Please sign in to comment.