]> git.ipfire.org Git - thirdparty/samba.git/log
thirdparty/samba.git
4 years agoselftest: Only set netbios aliases for the ad_member env
Andreas Schneider [Thu, 10 Jun 2021 14:20:28 +0000 (16:20 +0200)] 
selftest: Only set netbios aliases for the ad_member env

The provision_ad_member() function is reused by different
setup_ad_member*() functions. Each environment needs to have unique
netbios aliases as they are all in the same network.
The aliases should only be set for the 'ad_member' environment.

Signed-Off-By: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Fri Jun 11 01:26:36 UTC 2021 on sn-devel-184

4 years agos3:tests: Fix passing the configuration to Smbclient_netbios_aliases test
Andreas Schneider [Thu, 10 Jun 2021 12:28:44 +0000 (12:28 +0000)] 
s3:tests: Fix passing the configuration to Smbclient_netbios_aliases test

Signed-Off-By: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agos3: smbd: Cleanup - Remove #ifdef'ed out load_inherited_info().
Jeremy Allison [Wed, 9 Jun 2021 16:39:04 +0000 (09:39 -0700)] 
s3: smbd: Cleanup - Remove #ifdef'ed out load_inherited_info().

I commented this out but forgot to remove in the
previous mega-patch.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jun 10 21:36:11 UTC 2021 on sn-devel-184

4 years agosmbd: remove more dead code from dos_mode_at_vfs_get_dosmode_done()
Jeremy Allison [Thu, 10 Jun 2021 17:55:30 +0000 (10:55 -0700)] 
smbd: remove more dead code from dos_mode_at_vfs_get_dosmode_done()

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agosmbd: remove unneeded code from dos_mode_at_vfs_get_dosmode_done()
Ralph Boehme [Thu, 10 Jun 2021 14:15:25 +0000 (16:15 +0200)] 
smbd: remove unneeded code from dos_mode_at_vfs_get_dosmode_done()

This is not used anymore since e7a90fd7a173d8e3cd5a2bb163df61758b2b973f.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jun 10 19:22:23 UTC 2021 on sn-devel-184

4 years agos3:smbd: Remove unnessesary NULL check for req
Andreas Schneider [Thu, 10 Jun 2021 07:14:22 +0000 (09:14 +0200)] 
s3:smbd: Remove unnessesary NULL check for req

We already dereference req earlier. So if it is NULL it already
segfaulted much earlier.

Found by covscan.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Pavel Filipenský <pfilipen@redhat.com>
Reviewed-by: Noel Power <npower@samba.org>
Autobuild-User(master): Noel Power <npower@samba.org>
Autobuild-Date(master): Thu Jun 10 11:22:19 UTC 2021 on sn-devel-184

4 years agos3:smbd: Remove unnessesary NULL check for fsp
Andreas Schneider [Thu, 10 Jun 2021 07:05:53 +0000 (09:05 +0200)] 
s3:smbd: Remove unnessesary NULL check for fsp

We already dereference fsp earlier. So if it is NULL it already
segfaulted much earlier.

Found by covscan.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Pavel Filipenský <pfilipen@redhat.com>
Reviewed-by: Noel Power <npower@samba.org>
4 years agolibrpc: Make sure num_protocols is initialized
Andreas Schneider [Thu, 10 Jun 2021 06:53:19 +0000 (08:53 +0200)] 
librpc: Make sure num_protocols is initialized

Found by covscan.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Pavel Filipenský <pfilipen@redhat.com>
Reviewed-by: Noel Power <npower@samba.org>
4 years agos3:smbd: Make sure smb_fname is set and not NULL in dos_mode_post()
Andreas Schneider [Thu, 10 Jun 2021 06:51:03 +0000 (08:51 +0200)] 
s3:smbd: Make sure smb_fname is set and not NULL in dos_mode_post()

