mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
* backport r29071 from ruby_1_8;
* ext/openssl/ossl_asn1.c (obj_to_asn1bool): fixed ASN1::Boolean encoding issue for OpenSSL 1.0.0 compatibility. ASN1::Boolean.new(false).to_der wrongly generated "\1\1\377" which means 'true'. ASN1_TYPE_set of OpenSSL <= 0.9.8 treats value 0x100 as 'false' but OpenSSL >= 1.0.0 treats it as 'true'. ruby-ossl was using 0x100 for 'false' for backward compatibility. Just use 0x0 for the case OpenSSL >= OpenSSL 0.9.7. * test/openssl/test_asn1.rb: test added. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@29075 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
d078f51f57
commit
298a6e1cd5
3 changed files with 34 additions and 0 deletions
16
ChangeLog
16
ChangeLog
|
@ -1,3 +1,19 @@
|
|||
Mon Aug 23 13:09:27 2010 NAKAMURA, Hiroshi <nahi@ruby-lang.org>
|
||||
|
||||
* backport r29071 from ruby_1_8;
|
||||
|
||||
* ext/openssl/ossl_asn1.c (obj_to_asn1bool): fixed ASN1::Boolean
|
||||
encoding issue for OpenSSL 1.0.0 compatibility.
|
||||
ASN1::Boolean.new(false).to_der wrongly generated "\1\1\377" which
|
||||
means 'true'.
|
||||
|
||||
ASN1_TYPE_set of OpenSSL <= 0.9.8 treats value 0x100 as 'false'
|
||||
but OpenSSL >= 1.0.0 treats it as 'true'. ruby-ossl was using
|
||||
0x100 for 'false' for backward compatibility. Just use 0x0 for
|
||||
the case OpenSSL >= OpenSSL 0.9.7.
|
||||
|
||||
* test/openssl/test_asn1.rb: test added.
|
||||
|
||||
Mon Aug 23 12:28:58 2010 NARUSE, Yui <naruse@ruby-lang.org>
|
||||
|
||||
* re.c (rb_reg_search): fix: 4th argument should be regexp
|
||||
|
|
|
@ -196,7 +196,11 @@ static ID sUNIVERSAL, sAPPLICATION, sCONTEXT_SPECIFIC, sPRIVATE;
|
|||
static ASN1_BOOLEAN
|
||||
obj_to_asn1bool(VALUE obj)
|
||||
{
|
||||
#if OPENSSL_VERSION_NUMBER < 0x00907000L
|
||||
return RTEST(obj) ? 0xff : 0x100;
|
||||
#else
|
||||
return RTEST(obj) ? 0xff : 0x0;
|
||||
#endif
|
||||
}
|
||||
|
||||
static ASN1_INTEGER*
|
||||
|
|
|
@ -194,4 +194,18 @@ class OpenSSL::TestASN1 < Test::Unit::TestCase
|
|||
cululated_sig = key.sign(OpenSSL::Digest::SHA1.new, tbs_cert.to_der)
|
||||
assert_equal(cululated_sig, sig_val.value)
|
||||
end
|
||||
|
||||
def test_encode_boolean
|
||||
encode_decode_test(OpenSSL::ASN1::Boolean, [true, false])
|
||||
end
|
||||
|
||||
def test_encode_integer
|
||||
encode_decode_test(OpenSSL::ASN1::Integer, [72, -127, -128, 128, -1, 0, 1, -(2**12345), 2**12345])
|
||||
end
|
||||
|
||||
def encode_decode_test(type, values)
|
||||
values.each do |v|
|
||||
assert_equal(v, OpenSSL::ASN1.decode(type.new(v).to_der).value)
|
||||
end
|
||||
end
|
||||
end if defined?(OpenSSL)
|
||||
|
|
Loading…
Add table
Reference in a new issue