summaryrefslogtreecommitdiff
path: root/source3
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2003-03-07 19:46:51 +0000
committerJeremy Allison <jra@samba.org>2003-03-07 19:46:51 +0000
commit24c95539d593ccb703a4c381060a0bcad5052458 (patch)
treeab70792d33e1a723893dac4e8a0121aa037c5977 /source3
parent6a141b7963de61b1223031ea4012605aeca13bc7 (diff)
downloadsamba-24c95539d593ccb703a4c381060a0bcad5052458.tar.gz
samba-24c95539d593ccb703a4c381060a0bcad5052458.tar.bz2
samba-24c95539d593ccb703a4c381060a0bcad5052458.zip
Missed parentheses around complex logic.
Jeremy. (This used to be commit e81427c2d69be166afad94bc083e750e8f48fba7)
Diffstat (limited to 'source3')
-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