[fix] integer overflow in hash_old (#218)

introduced in #216

Author: MariuszCwikla

src/main/java/org/jruby/ext/openssl/X509Name.java

@@ -633,7 +633,7 @@ public RubyFixnum hash() {
 
     @JRubyMethod
     public RubyFixnum hash_old() {
-        int hash;
+        long hash;
         try {
             hash = Name.hashOld( getX500Name() );
         }

src/main/java/org/jruby/ext/openssl/x509store/Name.java

@@ -59,12 +59,12 @@ public Name(final X500Name name) {
         this.name = name;
     }
 
-    public static int hashOld(final X500Name name) throws IOException {
+    public static long hashOld(final X500Name name) throws IOException {
         try {
             final byte[] bytes = name.getEncoded();
             MessageDigest md5 = SecurityHelper.getMessageDigest("MD5");
             final byte[] digest = md5.digest(bytes);
-            int result = 0;
+            long result = 0;
             result |= digest[3] & 0xff; result <<= 8;
             result |= digest[2] & 0xff; result <<= 8;
             result |= digest[1] & 0xff; result <<= 8;

src/test/ruby/x509/test_x509name.rb

@@ -84,6 +84,8 @@ def test_hash_long_name
   def test_hash_old
     name = OpenSSL::X509::Name.new [['CN', 'nobody'], ['DC', 'example']]
     assert_equal 1460400684, name.hash_old
+    name = OpenSSL::X509::Name.new([['CN', 'foo'], ['DC', 'bar']])
+    assert_equal 3294068023, name.hash_old
   end
 
 end