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 | 0f30afc9128f661226c8d20ce4e447e3715703c8 (patch) | |
tree | 862f96440ceb7397c54187d3a8a595f93a721946 /source3 | |
parent | e9f51a6e38eb270c0d3643292978410e048ca173 (diff) | |
download | samba-0f30afc9128f661226c8d20ce4e447e3715703c8.tar.gz samba-0f30afc9128f661226c8d20ce4e447e3715703c8.tar.bz2 samba-0f30afc9128f661226c8d20ce4e447e3715703c8.zip |
Missed parentheses around complex logic.
Jeremy.
(This used to be commit 7f8d3a49b2cebab4b94db3cda54b3923442378c8)
Diffstat (limited to 'source3')
-rw-r--r-- | source3/smbd/posix_acls.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/source3/smbd/posix_acls.c b/source3/smbd/posix_acls.c index 2aea3a2c90..01e3c31ba7 100644 --- a/source3/smbd/posix_acls.c +++ b/source3/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 |