Fix Error clone() when structuredClone uses bad prototype

lib/clone.js

@@ -175,7 +175,7 @@ internals.base = function (obj, baseProto, options) {
     /* $lab:coverage:off$ */
     else if (baseProto === Types.error && internals.structuredCloneExists) {
         const err = structuredClone(obj);                    // Needed to copy internal stack state
-        if (proto !== baseProto) {
+        if (Object.getPrototypeOf(err) !== proto) {
             Object.setPrototypeOf(err, proto);               // Fix prototype
         }
 

test/clone.js

@@ -886,4 +886,32 @@ describe('clone()', () => {
         const b = Hoek.clone(a);
         expect(b.x).to.not.exist();
     });
+
+    it('handles structuredClone not returning proper Error instances', { skip: typeof structuredClone !== 'function' }, () => {
+
+        // This can happen when running in a VM
+
+        const error = new Error('blam');
+
+        const origStructuredClone = structuredClone;
+        try {
+            structuredClone = function (obj) {
+
+                const clone = origStructuredClone.call(this, obj);
+                if (obj === error) {
+                    Object.setPrototypeOf(clone, Object);
+                }
+
+                return clone;
+            };
+
+            var cloned = Hoek.clone(error);
+        }
+        finally {
+            structuredClone = origStructuredClone;
+        }
+
+        expect(cloned).to.be.instanceOf(Error);
+        expect(cloned).to.equal(error);
+    });
 });