Found by covscan.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Pavel Filipenský <pfilipen@redhat.com>
Reviewed-by: Noel Power <npower@samba.org>
4 years agos3: VFS: default: Add proc_fd's fallback for vfswrap_fchown().
Jeremy Allison [Wed, 9 Jun 2021 22:57:38 +0000 (15:57 -0700)] 
s3: VFS: default: Add proc_fd's fallback for vfswrap_fchown().

https://bugzilla.samba.org/show_bug.cgi?id=14734

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Noel Power <npower@samba.org>
Autobuild-User(master): Noel Power <npower@samba.org>
Autobuild-Date(master): Thu Jun 10 09:16:22 UTC 2021 on sn-devel-184

4 years agoselftest/gdb_backtrace: remove duplicate assignment.
Douglas Bagnall [Thu, 27 May 2021 02:50:55 +0000 (14:50 +1200)] 
selftest/gdb_backtrace: remove duplicate assignment.

See 18 or so lines up.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jun 10 01:17:52 UTC 2021 on sn-devel-184

4 years agosamba-tool: stick to the point with --version
Douglas Bagnall [Tue, 1 Jun 2021 00:48:24 +0000 (00:48 +0000)] 
samba-tool: stick to the point with --version

We were doing this:

    $ bin/samba-tool --version
    samba-tool: no such subcommand: --version

    4.15.0pre1-DEVELOPERBUILD
    $

which is silly. Now we'll just see the version.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agosamba-tool dns zoneoptions --help improvement
Douglas Bagnall [Tue, 1 Jun 2021 00:58:28 +0000 (12:58 +1200)] 
samba-tool dns zoneoptions --help improvement

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agoidl: two comment typos in two lines
Douglas Bagnall [Thu, 22 Apr 2021 13:00:56 +0000 (01:00 +1200)] 
idl: two comment typos in two lines

it is the density that got to me.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agos3:pylibsmb: improve return types (false => NULL)
Douglas Bagnall [Thu, 27 May 2021 03:39:19 +0000 (15:39 +1200)] 
s3:pylibsmb: improve return types (false => NULL)

NULL, a.k.a. (void *)false;

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agos3:pylibsmb: avoid small leaks in cli_notify_get_changes
Douglas Bagnall [Thu, 27 May 2021 03:32:45 +0000 (15:32 +1200)] 
s3:pylibsmb: avoid small leaks in cli_notify_get_changes

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agopython: remove obsolete samba_external directory
Douglas Bagnall [Fri, 21 May 2021 04:50:02 +0000 (16:50 +1200)] 
python: remove obsolete samba_external directory

It seems this was once meant to contain third-party python libraries
that we hoped would already be on the system, of which there was only
ever one candidate, dnspython, which we moved then stopped bundling
years ago.

The ancestor of this directory, 'source4/scripting/python/samba_external/'
was unused since 4bbc3ff037026c72f3249f59c1b5af69a6ad6d69 (2010) and
the current location has never done anything.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agosamba-tool domain: improve error message when `patch` fails
Douglas Bagnall [Thu, 20 May 2021 23:41:37 +0000 (11:41 +1200)] 
samba-tool domain: improve error message when `patch` fails

The old message confused even the wisest among us:

https://lists.samba.org/archive/samba/2021-May/236021.html

and while /user/bin/patch might be overly specific, it should point
people in the right direction.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agos3:tests: Fix the test_smbclient_netbios_aliases
Andreas Schneider [Wed, 9 Jun 2021 15:06:45 +0000 (17:06 +0200)] 
s3:tests: Fix the test_smbclient_netbios_aliases

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jun 10 00:24:22 UTC 2021 on sn-devel-184

4 years agosamba-tool: gpo admxload mkdir -p
David Mulder [Tue, 8 Jun 2021 15:30:37 +0000 (09:30 -0600)] 
samba-tool: gpo admxload mkdir -p

Ensure all directories in the path are created,
otherwise admx upload fails here.

Signed-off-by: David Mulder <dmulder@suse.com>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Autobuild-User(master): David Mulder <dmulder@samba.org>
Autobuild-Date(master): Wed Jun  9 22:26:42 UTC 2021 on sn-devel-184

