diff options
author | gotoyuzo <gotoyuzo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2004-12-21 13:09:26 +0000 |
---|---|---|
committer | gotoyuzo <gotoyuzo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2004-12-21 13:09:26 +0000 |
commit | 8180c1bd50d84563900bfe81dfa45d0ea4c1fa0f (patch) | |
tree | f49517adf454eb190e4f561679b72d618da9476a | |
parent | 21d42dc45b9f2830294fc0c2d6b1facf0a9fd451 (diff) | |
download | ruby-8180c1bd50d84563900bfe81dfa45d0ea4c1fa0f.tar.gz |
* ext/openssl/ossl_asn1.c (ossl_asn1_decode_all): use rb_str_new4
to avoid SEGV.
* ext/openssl/ossl_asn1.c (ossl_asn1_traverse, ossl_asn1_decode,
ossl_asn1_decode_all): temporary value should be marked volatile.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@7626 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 8 | ||||
-rw-r--r-- | ext/openssl/ossl_asn1.c | 21 |
2 files changed, 20 insertions, 9 deletions
@@ -1,3 +1,11 @@ +Tue Dec 21 22:07:41 2004 GOTOU Yuuzou <gotoyuzo@notwork.org> + + * ext/openssl/ossl_asn1.c (ossl_asn1_decode_all): use rb_str_new4 + to avoid SEGV. + + * ext/openssl/ossl_asn1.c (ossl_asn1_traverse, ossl_asn1_decode, + ossl_asn1_decode_all): temporary value should be marked volatile. + Tue Dec 21 12:42:34 2004 GOTOU Yuuzou <gotoyuzo@notwork.org> * ext/openssl/ossl_asn1.c (ossl_asn1_traverse, ossl_asn1_decode): diff --git a/ext/openssl/ossl_asn1.c b/ext/openssl/ossl_asn1.c index f353ce9b7a..1ce5fa50c5 100644 --- a/ext/openssl/ossl_asn1.c +++ b/ext/openssl/ossl_asn1.c @@ -813,11 +813,12 @@ ossl_asn1_traverse(VALUE self, VALUE obj) { unsigned char *p; long offset = 0; + volatile VALUE tmp; obj = ossl_to_der_if_possible(obj); - obj = rb_str_new4(StringValue(obj)); - p = RSTRING(obj)->ptr; - ossl_asn1_decode0(&p, RSTRING(obj)->len, &offset, 0, 0, 1); + tmp = rb_str_new4(StringValue(obj)); + p = RSTRING(tmp)->ptr; + ossl_asn1_decode0(&p, RSTRING(tmp)->len, &offset, 0, 0, 1); return Qnil; } @@ -828,11 +829,12 @@ ossl_asn1_decode(VALUE self, VALUE obj) VALUE ret, ary; unsigned char *p; long offset = 0; + volatile VALUE tmp; obj = ossl_to_der_if_possible(obj); - obj = rb_str_new4(StringValue(obj)); - p = RSTRING(obj)->ptr; - ary = ossl_asn1_decode0(&p, RSTRING(obj)->len, &offset, 0, 1, 0); + tmp = rb_str_new4(StringValue(obj)); + p = RSTRING(tmp)->ptr; + ary = ossl_asn1_decode0(&p, RSTRING(tmp)->len, &offset, 0, 1, 0); ret = rb_ary_entry(ary, 0); return ret; @@ -844,11 +846,12 @@ ossl_asn1_decode_all(VALUE self, VALUE obj) VALUE ret; unsigned char *p; long offset = 0; + volatile VALUE tmp; obj = ossl_to_der_if_possible(obj); - StringValue(obj); - p = RSTRING(obj)->ptr; - ret = ossl_asn1_decode0(&p, RSTRING(obj)->len, &offset, 0, 0, 0); + tmp = rb_str_new4(StringValue(obj)); + p = RSTRING(tmp)->ptr; + ret = ossl_asn1_decode0(&p, RSTRING(tmp)->len, &offset, 0, 0, 0); return ret; } |