fs: throw fs.chownSync 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, 3 insertions, 1 deletions

diff --git a/lib/fs.js b/lib/fs.js
index 0d8d8ec4dc..e86360fc1a 100644
--- a/lib/fs.js
+++ b/lib/fs.js
@@ -1124,7 +1124,9 @@ fs.chownSync = function(path, uid, gid) {
   validatePath(path);
   validateUint32(uid, 'uid');
   validateUint32(gid, 'gid');
-  return binding.chown(pathModule.toNamespacedPath(path), uid, gid);
+  const ctx = { path };
+  binding.chown(pathModule.toNamespacedPath(path), uid, gid, undefined, ctx);
+  handleErrorFromBinding(ctx);
 };
 
 // exported for unit tests, not for public consumption