4 years agosmbd: return correct timestamps for quota fake file
Ralph Boehme [Mon, 7 Jun 2021 17:02:56 +0000 (19:02 +0200)] 
smbd: return correct timestamps for quota fake file

Prior to 572d4e3a56eef00e29f93482daa21647af7310d0 it was sufficient to
initialize struct timespec to zero to return NTTIME 0 (ie not set) over
SMB.

This fixes the same problem from bug 14714 where the timestamps in an SMB2 CLOSE
response.

Windows of course does return *some* timestamps, but as it's neither documented
nor was I able to figure out where they would be coming from, as well as the
Windows client apparently doesn't care, I didn't bother with implementing some
sophisticated heuristic to return some timestamps.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14731

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Jun  9 20:38:02 UTC 2021 on sn-devel-184

4 years agosmbd: handle fake file handles in fdos_mode()
Ralph Boehme [Fri, 4 Jun 2021 13:54:20 +0000 (15:54 +0200)] 
smbd: handle fake file handles in fdos_mode()

This ensures SMB requests on the quote fake file "$Extend/$Quota" don't hit the
VFS, where specifically in vfs_gpfs we log an error message if we fail to read
the DOS attributes for a file with

  vfs_gpfs_get_dos_attributes: Getting winattrs failed for $Extend/$Quota

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14731

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agosmbd: add dosmode_from_fake_filehandle()
Ralph Boehme [Fri, 4 Jun 2021 14:31:20 +0000 (16:31 +0200)] 
smbd: add dosmode_from_fake_filehandle()

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14731

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agosmbtorture: verify attributes on fake quota file handle
Ralph Boehme [Mon, 7 Jun 2021 17:03:05 +0000 (19:03 +0200)] 
smbtorture: verify attributes on fake quota file handle

The expected DOS attributes are taken from a Windows 2016 server. The expected
timestamps are what Samba has returned before commit 572d4e3a56eef00e29f9348:
NTTIME(0), ie no value.

The upcoming fix will restore this behaviour. Windows of course does
return *some* timestamps, but as it's neither documented nor was I able to
figure out where they would be coming from, as well as the Windows client apparently
doesn't care, I didn't bother with implementing some sophisticated heuristic to
return some timestamps.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14731

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agogpo: Apply Group Policy GNOME Settings
David Mulder [Tue, 18 May 2021 15:37:10 +0000 (15:37 +0000)] 
gpo: Apply Group Policy GNOME Settings

Signed-off-by: David Mulder <dmulder@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Jun  9 17:44:25 UTC 2021 on sn-devel-184

4 years agogpo: Test Group Policy GNOME Setting
David Mulder [Tue, 18 May 2021 15:35:24 +0000 (15:35 +0000)] 
gpo: Test Group Policy GNOME Setting

Signed-off-by: David Mulder <dmulder@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agogpo: Add GNOME Settings ADMX templates
David Mulder [Mon, 17 May 2021 18:46:56 +0000 (12:46 -0600)] 
gpo: Add GNOME Settings ADMX templates

Signed-off-by: David Mulder <dmulder@suse.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
4 years agolib:mscat: Don't use deprecated types
Andreas Schneider [Tue, 8 Jun 2021 10:53:00 +0000 (12:53 +0200)] 
lib:mscat: Don't use deprecated types

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Alexander Bokovoy <ab@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Wed Jun  9 16:54:23 UTC 2021 on sn-devel-184

4 years agos3: VFS: solarisacl: Remove solarisacl_sys_acl_set_file().
Jeremy Allison [Wed, 9 Jun 2021 00:09:51 +0000 (17:09 -0700)] 
s3: VFS: solarisacl: Remove solarisacl_sys_acl_set_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Autobuild-User(master): Ralph Böhme <slow@samba.org>
Autobuild-Date(master): Wed Jun  9 14:04:13 UTC 2021 on sn-devel-184

