LBRYCommunity/bitcoinjs-lib
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

multisig.input.encodestack - replace OP_0 (permitted by partialSignature) with EMPTY_BUFFER

Browse source
This commit is contained in:
Thomas Kerin 2017-09-06 14:26:23 +02:00 committed by Daniel Cousens
parent 105a82762c
commit da49e389c4
2 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/templates/multisig/input.js
View file

@ -39,7 +39,12 @@ function encodeStack (signatures, scriptPubKey) {
}
}
return [].concat(EMPTY_BUFFER, signatures)
return [].concat(EMPTY_BUFFER, signatures.map(function (sig) {
if (sig === OPS.OP_0) {
return EMPTY_BUFFER
}
return sig
}))
}
function encode (signatures, scriptPubKey) {

6
src/transaction_builder.js
View file

@ -389,14 +389,14 @@ function buildStack (type, signatures, pubKeys, allowIncomplete) {
} else if (type === scriptTypes.MULTISIG) {
if (signatures.length > 0) {
signatures = signatures.map(function (signature) {
return signature || Buffer.from('', 'hex')
return signature || ops.OP_0
})
if (!allowIncomplete) {
// remove blank signatures
signatures = signatures.filter(function (x) { return x.length !== 0 })
signatures = signatures.filter(function (x) { return x !== ops.OP_0 })
}
return [].concat(Buffer.from('', 'hex'), signatures)
return bscript.multisig.input.encodeStack(signatures)
}
} else {
throw new Error('Not yet supported')