util: fix deprecated class prototype

Ensure the wrapped class prototype is exactly the unwrapped class
prototype, rather than an object whose prototype is the unwrapped
class prototype.

This ensures that instances of the unwrapped class are instances
of the wrapped class. This is useful when both a wrapped class and
a factory for the unwrapped class are both exposed.

Ref: https://github.com/nodejs/node/pull/8103
PR-URL: https://github.com/nodejs/node/pull/8105
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

1 files changed, 4 insertions, 1 deletions

diff --git a/lib/internal/util.js b/lib/internal/util.js
index 073879d9e9..055f8779b1 100644
--- a/lib/internal/util.js
+++ b/lib/internal/util.js
@@ -69,7 +69,10 @@ exports._deprecate = function(fn, msg) {
   // The wrapper will keep the same prototype as fn to maintain prototype chain
   Object.setPrototypeOf(deprecated, fn);
   if (fn.prototype) {
-    Object.setPrototypeOf(deprecated.prototype, fn.prototype);
+    // Setting this (rather than using Object.setPrototype, as above) ensures
+    // that calling the unwrapped constructor gives an instanceof the wrapped
+    // constructor.
+    deprecated.prototype = fn.prototype;
   }
 
   return deprecated;