4 years agos3: VFS: aixacl: Remove aixacl_sys_acl_set_file().
Jeremy Allison [Wed, 9 Jun 2021 00:08:42 +0000 (17:08 -0700)] 
s3: VFS: aixacl: Remove aixacl_sys_acl_set_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: posixacl: Remove posixacl_sys_acl_set_file().
Jeremy Allison [Wed, 9 Jun 2021 00:07:23 +0000 (17:07 -0700)] 
s3: VFS: posixacl: Remove posixacl_sys_acl_set_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: lib: sysacls: Remove all implementations of sys_acl_set_file().
Jeremy Allison [Wed, 9 Jun 2021 00:05:53 +0000 (17:05 -0700)] 
s3: lib: sysacls: Remove all implementations of sys_acl_set_file().

No longer called.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: default: Remove the sys_proc_fd_path() fallback code in vfswrap_sys_acl_set_...
Jeremy Allison [Wed, 9 Jun 2021 00:01:02 +0000 (17:01 -0700)] 
s3: VFS: default: Remove the sys_proc_fd_path() fallback code in vfswrap_sys_acl_set_fd().

Just pass through to sys_acl_set_fd(), which goes to posixacl_sys_acl_set_fd()
on posix ACL systems.

We already have identical code in posixacl_sys_acl_set_fd()
and these fallbacks are really system specific so we shouldn't
be doing them in more than one place.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: hpuxacl: Fix the funtion signature for hpuxacl_sys_acl_set_fd()
Jeremy Allison [Tue, 8 Jun 2021 23:56:44 +0000 (16:56 -0700)] 
s3: VFS: hpuxacl: Fix the funtion signature for hpuxacl_sys_acl_set_fd()

This is really unmaintained code and should be removed unless
someone from HP steps up..

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: lib: sysacls: Add the 'SMB_ACL_TYPE_T type' parameter to sys_acl_set_fd().
Jeremy Allison [Tue, 8 Jun 2021 23:49:33 +0000 (16:49 -0700)] 
s3: lib: sysacls: Add the 'SMB_ACL_TYPE_T type' parameter to sys_acl_set_fd().

Pass it through to the backends. The default posixacl_sys_acl_set_fd()
already copes with this anyway, as does the AIX, and Solaris backends.

The HPUX code isn't compiled and was broken anyway (there was a
missmatch of the number of parameters being passed to the
sys_acl_set_fd_fn backend hpuxacl_sys_acl_set_fd()), and HPUX
doesn't have fd-based ACLs, so just switch to calling hpuxacl_sys_acl_set_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: non_posix_acls: Remove non_posix_sys_acl_blob_get_file_helper(). No longer...
Jeremy Allison [Tue, 8 Jun 2021 23:35:24 +0000 (16:35 -0700)] 
s3: VFS: non_posix_acls: Remove non_posix_sys_acl_blob_get_file_helper(). No longer used.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: posixacl: Remove posixacl_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:34:40 +0000 (16:34 -0700)] 
s3: VFS: posixacl: Remove posixacl_sys_acl_get_file().

