fs: throw fs.mkdtempSync errors in JS land

PR-URL: https://github.com/nodejs/node/pull/18871
Refs: https://github.com/nodejs/node/issues/18106
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

1 files changed, 6 insertions, 1 deletions

diff --git a/lib/fs.js b/lib/fs.js
index 49b4d434e8..99d6347690 100644
--- a/lib/fs.js
+++ b/lib/fs.js
@@ -1839,7 +1839,12 @@ fs.mkdtempSync = function(prefix, options) {
                                prefix);
   }
   nullCheck(prefix, 'prefix');
-  return binding.mkdtemp(`${prefix}XXXXXX`, options.encoding);
+  const path = `${prefix}XXXXXX`;
+  const ctx = { path };
+  const result = binding.mkdtemp(path, options.encoding,
+                                 undefined, ctx);
+  handleErrorFromBinding(ctx);
+  return result;
 };