Throw IllegalStateException instead of NPE if field not set

Author: zzz

src/net/i2p/crypto/eddsa/math/bigint/BigIntegerLittleEndianEncoding.java

@@ -40,8 +40,11 @@ public byte[] encode(FieldElement x) {
      *  Constant time.
      *
      *  @return array of length b/8
+     *  @throws IllegalStateException if field not set
      */
     public byte[] encode(BigInteger x) {
+        if (f == null)
+            throw new IllegalStateException("field not set");
         byte[] in = x.toByteArray();
         byte[] out = new byte[f.getb()/8];
         for (int i = 0; i < in.length; i++) {
@@ -53,7 +56,18 @@ public byte[] encode(BigInteger x) {
         return out;
     }
 
+    /**
+     *  Decode a FieldElement from its (b-1)-bit encoding.
+     *  The highest bit is masked out.
+     *
+     *  @param in the (b-1)-bit encoding of a FieldElement.
+     *  @return the FieldElement represented by 'val'.
+     *  @throws IllegalStateException if field not set
+     *  @throws IllegalArgumentException if encoding is invalid
+     */
     public FieldElement decode(byte[] in) {
+        if (f == null)
+            throw new IllegalStateException("field not set");
         if (in.length != f.getb()/8)
             throw new IllegalArgumentException("Not a valid encoding");
         return new BigIntegerFieldElement(f, toBigInteger(in).and(mask));