No longer used.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: lib: sysacls: Remove sys_acl_get_file(). No longer used.
Jeremy Allison [Tue, 8 Jun 2021 23:21:14 +0000 (16:21 -0700)] 
s3: lib: sysacls: Remove sys_acl_get_file(). No longer used.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: Update status of SMB_VFS_SYS_ACL_BLOB_GET_FILE
Jeremy Allison [Tue, 8 Jun 2021 23:17:59 +0000 (16:17 -0700)] 
s3: VFS: Update status of SMB_VFS_SYS_ACL_BLOB_GET_FILE

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: RIP SMB_VFS_SYS_ACL_BLOB_GET_FILE()
Jeremy Allison [Tue, 8 Jun 2021 23:16:32 +0000 (16:16 -0700)] 
s3: VFS: RIP SMB_VFS_SYS_ACL_BLOB_GET_FILE()

                        (\  _  /)
                        ( \ O / )
                         (// \\)
                            X
                           / \
                          /___\
                   _____/      \\_____
                  |         +         ||
                  |                   ||
                  |   SMB_VFS_SYS_ACL ||
                  | BLOB_GET_FILE()   ||
                  |                   ||
                  |                   ||
                  |                   ||
                  |  _     ___   _    ||
                  | | \     |   | \   ||
                  | |  |    |   |  |  ||
                  | |_/     |   |_/   ||
                  | | \     |   |     ||
                  | |  \    |   |     ||
                  | |   \. _|_. | .   ||
                  |                   ||
          *     * | *   **    * **    |**     **
           \)),.,\(/.,(//,,..,,\||(,,.,\\,.((//

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: posixacl: Remove call to posix_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:10:45 +0000 (16:10 -0700)] 
s3: VFS: posixacl: Remove call to posix_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: Remove zfsacl_fail__sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:09:09 +0000 (16:09 -0700)] 
s3: VFS: Remove zfsacl_fail__sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: Remove call to posix_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:08:29 +0000 (16:08 -0700)] 
s3: VFS: Remove call to posix_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: nfs4acl_xattr: Remove call to nfs4acl_xattr_fail__sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:07:43 +0000 (16:07 -0700)] 
s3: VFS: nfs4acl_xattr: Remove call to nfs4acl_xattr_fail__sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: hpuxacl: Remove call to posix_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:07:02 +0000 (16:07 -0700)] 
s3: VFS: hpuxacl: Remove call to posix_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: gpfs: Remove gpfsacl_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:06:26 +0000 (16:06 -0700)] 
s3: VFS: gpfs: Remove gpfsacl_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: glusterfs: Remove call to posix_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:05:33 +0000 (16:05 -0700)] 
s3: VFS: glusterfs: Remove call to posix_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: fake_acls: Remove call to posix_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:04:55 +0000 (16:04 -0700)] 
s3: VFS: fake_acls: Remove call to posix_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: ceph: Remove call to posix_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:04:21 +0000 (16:04 -0700)] 
s3: VFS: ceph: Remove call to posix_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: aixaxcl2: Remove aixjfs2_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:03:40 +0000 (16:03 -0700)] 
s3: VFS: aixaxcl2: Remove aixjfs2_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: aixacl: Remove call to posix_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:02:48 +0000 (16:02 -0700)] 
s3: VFS: aixacl: Remove call to posix_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: afsacl: Remove afsacl_sys_acl_blob_get_file().
Jeremy Allison [Tue, 8 Jun 2021 23:01:50 +0000 (16:01 -0700)] 
s3: VFS: afsacl: Remove afsacl_sys_acl_blob_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: Update status of SMB_VFS_SYS_ACL_GET_FILE
Jeremy Allison [Tue, 8 Jun 2021 22:59:34 +0000 (15:59 -0700)] 
s3: VFS: Update status of SMB_VFS_SYS_ACL_GET_FILE

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: RIP SMB_VFS_SYS_ACL_GET_FILE()
Jeremy Allison [Tue, 8 Jun 2021 22:56:20 +0000 (15:56 -0700)] 
s3: VFS: RIP SMB_VFS_SYS_ACL_GET_FILE()

                    (\  _  /)
                    ( \ O / )
                     (// \\)
                        X
                       / \
                      /___\
               _____/      \\_____
              |         +         ||
              |                   ||
              |   SMB_VFS_SYS_    ||
              |  ACL_GET_FILE()   ||
              |                   ||
              |                   ||
              |                   ||
              |  _     ___   _    ||
              | | \     |   | \   ||
              | |  |    |   |  |  ||
              | |_/     |   |_/   ||
              | | \     |   |     ||
              | |  \    |   |     ||
              | |   \. _|_. | .   ||
              |                   ||
      *     * | *   **    * **    |**     **
       \)),.,\(/.,(//,,..,,\||(,,.,\\,.((//

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: posixacl: Remove call to posixacl_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:37:35 +0000 (15:37 -0700)] 
s3: VFS: posixacl: Remove call to posixacl_sys_acl_get_file().

We can't remove the code yet until the callers inside sysacl.c
are gone. We must remove the VFS function first.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: zfsacl: Remove zfsacl_fail__sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:26:40 +0000 (15:26 -0700)] 
s3: VFS: zfsacl: Remove zfsacl_fail__sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: unityed_media: Remove um_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:26:04 +0000 (15:26 -0700)] 
s3: VFS: unityed_media: Remove um_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: solarisacl: Make solarisacl_sys_acl_get_file() static. Still called internally.
Jeremy Allison [Tue, 8 Jun 2021 22:25:14 +0000 (15:25 -0700)] 
s3: VFS: solarisacl: Make solarisacl_sys_acl_get_file() static. Still called internally.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: nfs4acl_xattr: Remove nfs4acl_xattr_fail__sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:23:47 +0000 (15:23 -0700)] 
s3: VFS: nfs4acl_xattr: Remove nfs4acl_xattr_fail__sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: media_harmony: Remove mh_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:22:59 +0000 (15:22 -0700)] 
s3: VFS: media_harmony: Remove mh_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: hpuxacl: Make hpuxacl_sys_acl_get_file() static.
Jeremy Allison [Tue, 8 Jun 2021 22:21:59 +0000 (15:21 -0700)] 
s3: VFS: hpuxacl: Make hpuxacl_sys_acl_get_file() static.

