Fixed bugs in two of the ZCert constructors with keys as argument

Author: sm2vsd

ZCert.cs

@@ -118,7 +118,7 @@ public ZCert(byte[] publicKey, byte[] secretKey)
             {
                 throw new InvalidOperationException("secret key length must be of length 32");
             }
-            Array.Copy(secretKey, this.secretKey, 32);
+            Array.Copy(publicKey, this.publicKey, 32);
             Array.Copy(secretKey, this.secretKey, 32);
 
             publicTxt = Encoding.UTF8.GetString(Z85.Encode(publicKey)).ToCharArray();
@@ -145,8 +145,9 @@ public ZCert(string publicTxt, string secretTxt)
             PublicTxt = publicTxt;
             SecretTxt = secretTxt;
 
-            publicKey = Z85.EncodeBytes(PublicTxt);
-            secretKey = Z85.EncodeBytes(SecretTxt);
+            publicKey = Z85.ToZ85DecodedBytes(PublicTxt);
+            secretKey = Z85.ToZ85DecodedBytes(SecretTxt);
+
         }
 
         /// <summary>
@@ -204,7 +205,7 @@ public void Apply(ZSocket socket)
         /// </summary>
         /// <param name="cert">Certificate to deep clone. Public and private keys must not be null.</param>
         /// <returns>A copy of the given certificate.</returns>
-        public ZCert Dup(ZCert cert)
+        public static ZCert Dup(ZCert cert)
         {
             if (cert == null)
                 return null;