Skip to content

Commit 15f7431

Browse files
committed
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 ca6b12e commit 15f7431

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
@@ -390,7 +390,7 @@ async function lchown(path, uid, gid) {
390390
if (O_SYMLINK !== undefined) {
391391
const fd = await open(path,
392392
O_WRONLY | O_SYMLINK);
393-
return fchmod(fd, uid, gid).finally(fd.close.bind(fd));
393+
return fchown(fd, uid, gid).finally(fd.close.bind(fd));
394394
}
395395
throw new ERR_METHOD_NOT_IMPLEMENTED();
396396
}

0 commit comments

Comments
 (0)