Called internally as HPUX has no fd-based acl functions.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: gpfs: Remove gpfsacl_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:21:06 +0000 (15:21 -0700)] 
s3: VFS: gpfs: Remove gpfsacl_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: glusterfs: Remove call to posixacl_xattr_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:20:05 +0000 (15:20 -0700)] 
s3: VFS: glusterfs: Remove call to posixacl_xattr_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: fake_acls: Remove fake_acls_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:19:03 +0000 (15:19 -0700)] 
s3: VFS: fake_acls: Remove fake_acls_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: ceph: Remove call to posixacl_xattr_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:18:14 +0000 (15:18 -0700)] 
s3: VFS: ceph: Remove call to posixacl_xattr_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: catia: Remove catia_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:17:02 +0000 (15:17 -0700)] 
s3: VFS: catia: Remove catia_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: cap: Remove cap_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:16:23 +0000 (15:16 -0700)] 
s3: VFS: cap: Remove cap_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: aixacl2: Remove aixjfs2_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:15:41 +0000 (15:15 -0700)] 
s3: VFS: aixacl2: Remove aixjfs2_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: aixacl: Remove aixacl_sys_acl_get_file().
Jeremy Allison [Tue, 8 Jun 2021 22:14:57 +0000 (15:14 -0700)] 
s3: VFS: aixacl: Remove aixacl_sys_acl_get_file().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: Remove posix_get_nt_acl().No longer used.
Jeremy Allison [Tue, 8 Jun 2021 20:09:43 +0000 (13:09 -0700)] 
s3: smbd: Remove posix_get_nt_acl().No longer used.

Comment out load_inherited_info() as that was the
only caller.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: Update status of SMB_VFS_NT_ACL_AT.
Jeremy Allison [Tue, 8 Jun 2021 20:06:42 +0000 (13:06 -0700)] 
s3: VFS: Update status of SMB_VFS_NT_ACL_AT.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agovfs: RIP SMB_VFS_GET_NT_ACL_AT()
Jeremy Allison [Tue, 8 Jun 2021 20:04:57 +0000 (13:04 -0700)] 
vfs: RIP SMB_VFS_GET_NT_ACL_AT()

                  .--. .-,       .-..-.__
                .'(`.-` \_.-'-./`  |\_( "\__
             __.>\ ';  _;---,._|   / __/`'--)
            /.--.  : |/' _.--.<|  /  | |
        _..-'    `\     /' /`  /_/ _/_/
         >_.-``-. `Y  /' _;---.`|/))))
        '` .-''. \|:  .'   __, .-'"`
         .'--._ `-:  \/:  /'  '.\             _|_
             /.'`\ :;   /'      `-           `-|-`
            -`    |     |                      |
                  :.; : |                  .-'~^~`-.
                  |:    |                .' _     _ `.
                  |:.   |                | |_) | |_) |
                  :. :  |                | | \ | |   |
                  : ;   |                |           |
                  : ;   |                |  SMB_VFS  |
                  : ;   |                |  GET_NT_  |
                  : ;   |                |  ACL_AT   |
                .jgs. : ;                |           |
        -."-/\\\/:::.    `\."-._'."-"_\\-|           |///."-
        " -."-.\\"-."//.-".`-."_\\-.".-\\`=.........=`//-".

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: acl_common: Remove the dirfsp parameter from validate_nt_acl_blob().
Jeremy Allison [Tue, 8 Jun 2021 20:02:42 +0000 (13:02 -0700)] 
s3: VFS: acl_common: Remove the dirfsp parameter from validate_nt_acl_blob().

