Merge pull request #3976 from Songhee120/pk-exception

Raise exception on libplanet for zero byte prive key

Author: OnedgeLee

src/Libplanet.Crypto/PrivateKey.cs

@@ -405,6 +405,11 @@ private static ECPrivateKeyParameters GenerateKeyParam()
 
         private static ECPrivateKeyParameters GenerateKeyFromBytes(byte[] privateKey)
         {
+            if (privateKey.All(b => b.Equals(0x00)))
+            {
+                throw new ArgumentException("Every bytes in Private key is zero value.");
+            }
+
             var param = new ECPrivateKeyParameters(
                 "ECDSA",
                 new BigInteger(1, privateKey),

test/Libplanet.Tests/Crypto/PrivateKeyTest.cs

@@ -360,5 +360,18 @@ public void PrivateKeyGenerateLongerThan31Bytes()
 
             Assert.Empty(faults);
         }
+
+        [Theory]
+        [InlineData(new byte[]
+        {
+            0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+            0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+            0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+            0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+        })]
+        public void ThrowsWhenZeroBytesPrivateKey(byte[] bytes)
+        {
+            Assert.Throws<ArgumentException>(() => new PrivateKey(bytes));
+        }
     }
-}
+}