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

HDWallet: adds PubKey validation

Browse source
This commit is contained in:
Daniel Cousens 2014-05-31 20:28:16 +10:00
parent 708e27aa2d
commit e8e862f632
2 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/hdwallet.js
View file

@ -119,6 +119,9 @@ HDWallet.fromBuffer = function(buffer) {
var decode = ECPointFp.decodeFrom(ecparams.getCurve(), data) var decode = ECPointFp.decodeFrom(ecparams.getCurve(), data)
assert.equal(decode.compressed, true, 'Invalid public key') assert.equal(decode.compressed, true, 'Invalid public key')
// When importing a serialized extended public key, implementations must verify whether the X coordinate in the public key data corresponds to a point on the curve. If not, the extended public key is invalid.
decode.Q.validate()
hd = new HDWallet(decode.Q, chainCode, params.network) hd = new HDWallet(decode.Q, chainCode, params.network)
} }

4
test/fixtures/hdwallet.json vendored
View file

@ -203,6 +203,10 @@
{ {
"exception": "Could not find version 22222222", "exception": "Could not find version 22222222",
"hex": "222222220000000000000000007ffc03d4a1f2fb41ef93374c69e4d19e42e27c9a87ec8b799a205eecd3b43b5f02948d03e260a571e21bcf5bfd8e3b6602800df154906e06b2bc88eee410aee355" "hex": "222222220000000000000000007ffc03d4a1f2fb41ef93374c69e4d19e42e27c9a87ec8b799a205eecd3b43b5f02948d03e260a571e21bcf5bfd8e3b6602800df154906e06b2bc88eee410aee355"
},
{
"exception": "Point is not on the curve",
"hex": "0488b21e000000000000000000873dff81c02f525623fd1fe5167eac3a55a049de3d314bb42ee227ffed37d508020045400697100007000037899988826500030092003000016366806305909050"
} }
] ]
} }