No longer needed.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: acl_common: Remove the pathname-based calls in validate_nt_acl_blob().
Jeremy Allison [Tue, 8 Jun 2021 20:01:12 +0000 (13:01 -0700)] 
s3: VFS: acl_common: Remove the pathname-based calls in validate_nt_acl_blob().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: acl_common: Remove get_nt_acl_common_at().
Jeremy Allison [Tue, 8 Jun 2021 19:59:31 +0000 (12:59 -0700)] 
s3: VFS: acl_common: Remove get_nt_acl_common_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: zfsacl: Remove zfsacl_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:51:26 +0000 (12:51 -0700)] 
s3: VFS: zfsacl: Remove zfsacl_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: snapper: Remove snapper_gmt_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:50:31 +0000 (12:50 -0700)] 
s3: VFS: snapper: Remove snapper_gmt_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: nfs4acl_xattr: Remove nfs4acl_xattr_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:49:03 +0000 (12:49 -0700)] 
s3: VFS: nfs4acl_xattr: Remove nfs4acl_xattr_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: media_harmony. Remove mh_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:48:12 +0000 (12:48 -0700)] 
s3: VFS: media_harmony. Remove mh_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: gpfs. Remove gpfsacl_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:47:27 +0000 (12:47 -0700)] 
s3: VFS: gpfs. Remove gpfsacl_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: ceph_snapshots. Remove ceph_snap_gmt_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:46:34 +0000 (12:46 -0700)] 
s3: VFS: ceph_snapshots. Remove ceph_snap_gmt_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: catia. Remove catia_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:45:43 +0000 (12:45 -0700)] 
s3: VFS: catia. Remove catia_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: acl_xattr: Remove unused get_acl_blob_at().
Jeremy Allison [Tue, 8 Jun 2021 19:44:51 +0000 (12:44 -0700)] 
s3: VFS: acl_xattr: Remove unused get_acl_blob_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: acl_xattr: Remove acl_xattr_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:44:05 +0000 (12:44 -0700)] 
s3: VFS: acl_xattr: Remove acl_xattr_get_nt_acl_at().

Comment out get_acl_blob_at() as no longer used.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: acl_tdb: Remove unused get_acl_blob_at().
Jeremy Allison [Tue, 8 Jun 2021 19:42:52 +0000 (12:42 -0700)] 
s3: VFS: acl_tdb: Remove unused get_acl_blob_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: acl_tdb. Remove acl_tdb_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:42:00 +0000 (12:42 -0700)] 
s3: VFS: acl_tdb. Remove acl_tdb_get_nt_acl_at().

