[fix] incorrect ASN.1 for wrapped Integer type

Author: kares

src/main/java/org/jruby/ext/openssl/ASN1.java

@@ -1683,7 +1683,7 @@ private static BigInteger bigIntegerValue(final IRubyObject val) {
             if ( val instanceof RubyInteger ) { // RubyBignum
                 return ((RubyInteger) val).getBigIntegerValue();
             }
-            if ( val instanceof BN ) ((BN) val).getValue();
+            if ( val instanceof BN ) return ((BN) val).getValue();
             return new BigInteger( val.asString().getBytes() );
         }
 

src/test/ruby/test_asn1.rb

@@ -505,6 +505,25 @@ def test_decode_application_specific
 #    assert_equal raw, asn1.to_der
   end
 
+
+  def test_encode_der_integer_wrapped
+    asn1 = OpenSSL::ASN1::Sequence([ OpenSSL::ASN1::Integer(42), OpenSSL::ASN1::Integer(84) ])
+
+    der = "0\x06\x02\x01*\x02\x01T"
+    assert_equal der, asn1.to_der
+
+    asn1 = OpenSSL::ASN1::Sequence([ OpenSSL::ASN1::Integer(OpenSSL::BN.new(42)), OpenSSL::ASN1::Integer(OpenSSL::BN.new(84)) ])
+
+    der = "0\x06\x02\x01*\x02\x01T"
+    assert_equal der, asn1.to_der
+
+    i = OpenSSL::ASN1::Integer(OpenSSL::BN.new('1234567890'))
+    assert_equal 1234567890, i.value.to_i
+
+    i = OpenSSL::ASN1::Integer('12345678901234567890')
+    assert_equal 12345678901234567890, i.value.to_i
+  end
+
   private
 
   def assert_universal(tag, asn1, inf_len=false)