Prevent polyglot exceptions from being marshaled since we wouldn't be able to restore them properly.

Author: nirvdrum

CHANGELOG.md

@@ -13,6 +13,7 @@ Bug fixes:
 * Update `Process` methods to use `module_function` (@bjfish).
 * Fix `File::Stat`'s `#executable?` and `#executable_real?` predicates that unconditionally returned `true` for a superuser (#2690, @andrykonchin).
 * The `strip` option `--keep-section=.llvmbc` is not supported on macOS (#2697, @eregon).
+* Disallow the marshaling of polyglot exceptions since we can't properly reconstruct them (@nirvdrum).
 
 Compatibility:
 

spec/truffle/interop/polyglot/foreign_exception_spec.rb

@@ -90,6 +90,12 @@
     end
   end
 
+  it 'cannot be marshaled' do
+    -> {
+      Marshal.dump(@foreign)
+    }.should raise_error(TypeError)
+  end
+
   describe "when reaching the top-level" do
     it "is printed like a Ruby exception" do
       out = ruby_exe('raise Truffle::Debug.foreign_exception "main"', args: "2>&1", exit_status: 1, escape: false)

src/main/ruby/truffleruby/core/truffle/polyglot.rb

@@ -312,6 +312,14 @@ def backtrace_locations
     def backtrace
       backtrace_locations&.map(&:to_s)
     end
+
+    def marshal_dump
+      raise TypeError, "TruffleRuby foreign exceptions cannot be dumped: #{message}"
+    end
+
+    def marshal_load(...)
+      raise TypeError, 'TruffleRuby foreign exceptions cannot be restored'
+    end
   end
 
   module ExecutableTrait