Comment out get_acl_blob_at() as no longer used.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: aixacl2: Remove aixjfs2_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:40:40 +0000 (12:40 -0700)] 
s3: VFS: aixacl2: Remove aixjfs2_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: afsacl: Remove afsacl_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:39:56 +0000 (12:39 -0700)] 
s3: VFS: afsacl: Remove afsacl_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: VFS: shadow_copy2: Remove shadow_copy2_get_nt_acl_at().
Jeremy Allison [Tue, 8 Jun 2021 19:38:43 +0000 (12:38 -0700)] 
s3: VFS: shadow_copy2: Remove shadow_copy2_get_nt_acl_at().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3 VFS: glusterfs: Remove get_nt_acl_at_fn().
Jeremy Allison [Tue, 8 Jun 2021 19:34:25 +0000 (12:34 -0700)] 
s3 VFS: glusterfs: Remove get_nt_acl_at_fn().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: open_directory(). Pass down the real parent_dir->fsp to smbd_check_access_r...
Jeremy Allison [Tue, 8 Jun 2021 19:28:24 +0000 (12:28 -0700)] 
s3: smbd: open_directory(). Pass down the real parent_dir->fsp to smbd_check_access_rights_fsp().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: open_file(). Pass down the real parent_dir->fsp to smbd_check_access_rights...
Jeremy Allison [Tue, 8 Jun 2021 19:27:09 +0000 (12:27 -0700)] 
s3: smbd: open_file(). Pass down the real parent_dir->fsp to smbd_check_access_rights_fsp().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: smbd_check_access_rights_fsp(). Add dirfsp parameter.
Jeremy Allison [Tue, 8 Jun 2021 19:24:17 +0000 (12:24 -0700)] 
s3: smbd: smbd_check_access_rights_fsp(). Add dirfsp parameter.

Pass down to smbd_check_access_rights_sd().

Always pass conn->cwd_fsp for now.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: smbd_check_access_rights_sd(). Add dirfsp parameter.
Jeremy Allison [Tue, 8 Jun 2021 19:16:04 +0000 (12:16 -0700)] 
s3: smbd: smbd_check_access_rights_sd(). Add dirfsp parameter.

Pass down to parent_override_delete().

Always pass fsp->conn->cwd_fsp for now.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: parent_override_delete(). Add dirfsp parameter.
Jeremy Allison [Tue, 8 Jun 2021 19:14:19 +0000 (12:14 -0700)] 
s3: smbd: parent_override_delete(). Add dirfsp parameter.

Pass down to can_delete_file_in_directory().

Always pass conn->cwd_fsp for now.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: open_directory(). Start passing a real parent dirfsp to smbd_calculate_acce...
Jeremy Allison [Tue, 8 Jun 2021 19:02:16 +0000 (12:02 -0700)] 
s3: smbd: open_directory(). Start passing a real parent dirfsp to smbd_calculate_access_mask_fsp().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: open_file_ntcreate(). Start passing a real parent dirfsp to smbd_calculate_...
Jeremy Allison [Tue, 8 Jun 2021 19:00:56 +0000 (12:00 -0700)] 
s3: smbd: open_file_ntcreate(). Start passing a real parent dirfsp to smbd_calculate_access_mask_fsp().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: smbd_calculate_access_mask_fsp(). Add dirfsp parameter.
Jeremy Allison [Tue, 8 Jun 2021 18:56:25 +0000 (11:56 -0700)] 
s3: smbd: smbd_calculate_access_mask_fsp(). Add dirfsp parameter.

Pass this down into smbd_calculate_maximum_allowed_access_fsp().

Currently pass fsp->conn->cwd_fsp everywhere.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: smbd_calculate_maximum_allowed_access_fsp(), add parent dirfsp parameter...
Jeremy Allison [Tue, 8 Jun 2021 18:40:11 +0000 (11:40 -0700)] 
s3: smbd: smbd_calculate_maximum_allowed_access_fsp(), add parent dirfsp parameter and pass to can_delete_file_in_directory().

Pass the same fsp->conn->cwd_fsp parameter to can_delete_file_in_directory()
dirfsp for now.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
4 years agos3: smbd: Remove smbd_check_access_rights(). No longer used.
Jeremy Allison [Tue, 8 Jun 2021 18:37:57 +0000 (11:37 -0700)] 
s3: smbd: Remove smbd_check_access_rights(). No longer used.

There are now no more callers of SMB_VFS_GET_NT_ACL_AT().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>