From 61b624d149204c2b1c17a2852d088b2eb2aa010b Mon Sep 17 00:00:00 2001 From: eregon Date: Wed, 1 Aug 2012 13:30:51 +0000 Subject: ext/digest/digest.c (hexencode_str_new): return an ASCII string * test/digest: tests for all kind of digests encodings [ruby-core:46792][Bug #6799] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@36588 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 7 +++++++ ext/digest/digest.c | 2 +- test/digest/test_digest.rb | 12 +++++++++--- test/digest/test_digest_extend.rb | 1 + 4 files changed, 18 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index 44a9d83351..ce7ca9ecbf 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +Wed Aug 1 22:29:12 2012 Benoit Daloze + + ext/digest/digest.c (hexencode_str_new): return an ASCII string + + * test/digest: tests for all kind of digests encodings + [ruby-core:46792][Bug #6799] + Wed Aug 1 05:50:53 2012 Hiroshi Shirosaki * test/ruby/test_rubyoptions.rb (TestRubyOptions#test_encoding): diff --git a/ext/digest/digest.c b/ext/digest/digest.c index 4d185a8e5d..c603441409 100644 --- a/ext/digest/digest.c +++ b/ext/digest/digest.c @@ -99,7 +99,7 @@ hexencode_str_new(VALUE str_digest) rb_raise(rb_eRuntimeError, "digest string too long"); } - str = rb_str_new(0, digest_len * 2); + str = rb_usascii_str_new(0, digest_len * 2); for (i = 0, p = RSTRING_PTR(str); i < digest_len; i++) { unsigned char byte = digest[i]; diff --git a/test/digest/test_digest.rb b/test/digest/test_digest.rb index 661129083a..4b1d55e256 100755 --- a/test/digest/test_digest.rb +++ b/test/digest/test_digest.rb @@ -25,21 +25,27 @@ module TestDigest def test_s_hexdigest self.class::DATA.each do |str, hexdigest| - assert_equal(hexdigest, self.class::ALGO.hexdigest(str)) + actual = self.class::ALGO.hexdigest(str) + assert_equal(hexdigest, actual) + assert_equal(Encoding::US_ASCII, actual.encoding) end end def test_s_base64digest self.class::DATA.each do |str, hexdigest| digest = [hexdigest].pack("H*") - assert_equal([digest].pack("m0"), self.class::ALGO.base64digest(str)) + actual = self.class::ALGO.base64digest(str) + assert_equal([digest].pack("m0"), actual) + assert_equal(Encoding::US_ASCII, actual.encoding) end end def test_s_digest self.class::DATA.each do |str, hexdigest| digest = [hexdigest].pack("H*") - assert_equal(digest, self.class::ALGO.digest(str)) + actual = self.class::ALGO.digest(str) + assert_equal(digest, actual) + assert_equal(Encoding::BINARY, actual.encoding) end end diff --git a/test/digest/test_digest_extend.rb b/test/digest/test_digest_extend.rb index 577fd7a455..2992e28877 100644 --- a/test/digest/test_digest_extend.rb +++ b/test/digest/test_digest_extend.rb @@ -49,6 +49,7 @@ class TestDigestExtend < Test::Unit::TestCase (0..0xff).to_a.map { |c| sprintf("%02x", c ) }.join(''), Digest.hexencode((0..0xff).to_a.map { |c| c.chr }.join('')) ) + assert_equal(Encoding::US_ASCII, Digest.hexencode("\1\2").encoding) end def test_class_reset -- cgit v1.2.3