[compat] undefine clone on Store and StoreContext

Author: kares

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

@@ -69,6 +69,7 @@ static void createX509Store(final Ruby runtime, final RubyModule X509, final Rub
         Store.addReadWriteAttribute(context, "chain");
         Store.defineAnnotatedMethods(X509Store.class);
         Store.undefineMethod("dup");
+        Store.undefineMethod("clone");
 
         X509StoreContext.createX509StoreContext(runtime, X509);
     }

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

@@ -67,6 +67,7 @@ public static void createX509StoreContext(final Ruby runtime, final RubyModule X
                 (r, klass) -> new X509StoreContext(r, klass));
         StoreContext.defineAnnotatedMethods(X509StoreContext.class);
         StoreContext.undefineMethod("dup");
+        StoreContext.undefineMethod("clone");
     }
 
     private static RubyClass _StoreContext(final Ruby runtime) {

src/test/ruby/x509/test_x509store.rb

@@ -101,6 +101,13 @@ def test_add_file_raises_with_invalid_pem # GH-285
     assert_raise(OpenSSL::X509::StoreError) { store.add_file(invalid) }
   end
 
+  # CRuby undefs initialize_copy, blocking both dup and clone
+  def test_store_dup_clone_not_allowed
+    store = OpenSSL::X509::Store.new
+    assert_raise(NoMethodError) { store.dup }
+    assert_raise(NoMethodError) { store.clone }
+  end
+
   def test_use_non_existing_cert_file
     ENV['SSL_CERT_FILE'] = 'non-existing-file.crt'
     store = OpenSSL::X509::Store.new