diff options
author | Jeremy Allison <jra@samba.org> | 2003-03-07 19:46:00 +0000 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2003-03-07 19:46:00 +0000 |
commit | 7f8d3a49b2cebab4b94db3cda54b3923442378c8 (patch) | |
tree | f89d17e62549e255230158e5e17f9bfeed94eb7b | |
parent | 1d3b8c528bebfa1971d1affe454a03453335786e (diff) | |
download | samba-7f8d3a49b2cebab4b94db3cda54b3923442378c8.tar.gz samba-7f8d3a49b2cebab4b94db3cda54b3923442378c8.tar.xz samba-7f8d3a49b2cebab4b94db3cda54b3923442378c8.zip |
Missed parentheses around complex logic.
Jeremy.
-rw-r--r-- | source/smbd/posix_acls.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/source/smbd/posix_acls.c b/source/smbd/posix_acls.c index 2aea3a2c902..01e3c31ba78 100644 --- a/source/smbd/posix_acls.c +++ b/source/smbd/posix_acls.c @@ -2593,7 +2593,8 @@ BOOL set_nt_acl(files_struct *fsp, uint32 security_info_sent, SEC_DESC *psd) * Do we need to chown ? */ - need_chown = (user != (uid_t)-1 && orig_uid != user || grp != (uid_t)-1 && orig_gid != grp); + if (((user != (uid_t)-1) && (orig_uid != user)) || (( grp != (uid_t)-1) && (orig_gid != grp))) + need_chown = True; /* * Chown before setting ACL only if we don't change the user, or |