ruby-changes:23109
From: emboss <ko1@a...>
Date: Thu, 29 Mar 2012 07:53:28 +0900 (JST)
Subject: [ruby-changes:23109] emboss:r35159 (trunk): * ext/openssl/ossl_asn1.c: raise TypeError when trying to encode nil
emboss 2012-03-29 07:53:18 +0900 (Thu, 29 Mar 2012) New Revision: 35159 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=35159 Log: * ext/openssl/ossl_asn1.c: raise TypeError when trying to encode nil values for Primitive instances. * test/openssl/test_asn1.rb: Assert consistent behavior when encoding nil values: Primitives raise TypeError, Constructives raise NoMethodError. Fixes [ruby-core:43009][Bug #6102] -This line, and those below, will be ignored-- M test/openssl/test_asn1.rb M ext/openssl/ossl_asn1.c M ChangeLog Modified files: trunk/ChangeLog trunk/ext/openssl/ossl_asn1.c trunk/test/openssl/test_asn1.rb Index: ChangeLog =================================================================== --- ChangeLog (revision 35158) +++ ChangeLog (revision 35159) @@ -1,3 +1,12 @@ +Thu Mar 29 07:45:36 2012 Martin Bosslet <Martin.Bosslet@g...> + + * ext/openssl/ossl_asn1.c: raise TypeError when trying to encode nil + values for Primitive instances. + * test/openssl/test_asn1.rb: Assert consistent behavior when + encoding nil values: Primitives raise TypeError, Constructives + raise NoMethodError. + Fixes [ruby-core:43009][Bug #6102] + Wed Mar 28 16:39:59 2012 Nobuyoshi Nakada <nobu@r...> * process.c (obj2uid, obj2gid): allow strings as input user/group id. Index: ext/openssl/ossl_asn1.c =================================================================== --- ext/openssl/ossl_asn1.c (revision 35158) +++ ext/openssl/ossl_asn1.c (revision 35159) @@ -149,11 +149,16 @@ ASN1_INTEGER * num_to_asn1integer(VALUE obj, ASN1_INTEGER *ai) { - BIGNUM *bn = GetBNPtr(obj); + BIGNUM *bn; + + if (NIL_P(obj)) + ossl_raise(rb_eTypeError, "Can't convert nil into Integer"); - if (!(ai = BN_to_ASN1_INTEGER(bn, ai))) { + bn = GetBNPtr(obj); + + if (!(ai = BN_to_ASN1_INTEGER(bn, ai))) ossl_raise(eOSSLError, NULL); - } + return ai; } #endif @@ -219,6 +224,9 @@ static ASN1_BOOLEAN obj_to_asn1bool(VALUE obj) { + if (NIL_P(obj)) + ossl_raise(rb_eTypeError, "Can't convert nil into Boolean"); + #if OPENSSL_VERSION_NUMBER < 0x00907000L return RTEST(obj) ? 0xff : 0x100; #else Index: test/openssl/test_asn1.rb =================================================================== --- test/openssl/test_asn1.rb (revision 35158) +++ test/openssl/test_asn1.rb (revision 35159) @@ -198,6 +198,18 @@ encode_decode_test(OpenSSL::ASN1::Integer, [72, -127, -128, 128, -1, 0, 1, -(2**12345), 2**12345]) end + def test_encode_nil + m = OpenSSL::ASN1 + [ + m::Boolean, m::Integer, m::BitString, m::OctetString, + m::ObjectId, m::Enumerated, m::UTF8String, m::UTCTime, + m::GeneralizedTime, m::Sequence, m::Set + ].each do |klass| + #Primitives raise TypeError, Constructives NoMethodError + assert_raise(TypeError, NoMethodError) { klass.send(:new, nil).to_der } + end + end + def encode_decode_test(type, values) values.each do |v| assert_equal(v, OpenSSL::ASN1.decode(type.new(v).to_der).value) -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/