Skip to content

Commit eb483db

Browse files
ChALkeRMylesBorins
authored andcommitted
fs: fchmod->fchown in promises/lchown
This was a clear error. chown should do chown, not chmod. PR-URL: #20407 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jamie Davis <davisjam@vt.edu> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
1 parent eb724f0 commit eb483db

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/internal/fs/promises.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -388,7 +388,7 @@ async function lchown(path, uid, gid) {
388388
if (O_SYMLINK !== undefined) {
389389
const fd = await open(path,
390390
O_WRONLY | O_SYMLINK);
391-
return fchmod(fd, uid, gid).finally(fd.close.bind(fd));
391+
return fchown(fd, uid, gid).finally(fd.close.bind(fd));
392392
}
393393
throw new ERR_METHOD_NOT_IMPLEMENTED();
394394
}

0 commit comments

Comments
 (0)