summaryrefslogtreecommitdiff
path: root/source3/smbd/posix_acls.c
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2003-03-07 19:46:00 +0000
committerJeremy Allison <jra@samba.org>2003-03-07 19:46:00 +0000
commit0f30afc9128f661226c8d20ce4e447e3715703c8 (patch)
tree862f96440ceb7397c54187d3a8a595f93a721946 /source3/smbd/posix_acls.c
parente9f51a6e38eb270c0d3643292978410e048ca173 (diff)
downloadsamba-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/smbd/posix_acls.c')
-rw-r--r--source3/smbd/posix_acls.c3
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