Review

When a DataOutputStream instance wraps an underlying ByteArrayOutputStream instance,
it is recommended to flush or close the DataOutputStream before invoking the underlying instances's toByteArray().
Although in these cases this is not strictly necessary because the
DataOutputStream's close and flush method has no effects. However, it is a good practice to call
flush/close explicitly as mentioned for example here.
This pull request adds a call to close.

• missing/call

net/sf/ntru/sign/SignaturePublicKey.java

getEncoded()

class SignaturePublicKey {
    /**
     * Converts the key to a byte array
     * @return the encoded key
     * @see #SignaturePublicKey(byte[])
     */
    public byte[] getEncoded() {
        ByteArrayOutputStream os = new ByteArrayOutputStream();
        DataOutputStream dataStream = new DataOutputStream(os);
        try {
            dataStream.writeShort(h.coeffs.length);
            dataStream.writeShort(q);
            dataStream.write(h.toBinary(q));
        } catch (IOException e) {
            throw new NtruException(e);
        }
        return os.toByteArray();
    }

}