[APPC-662] - ECDSASignature toHex() now returns a normalized encoded …

…string.
AppStoreFoundation · Oct 8, 2018 · 0c2271f · 0c2271f
1 parent 984089a
commit 0c2271f
Showing 1 changed file with 13 additions and 1 deletion.
diff --git a/ethereumj-android/src/main/java/ethereumj/crypto/ECKey.java b/ethereumj-android/src/main/java/ethereumj/crypto/ECKey.java
@@ -902,6 +902,9 @@ public boolean isPubKeyCanonical() {
    * components can be useful for doing further EC maths on them.
    */
   public static class ECDSASignature {
+
+    private static final int SIGNATURE_HEX_LENGTH = 130;
+
     /**
      * The two components of the signature.
      */
@@ -1029,7 +1032,16 @@ public byte[] toByteArray() {
     }
 
     public String toHex() {
-      return Hex.toHexString(toByteArray());
+      return normalize(Hex.toHexString(toByteArray()));
+    }
+
+    private String normalize(String s) {
+      StringBuilder builder = new StringBuilder(s);
+
+      for (int i = 0; i < (SIGNATURE_HEX_LENGTH - s.length()); i++) {
+        builder.insert(0, '0');
+      }
+      return builder.toString();
     }
 
     @Override public int hashCode() {