diff options
author | John Högberg <[email protected]> | 2018-03-19 13:18:08 +0100 |
---|---|---|
committer | John Högberg <[email protected]> | 2018-03-19 14:02:22 +0100 |
commit | f14a5306622994a6b49b25c63ec882a1551398ef (patch) | |
tree | f2e1c57288f586296138e9a2c305248c98434cd7 /erts/emulator/nifs/common | |
parent | d171399bb5a0e9f4c6029564ece4fbf89f3c76f8 (diff) | |
download | otp-f14a5306622994a6b49b25c63ec882a1551398ef.tar.gz otp-f14a5306622994a6b49b25c63ec882a1551398ef.tar.bz2 otp-f14a5306622994a6b49b25c63ec882a1551398ef.zip |
Fix file:change_group/change_owner
It wasn't possible to change group/owner separately, and our test
suite lacked coverage for that.
ERL-589
Diffstat (limited to 'erts/emulator/nifs/common')
-rw-r--r-- | erts/emulator/nifs/common/prim_file_nif.c | 6 | ||||
-rw-r--r-- | erts/emulator/nifs/common/prim_file_nif.h | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/erts/emulator/nifs/common/prim_file_nif.c b/erts/emulator/nifs/common/prim_file_nif.c index 6874f41d75..bbd9becb47 100644 --- a/erts/emulator/nifs/common/prim_file_nif.c +++ b/erts/emulator/nifs/common/prim_file_nif.c @@ -891,10 +891,10 @@ static ERL_NIF_TERM set_owner_nif(ErlNifEnv *env, int argc, const ERL_NIF_TERM a posix_errno_t posix_errno; efile_path_t path; - Uint32 uid, gid; + Sint32 uid, gid; - if(argc != 3 || !enif_get_uint(env, argv[1], &uid) - || !enif_get_uint(env, argv[2], &gid)) { + if(argc != 3 || !enif_get_int(env, argv[1], &uid) + || !enif_get_int(env, argv[2], &gid)) { return enif_make_badarg(env); } diff --git a/erts/emulator/nifs/common/prim_file_nif.h b/erts/emulator/nifs/common/prim_file_nif.h index cc9bc8f5c3..4194cdc7d9 100644 --- a/erts/emulator/nifs/common/prim_file_nif.h +++ b/erts/emulator/nifs/common/prim_file_nif.h @@ -177,7 +177,7 @@ posix_errno_t efile_set_permissions(const efile_path_t *path, Uint32 permissions /** @brief On Unix, this will set the owner/group to the given values. It will * do nothing on other platforms. */ -posix_errno_t efile_set_owner(const efile_path_t *path, Uint32 owner, Uint32 group); +posix_errno_t efile_set_owner(const efile_path_t *path, Sint32 owner, Sint32 group); /** @brief Resolves the final path of the given link. */ posix_errno_t efile_read_link(ErlNifEnv *env, const efile_path_t *path, ERL_NIF_TERM *result); |