* ext/openssl/ossl_asn1.c (ossl_asn1_traverse, ossl_asn1_decode):

  use rb_str_new4 to avoid SEGV. fix [ruby-dev:25261]

* test/openssl/test_asn1.rb: add tests for OpenSSL::ASN1.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@7623 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 3 insertions, 3 deletions

diff --git a/ext/openssl/ossl_asn1.c b/ext/openssl/ossl_asn1.c
index a961c16bc0..f353ce9b7a 100644
--- a/ext/openssl/ossl_asn1.c
+++ b/ext/openssl/ossl_asn1.c
@@ -815,11 +815,11 @@ ossl_asn1_traverse(VALUE self, VALUE obj)
     long offset = 0;
 
     obj = ossl_to_der_if_possible(obj);
-    StringValue(obj);
+    obj = rb_str_new4(StringValue(obj));
     p = RSTRING(obj)->ptr;
     ossl_asn1_decode0(&p, RSTRING(obj)->len, &offset, 0, 0, 1);
 
-    return obj;
+    return Qnil;
 }
 
 static VALUE
@@ -830,7 +830,7 @@ ossl_asn1_decode(VALUE self, VALUE obj)
     long offset = 0;
 
     obj = ossl_to_der_if_possible(obj);
-    StringValue(obj);
+    obj = rb_str_new4(StringValue(obj));
     p = RSTRING(obj)->ptr;
     ary = ossl_asn1_decode0(&p, RSTRING(obj)->len, &offset, 0, 1, 0);
     ret = rb_ary_entry(ary, 0);