diff options
author | Jeremy Allison <jra@samba.org> | 2011-06-23 15:06:16 -0700 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2011-06-24 01:18:11 +0200 |
commit | 5d7d52ceea6ac046940ae0a8112ee7ba79ce886d (patch) | |
tree | 63814023c1bb201d3eff7443a654a4b8d1804435 /source3/smbd/file_access.c | |
parent | a73abc0403b8267eeb15f9d5f394109495d1b667 (diff) | |
download | samba-5d7d52ceea6ac046940ae0a8112ee7ba79ce886d.tar.gz samba-5d7d52ceea6ac046940ae0a8112ee7ba79ce886d.tar.bz2 samba-5d7d52ceea6ac046940ae0a8112ee7ba79ce886d.zip |
Fix bug #8254 - "acl check permissions = no" does not work in all cases
Move lp_acl_check_permissions() into can_delete_file_in_directory()
where it makes sense. Remove ACL check when requesting DELETE_ACCESS
when lp_acl_check_permissions is false.
Thanks to John Janosik @ IBM for noticing this.
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Jun 24 01:18:11 CEST 2011 on sn-devel-104
Diffstat (limited to 'source3/smbd/file_access.c')
-rw-r--r-- | source3/smbd/file_access.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/source3/smbd/file_access.c b/source3/smbd/file_access.c index 960dcb703a..74855649ea 100644 --- a/source3/smbd/file_access.c +++ b/source3/smbd/file_access.c @@ -92,6 +92,11 @@ bool can_delete_file_in_directory(connection_struct *conn, return False; } + if (!lp_acl_check_permissions(SNUM(conn))) { + /* This option means don't check. */ + return true; + } + /* Get the parent directory permission mask and owners. */ if (!parent_dirname(ctx, smb_fname->base_name, &dname, NULL)) { return False; |