Sanity check on new PrivateKey()

planetarium / libplanet

Blockchain in C#/.NET for on-chain, decentralized gaming

https://docs.libplanet.io/

GNU Lesser General Public License v2.1

506 stars 142 forks source link

Sanity check on new PrivateKey() #358

Closed longfin closed 5 years ago

longfin commented 5 years ago

Currently, there is no validation of privateKey parameter on Libplanet.Crypto.PrivateKey's constructor. so illegal bytes can be passed to new PrivateKey() and occurs error when being used. it would be good if we catch this error in the constructor.

ipdae commented 5 years ago

i'll try this issue.