diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | siphash.c | 4 | ||||
-rw-r--r-- | test/ruby/test_string.rb | 2 |
3 files changed, 9 insertions, 2 deletions
@@ -1,3 +1,8 @@ +Sat Nov 30 13:28:13 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * siphash.c (sip_hash24): fix for aligned word access little endian + platforms. [ruby-core:58658] [Bug #9172] + Sat Nov 30 13:21:15 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> * vm_eval.c (rb_yield_block): implement non-nil block argument. @@ -417,7 +417,7 @@ sip_hash24(const uint8_t key[16], const uint8_t *data, size_t len) SIP_2_ROUND(m, v0, v1, v2, v3); } } -#elif BYTE_ORDER == BIG_ENDIAN +#else for (; data != end; data += sizeof(uint64_t)) { m = U8TO64_LE(data); SIP_2_ROUND(m, v0, v1, v2, v3); @@ -453,7 +453,7 @@ sip_hash24(const uint8_t key[16], const uint8_t *data, size_t len) last.lo |= ((uint32_t *) end)[0]; #endif break; -#elif BYTE_ORDER == BIG_ENDIAN +#else OR_BYTE(3); #endif case 3: diff --git a/test/ruby/test_string.rb b/test/ruby/test_string.rb index 2ff8458d9b..7ce1c0666c 100644 --- a/test/ruby/test_string.rb +++ b/test/ruby/test_string.rb @@ -886,6 +886,8 @@ class TestString < Test::Unit::TestCase assert_not_equal(S("hello").hash, S("helLO").hash) bug4104 = '[ruby-core:33500]' assert_not_equal(S("a").hash, S("a\0").hash, bug4104) + bug9172 = '[ruby-core:58658] [Bug #9172]' + assert_not_equal(S("sub-setter").hash, S("discover").hash, bug9172) end def test_hash_random |