diff options
| author | Jeremy Allison <jra@samba.org> | 2001-09-07 20:08:16 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2001-09-07 20:08:16 +0000 |
| commit | 84cca757397dba107fde479bc2043d8412713cf7 (patch) | |
| tree | 192875a955dcc454711504037e0969ee4f7a374d /source3/smbd/posix_acls.c | |
| parent | 0135666934c5a8db796539b5414b72e1c2c8348d (diff) | |
| download | samba-84cca757397dba107fde479bc2043d8412713cf7.tar.gz samba-84cca757397dba107fde479bc2043d8412713cf7.tar.bz2 samba-84cca757397dba107fde479bc2043d8412713cf7.zip | |
Don't fail if no owner/group owner set. Use existing owners.
Jeremy.
(This used to be commit 9961c4c1a3b2dbf8d1062bc1fa103488c0d0ba79)
Diffstat (limited to 'source3/smbd/posix_acls.c')
| -rw-r--r-- | source3/smbd/posix_acls.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/source3/smbd/posix_acls.c b/source3/smbd/posix_acls.c index 39d0b83bf6..48fac44bbf 100644 --- a/source3/smbd/posix_acls.c +++ b/source3/smbd/posix_acls.c @@ -419,7 +419,7 @@ static BOOL unpack_nt_owners(SMB_STRUCT_STAT *psbuf, uid_t *puser, gid_t *pgrp, if(security_info_sent == 0) { DEBUG(0,("unpack_nt_owners: no security info sent !\n")); - return False; + return True; } /* @@ -438,8 +438,10 @@ static BOOL unpack_nt_owners(SMB_STRUCT_STAT *psbuf, uid_t *puser, gid_t *pgrp, if (security_info_sent & OWNER_SECURITY_INFORMATION) { sid_copy(&owner_sid, psd->owner_sid); - if (!sid_to_uid( &owner_sid, puser, &sid_type)) + if (!sid_to_uid( &owner_sid, puser, &sid_type)) { DEBUG(3,("unpack_nt_owners: unable to validate owner sid.\n")); + return False; + } } /* @@ -449,8 +451,10 @@ static BOOL unpack_nt_owners(SMB_STRUCT_STAT *psbuf, uid_t *puser, gid_t *pgrp, if (security_info_sent & GROUP_SECURITY_INFORMATION) { sid_copy(&grp_sid, psd->grp_sid); - if (!sid_to_gid( &grp_sid, pgrp, &sid_type)) + if (!sid_to_gid( &grp_sid, pgrp, &sid_type)) { DEBUG(3,("unpack_nt_owners: unable to validate group sid.\n")); + return False; + } } DEBUG(5,("unpack_nt_owners: owner_sids validated.\n")); |