diff options
author | Jeremy Allison <jra@samba.org> | 2003-03-07 19:46:51 +0000 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2003-03-07 19:46:51 +0000 |
commit | 24c95539d593ccb703a4c381060a0bcad5052458 (patch) | |
tree | ab70792d33e1a723893dac4e8a0121aa037c5977 /source3/smbd/posix_acls.c | |
parent | 6a141b7963de61b1223031ea4012605aeca13bc7 (diff) | |
download | samba-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/smbd/posix_acls.c')
-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 |