samba.git
6 years agolib: Update pam_wrapper to 1.0.4
Andreas Schneider [Mon, 15 May 2017 15:25:27 +0000 (17:25 +0200)]
lib: Update pam_wrapper to 1.0.4

* Fix build on OpenBSD
* Fix a resource leak

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
6 years agolibads: abstract out SASL wrapping code
Alexander Bokovoy [Fri, 5 May 2017 12:37:20 +0000 (15:37 +0300)]
libads: abstract out SASL wrapping code

Prepare for rebasing libads on top of libsmbldap.

To make libads using 'struct smbldap_state' instead of direct LDAP
structure, we need to abstract out libads logic from connection
handling. SASL wrapping does not really depend on availability of LDAP
handle and does not need direct access to ADS_STRUCT. As result, we'll
be able to move SASL wrapping code under smbldap once the latter is able
to pass settings that libads passes to the SASL wrapping.

Signed-off-by: Alexander Bokovoy <ab@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
6 years agoSMB_INFO_ALLOCATION: return with UINT32_MAX rather than cutting at 4 bytes
Peter Somogyi [Mon, 15 May 2017 08:21:01 +0000 (10:21 +0200)]
SMB_INFO_ALLOCATION: return with UINT32_MAX rather than cutting at 4 bytes

An AIX cifs client queries SMB_INFO_ALLOCATION that has only 4 bytes to report
dfree and dsize. Return the max instead of random unexpeced numbers.

Signed-off-by: Peter Somogyi <psomogyi@hu.ibm.com>
Reviewed-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed May 17 05:49:46 CEST 2017 on sn-devel-144

6 years agowinbindd: Give winbindd_ads.c its own header
Volker Lendecke [Thu, 11 May 2017 19:38:50 +0000 (21:38 +0200)]
winbindd: Give winbindd_ads.c its own header

Not necessary to compile all of winbind when playing with ads.h

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
6 years agolibsmb: proto.h does not need ads.h
Volker Lendecke [Thu, 11 May 2017 19:30:30 +0000 (21:30 +0200)]
libsmb: proto.h does not need ads.h

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
6 years agosmbd/smb2_ioctl: check for NULL dst_fsp before use
David Disseldorp via samba-technical [Tue, 16 May 2017 12:03:32 +0000 (14:03 +0200)]
smbd/smb2_ioctl: check for NULL dst_fsp before use

...not after. CID #1409040.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
Autobuild-User(master): Ralph Böhme <slow@samba.org>
Autobuild-Date(master): Wed May 17 01:46:52 CEST 2017 on sn-devel-144

6 years agoldb-samba: Fix a possible NULL pointer dereference
Andreas Schneider [Mon, 8 May 2017 09:44:34 +0000 (11:44 +0200)]
ldb-samba: Fix a possible NULL pointer dereference

CID: #1402566
CID: #1402567

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Tue May 16 16:45:43 CEST 2017 on sn-devel-144

6 years agos3:winbind: Use a talloc stackframe for rpc_query_user_list
Andreas Schneider [Mon, 8 May 2017 09:34:54 +0000 (11:34 +0200)]
s3:winbind: Use a talloc stackframe for rpc_query_user_list

CID #1401581

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agolibrpc:ndr: Set the length to 1 if we assign and empty string
Andreas Schneider [Mon, 8 May 2017 09:22:51 +0000 (11:22 +0200)]
librpc:ndr: Set the length to 1 if we assign and empty string

CID #1399648

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agoctdb-tests: Use tighter pattern for matching expected output
Amitay Isaacs [Thu, 20 Apr 2017 04:34:54 +0000 (14:34 +1000)]
ctdb-tests: Use tighter pattern for matching expected output

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

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
Autobuild-User(master): Martin Schwenke <martins@samba.org>
Autobuild-Date(master): Mon May 15 08:08:10 CEST 2017 on sn-devel-144

6 years agoctdb-tests: Explicitly search for the specific log entry
Amitay Isaacs [Tue, 7 Mar 2017 05:52:26 +0000 (16:52 +1100)]
ctdb-tests: Explicitly search for the specific log entry

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

... instead of expecting the last line to be the matching log entry.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
6 years agos4: nmblookup: Allocate event context off NULL instead of talloc_autofree_context().
Jeremy Allison [Thu, 11 May 2017 23:15:56 +0000 (16:15 -0700)]
s4: nmblookup: Allocate event context off NULL instead of talloc_autofree_context().

It's already correctly freed on exit.

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): Sat May 13 21:01:25 CEST 2017 on sn-devel-144

6 years agos4: cifsdd: Allocate the event context off NULL, not talloc_autofree_context().
Jeremy Allison [Thu, 11 May 2017 23:10:31 +0000 (16:10 -0700)]
s4: cifsdd: Allocate the event context off NULL, not talloc_autofree_context().

Ensure it's freed on exit paths.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
6 years agos4: client: Allocate event context off struct smbclient_context *, not talloc_autofre...
Jeremy Allison [Thu, 11 May 2017 23:01:59 +0000 (16:01 -0700)]
s4: client: Allocate event context off struct smbclient_context *, not talloc_autofree_context().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
6 years agogensec: Add a TALLOC_CTX * to gensec_register().
Jeremy Allison [Thu, 11 May 2017 22:56:29 +0000 (15:56 -0700)]
gensec: Add a TALLOC_CTX * to gensec_register().

Pass in the TALLOC_CTX * from the module init to remove
another talloc_autofree_context() use.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
6 years agos4: ntvfs: Add a TALLOC_CTX * to sys_notify_register().
Jeremy Allison [Thu, 11 May 2017 22:50:08 +0000 (15:50 -0700)]
s4: ntvfs: Add a TALLOC_CTX * to sys_notify_register().

Pass in the TALLOC_CTX * from the module init to remove
another talloc_autofree_context() use.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
6 years agos4: ntvfs: Add a TALLOC_CTX * to sys_lease_register().
Jeremy Allison [Thu, 11 May 2017 22:49:28 +0000 (15:49 -0700)]
s4: ntvfs: Add a TALLOC_CTX * to sys_lease_register().

Pass in the TALLOC_CTX * from the module init to remove
another talloc_autofree_context() use.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
6 years agos4: ntvfs: Add a TALLOC_CTX * paramter to pvfs_acl_register()
Jeremy Allison [Thu, 11 May 2017 22:45:57 +0000 (15:45 -0700)]
s4: ntvfs: Add a TALLOC_CTX * paramter to pvfs_acl_register()

Make use of the passed in TALLOC_CTX * from the init function
to remove a talloc_autofree_context() use.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
6 years agos4: torture: Remove use of local variables that are simply mirroring popt_get_cmdline...
Jeremy Allison [Tue, 9 May 2017 23:28:53 +0000 (16:28 -0700)]
s4: torture: Remove use of local variables that are simply mirroring popt_get_cmdline_credentials().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Fri May 12 00:37:29 CEST 2017 on sn-devel-144

6 years agos4: popt: Change from talloc_autofree_context() to NULL context.
Jeremy Allison [Tue, 9 May 2017 23:20:07 +0000 (16:20 -0700)]
s4: popt: Change from talloc_autofree_context() to NULL context.

Call popt_free_cmdline_credentials() on successful exit from torture.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4: popt: Make cmdline_credentials static.
Jeremy Allison [Tue, 9 May 2017 23:14:03 +0000 (16:14 -0700)]
s4: popt: Make cmdline_credentials static.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4: popt: Global replace of cmdline_credentials -> popt_get_cmdline_credentials().
Jeremy Allison [Tue, 9 May 2017 23:10:03 +0000 (16:10 -0700)]
s4: popt: Global replace of cmdline_credentials -> popt_get_cmdline_credentials().

Add one use of popt_set_cmdline_credentials().
Fix 80 column limits when cmdline_credentials changes
to popt_get_cmdline_credentials().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4: popt: Add set/get/free functions for cmdline_credentials.
Jeremy Allison [Tue, 9 May 2017 21:34:35 +0000 (14:34 -0700)]
s4: popt: Add set/get/free functions for cmdline_credentials.

Preparing to make this static instead of a global.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4: Add TALLOC_CTX * to register_server_service().
Jeremy Allison [Tue, 9 May 2017 20:46:45 +0000 (13:46 -0700)]
s4: Add TALLOC_CTX * to register_server_service().

Use the passed in context from callers. Remove one
talloc_autofree_context().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4: auth: Remove a talloc_autofree_context() use.
Jeremy Allison [Tue, 9 May 2017 20:28:49 +0000 (13:28 -0700)]
s4: auth: Remove a talloc_autofree_context() use.

Use NULL here, not the autofree context for this
static pointer. The destructor prevents freeing this
memory anyway.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4: auth: Add TALLOC_CTX * to auth_register()
Jeremy Allison [Tue, 9 May 2017 19:39:14 +0000 (12:39 -0700)]
s4: auth: Add TALLOC_CTX * to auth_register()

Use the talloc context passed into all modules.
Remove one more talloc_autofree_context().

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agotestprogs: Add 'net rpc user' test against AD DC
Andreas Schneider [Tue, 2 May 2017 10:31:48 +0000 (12:31 +0200)]
testprogs: Add 'net rpc user' test against AD DC

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
6 years agodocs: Remove old docs.
Karolin Seeger [Fri, 5 May 2017 09:02:28 +0000 (11:02 +0200)]
docs: Remove old docs.

This does not yet remove the Makefile rules for the removed xml, someone
with better make-fu should step in here.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Thu May 11 19:37:37 CEST 2017 on sn-devel-144

6 years agodocs: Rename Samba3-Developers-Guide to Samba-Developers-Guide
Karolin Seeger [Fri, 5 May 2017 09:11:07 +0000 (11:11 +0200)]
docs: Rename Samba3-Developers-Guide to Samba-Developers-Guide

This guide is not obsolete but needs an update.

Signed-off-by: Karolin Seeger <kseeger@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Wed May 10 19:57:36 CEST 2017 on sn-devel-144

6 years agos3-lib/idmap_cache: remove some dead prototypes
Günther Deschner [Wed, 10 May 2017 08:05:59 +0000 (10:05 +0200)]
s3-lib/idmap_cache: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4-lib/policy: remove some dead prototypes
Günther Deschner [Wed, 10 May 2017 07:34:33 +0000 (09:34 +0200)]
s4-lib/policy: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4-libcli: remove some dead prototypes
Günther Deschner [Wed, 10 May 2017 07:33:47 +0000 (09:33 +0200)]
s4-libcli: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4-auth: remove some dead prototypes
Günther Deschner [Wed, 10 May 2017 07:32:06 +0000 (09:32 +0200)]
s4-auth: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos4-rpc_server: remove some dead prototypes
Günther Deschner [Wed, 10 May 2017 07:28:53 +0000 (09:28 +0200)]
s4-rpc_server: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-rpc_server: remove some dead prototypes
Günther Deschner [Wed, 10 May 2017 07:28:40 +0000 (09:28 +0200)]
s3-rpc_server: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-proto: remove some dead prototypes
Günther Deschner [Tue, 9 May 2017 09:19:00 +0000 (11:19 +0200)]
s3-proto: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-libsmb: remove some dead prototype
Günther Deschner [Fri, 5 May 2017 16:14:29 +0000 (18:14 +0200)]
s3-libsmb: remove some dead prototype

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-lib: remove some dead prototypes
Günther Deschner [Fri, 5 May 2017 16:12:56 +0000 (18:12 +0200)]
s3-lib: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-winbindd: remove some dead prototypes
Günther Deschner [Fri, 5 May 2017 16:11:28 +0000 (18:11 +0200)]
s3-winbindd: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-smbd: remove some dead prototypes
Günther Deschner [Fri, 5 May 2017 16:08:02 +0000 (18:08 +0200)]
s3-smbd: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agolib/krb5_samba: remove some dead prototypes
Günther Deschner [Fri, 5 May 2017 14:26:31 +0000 (16:26 +0200)]
lib/krb5_samba: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-passdb: remove some dead prototypes
Günther Deschner [Fri, 5 May 2017 13:37:31 +0000 (15:37 +0200)]
s3-passdb: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-auth: remove some dead prototypes
Günther Deschner [Fri, 5 May 2017 13:21:03 +0000 (15:21 +0200)]
s3-auth: remove some dead prototypes

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agosmbd/smb2_ioctl: instruct VFS to ignore locks for dup extents
David Disseldorp [Thu, 4 May 2017 13:09:36 +0000 (15:09 +0200)]
smbd/smb2_ioctl: instruct VFS to ignore locks for dup extents

As described in the previous commit - Windows Server 2016 (ReFS) ignores
locks for FSCTL_DUPLICATE_EXTENTS_TO_FILE. Do the same for Samba.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
Autobuild-User(master): Ralph Böhme <slow@samba.org>
Autobuild-Date(master): Wed May 10 12:27:00 CEST 2017 on sn-devel-144

6 years agovfs: add VFS_COPY_CHUNK_FL_IGNORE_LOCKS for dup extents
David Disseldorp [Thu, 4 May 2017 12:55:43 +0000 (14:55 +0200)]
vfs: add VFS_COPY_CHUNK_FL_IGNORE_LOCKS for dup extents

As confirmed by the Microsoft Protocol Open Specifications Team: Windows
Server 2016 (ReFS) ignores locks for FSCTL_DUPLICATE_EXTENTS_TO_FILE...

From: Jeff McCashland
To: David Disseldorp
CC: "cifs-protocol@lists.samba.org" <cifs-protocol@lists.samba.org>
Subject: RE: [116100414754619] FSCTL_DUPLICATE_EXTENTS_TO_FILE appears
 to completely bypass file locks
Date: Mon, 12 Dec 2016 20:44:08 +0000

Hi David,

We have made the following spec changes for the next doc release:

In section 2.1.5.9.4 FSCTL_DUPLICATE_EXTENTS_TO_FILE behavior notes have
been added to the following paragraphs.

Before:
§ The object store MUST check for byte range lock conflicts on
Open.Stream using the algorithm described in section 2.1.4.10 with
ByteOffset set to InputBuffer.TargetFileOffset, Length set to
InputBuffer.ByteCount, IsExclusive set to TRUE, LockIntent set to FALSE,
and Open set to Open. If a conflict is detected, the operation MUST be
failed with STATUS_FILE_LOCK_CONFLICT.

§ The object store MUST check for byte range lock conflicts on Source
using the algorithm described in section 2.1.4.10 with ByteOffset set to
InputBuffer.SourceFileOffset, Length set to InputBuffer.ByteCount,
IsExclusive set to FALSE, LockIntent set to FALSE, and Open set to
InputBuffer.FileHandle. If a conflict is detected, the operation MUST be
failed with STATUS_FILE_LOCK_CONFLICT.

After:
§ The object store SHOULD<WBN1> check for byte range lock conflicts on
Open.Stream using the algorithm described in section 2.1.4.10 with
ByteOffset set to InputBuffer.TargetFileOffset, Length set to
InputBuffer.ByteCount, IsExclusive set to TRUE, LockIntent set to FALSE,
and Open set to Open. If a conflict is detected, the operation MUST be
failed with STATUS_FILE_LOCK_CONFLICT.

§ The object store SHOULD<WBN2> check for byte range lock conflicts on
Source using the algorithm described in section 2.1.4.10 with ByteOffset
set to InputBuffer.SourceFileOffset, Length set to
InputBuffer.ByteCount, IsExclusive set to FALSE, LockIntent set to
FALSE, and Open set to InputBuffer.FileHandle. If a conflict is
detected, the operation MUST be failed with STATUS_FILE_LOCK_CONFLICT.

WBN1: The ReFS file system in Windows Server 2016 does not check for
byte range lock conflicts on Open.Stream.
WBN2: The ReFS file system in Windows Server 2016 does not check for
byte range lock conflicts on Source.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agosmbd/smb2_ioctl: validate dup_extent request lengths
David Disseldorp [Sat, 1 Oct 2016 21:40:13 +0000 (23:40 +0200)]
smbd/smb2_ioctl: validate dup_extent request lengths

FSCTL_DUPLICATE_EXTENTS_TO_FILE has some interesting behaviour when
exceeding the destination file length - the clone is truncated to only
cover the existing file region. If the existing length is zero, then
nothing is cloned.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agosmbd/smb2_ioctl: check sparseness for dup extents
David Disseldorp [Fri, 30 Sep 2016 18:53:07 +0000 (20:53 +0200)]
smbd/smb2_ioctl: check sparseness for dup extents

FSCTL_DUPLICATE_EXTENTS_TO_FILE should fail if the source is marked
sparse while the target is not:

From: Jeff McCashland
To: David Disseldorp
Subject: RE: FSCTL_DUPLICATE_EXTENTS_TO_FILE questions, 116092214702946
Date: Tue, 27 Dec 2016 18:06:14 +0000

...
We have updated the spec for future release:

Section 2.3.8 FSCTL_DUPLICATE_EXTENTS_TO_FILE Reply
Changed description of STATUS_NOT_SUPPORTED error code to:
"--The source and target destination ranges overlap on the same file.
--Source file is sparse, while -target is a non-sparse file.
--The source range is beyond the source file's allocation size."

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agovfs_btrfs: report FILE_SUPPORTS_BLOCK_REFCOUNTING capability
Aurelien Aptel [Thu, 30 Jul 2015 14:33:08 +0000 (16:33 +0200)]
vfs_btrfs: report FILE_SUPPORTS_BLOCK_REFCOUNTING capability

Signed-off-by: Aurelien Aptel <aaptel@suse.com>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agosmbd/smb2_ioctl: check for for overlap of dup extent ranges
David Disseldorp [Tue, 20 Sep 2016 16:48:37 +0000 (09:48 -0700)]
smbd/smb2_ioctl: check for for overlap of dup extent ranges

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agosmbd/smb2_ioctl: add support for FSCTL_DUPLICATE_EXTENTS_TO_FILE
Aurelien Aptel [Thu, 30 Jul 2015 14:16:15 +0000 (16:16 +0200)]
smbd/smb2_ioctl: add support for FSCTL_DUPLICATE_EXTENTS_TO_FILE

Signed-off-by: Aurelien Aptel <aaptel@suse.com>
Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
6 years agovfs: add parameter to copy chunk VFS function to handle dup_extents
Aurelien Aptel [Mon, 24 Aug 2015 16:22:56 +0000 (18:22 +0200)]
vfs: add parameter to copy chunk VFS function to handle dup_extents

FSCTL_DUPLICATE_EXTENTS_TO_FILE must be handled as a COW clone. Add a
copy-chunk flags parameter to the VFS to handle this.

Signed-off-by: Aurelien Aptel <aaptel@suse.com>
Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agotorture/ioctl: expect dup_extents(dest=compressed) to pass
David Disseldorp [Fri, 30 Sep 2016 19:35:09 +0000 (21:35 +0200)]
torture/ioctl: expect dup_extents(dest=compressed) to pass

The MS-FSCC spec doesn't mention anything about dup-extents against
compressed files.
This can't be tested against Windows, as ReFS doesn't support
compression, but COW clones of compressed files work on Btrfs.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agotorture/ioctl: fix dup_extents destination truncate
David Disseldorp [Wed, 3 May 2017 07:26:43 +0000 (09:26 +0200)]
torture/ioctl: fix dup_extents destination truncate

The dup_extents_compressed_dest test fails to correctly truncate the
dup_extents destination. Fix it.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
6 years agopackaging: Remove Mandrake
David Disseldorp [Tue, 9 May 2017 15:01:20 +0000 (17:01 +0200)]
packaging: Remove Mandrake

Mandriva was discontinued in 2011, and the link in the readme is dead.

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Wed May 10 07:35:20 CEST 2017 on sn-devel-144

6 years agopackaging: Remove SWAT references from Solaris
Andreas Schneider [Tue, 9 May 2017 14:33:49 +0000 (16:33 +0200)]
packaging: Remove SWAT references from Solaris

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
6 years agopackaging: Remove SWAT references from RHEL
Andreas Schneider [Tue, 9 May 2017 14:03:43 +0000 (16:03 +0200)]
packaging: Remove SWAT references from RHEL

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
6 years agopackaging: Remove SWAT references from RHEL-CTDB
Andreas Schneider [Tue, 9 May 2017 13:51:08 +0000 (15:51 +0200)]
packaging: Remove SWAT references from RHEL-CTDB

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
6 years agopackaging: Remove LSB packaging
Andreas Schneider [Tue, 9 May 2017 13:48:09 +0000 (15:48 +0200)]
packaging: Remove LSB packaging

This hasn't been touched since 2001.

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
6 years agopackaging: Remove setup script for SWAT
Andreas Schneider [Tue, 9 May 2017 13:47:19 +0000 (15:47 +0200)]
packaging: Remove setup script for SWAT

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
6 years agodbwrap_ctdb: Fix a typo
Volker Lendecke [Mon, 8 May 2017 10:16:16 +0000 (12:16 +0200)]
dbwrap_ctdb: Fix a typo

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed May 10 03:25:40 CEST 2017 on sn-devel-144

6 years agos3-gpo: Fix CID #1405972 Resource leak
Günther Deschner [Tue, 9 May 2017 12:31:15 +0000 (14:31 +0200)]
s3-gpo: Fix CID #1405972 Resource leak

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
6 years agos4:tls: Do not use deprecated GnuTLS types
Andreas Schneider [Tue, 9 May 2017 11:17:13 +0000 (13:17 +0200)]
s4:tls: Do not use deprecated GnuTLS types

Those have been deprecated with GnuTLS 1.0.20 in 2004. I think it is
safe to use them now ;)

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
6 years agos3-spoolss: add winreg_del_driver_package_internal()
Günther Deschner [Wed, 2 Nov 2016 17:17:58 +0000 (18:17 +0100)]
s3-spoolss: add winreg_del_driver_package_internal()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Günther Deschner <gd@samba.org>
Autobuild-Date(master): Tue May  9 20:44:17 CEST 2017 on sn-devel-144

6 years agos3-rpc_cli: add winreg_del_driver_package()
Günther Deschner [Wed, 2 Nov 2016 17:17:37 +0000 (18:17 +0100)]
s3-rpc_cli: add winreg_del_driver_package()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-spoolss: add winreg_get_driver_package_internal()
Günther Deschner [Wed, 2 Nov 2016 13:37:02 +0000 (14:37 +0100)]
s3-spoolss: add winreg_get_driver_package_internal()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-rpc_cli: add winreg_get_driver_package()
Günther Deschner [Wed, 2 Nov 2016 13:36:45 +0000 (14:36 +0100)]
s3-rpc_cli: add winreg_get_driver_package()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-spoolss: add winreg_add_driver_package_internal()
Günther Deschner [Wed, 2 Nov 2016 12:43:34 +0000 (13:43 +0100)]
s3-spoolss: add winreg_add_driver_package_internal()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-rpc_cli: add winreg_add_driver_package()
Günther Deschner [Wed, 2 Nov 2016 12:40:00 +0000 (13:40 +0100)]
s3-rpc_cli: add winreg_add_driver_package()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-spoolss: add winreg_add_core_driver_internal()
Günther Deschner [Mon, 31 Oct 2016 17:07:50 +0000 (18:07 +0100)]
s3-spoolss: add winreg_add_core_driver_internal()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-rpc_cli: add winreg_add_core_driver()
Günther Deschner [Mon, 31 Oct 2016 17:07:33 +0000 (18:07 +0100)]
s3-rpc_cli: add winreg_add_core_driver()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-spoolss: add winreg_get_core_driver_internal()
Günther Deschner [Mon, 31 Oct 2016 16:43:58 +0000 (17:43 +0100)]
s3-spoolss: add winreg_get_core_driver_internal()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-rpc_cli: add winreg_get_core_driver()
Günther Deschner [Mon, 31 Oct 2016 16:43:28 +0000 (17:43 +0100)]
s3-rpc_cli: add winreg_get_core_driver()

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3: popt: Change to NULL from talloc_autofree_context() now we correctly free on...
Jeremy Allison [Wed, 26 Apr 2017 00:09:18 +0000 (17:09 -0700)]
s3: popt: Change to NULL from talloc_autofree_context() now we correctly free on exit.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Tue May  9 14:00:40 CEST 2017 on sn-devel-144

6 years agos3: client tools: Call popt_free_cmdline_auth_info() on all normal exits.
Jeremy Allison [Wed, 26 Apr 2017 00:08:30 +0000 (17:08 -0700)]
s3: client tools: Call popt_free_cmdline_auth_info() on all normal exits.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3: client tools. Remove direct access to struct user_auth_info *cmdline_auth_info.
Jeremy Allison [Wed, 26 Apr 2017 00:03:10 +0000 (17:03 -0700)]
s3: client tools. Remove direct access to struct user_auth_info *cmdline_auth_info.

Only access through utility functions. Remove all the local pointer aliases
that were just being set to cmdline_auth_info in the client tools.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3: popt: Add utility functions popt_get_cmdline_auth_info(), popt_free_cmdline_auth_...
Jeremy Allison [Wed, 26 Apr 2017 00:01:25 +0000 (17:01 -0700)]
s3: popt: Add utility functions popt_get_cmdline_auth_info(), popt_free_cmdline_auth_info().

Leave the global cmdline_auth_info still exposed, we will make it static
once the users have been converted to the utility functions.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3: popt: When using a global variable, don't hide it by helper locals.
Jeremy Allison [Tue, 25 Apr 2017 23:32:05 +0000 (16:32 -0700)]
s3: popt: When using a global variable, don't hide it by helper locals.

That makes it very unclear when you're using a global.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
6 years agos3-gpo: Build scripts, security and registry CSE with --enable-developer
Günther Deschner [Thu, 4 May 2017 13:07:14 +0000 (15:07 +0200)]
s3-gpo: Build scripts, security and registry CSE with --enable-developer

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Tue May  9 05:18:52 CEST 2017 on sn-devel-144

6 years agos3-gpo: Fix build of scripts CSE
Günther Deschner [Thu, 4 May 2017 13:06:39 +0000 (15:06 +0200)]
s3-gpo: Fix build of scripts CSE

Guenther

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agoWHATSNEW: Add new 'net tdb locking' command
Christof Schmitt [Mon, 20 Feb 2017 18:52:58 +0000 (11:52 -0700)]
WHATSNEW: Add new 'net tdb locking' command

Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Christof Schmitt <cs@samba.org>
Autobuild-Date(master): Tue May  9 01:14:25 CEST 2017 on sn-devel-144

6 years agoselftest: Add test for 'net tdb' command
Christof Schmitt [Thu, 16 Feb 2017 23:23:47 +0000 (16:23 -0700)]
selftest: Add test for 'net tdb' command

Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agoselftest: Make lockdir available in test environment
Christof Schmitt [Thu, 16 Feb 2017 23:23:43 +0000 (16:23 -0700)]
selftest: Make lockdir available in test environment

Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agodocs-xml: Document net tdb command
Christof Schmitt [Thu, 16 Feb 2017 23:23:34 +0000 (16:23 -0700)]
docs-xml: Document net tdb command

Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agonet: Add net tdb command to print information from tdb records
Christof Schmitt [Thu, 16 Feb 2017 23:22:38 +0000 (16:22 -0700)]
net: Add net tdb command to print information from tdb records

The main purpose is to debug "hot" records from ctdb. ctdb tracks
contended records and identifies them by key in the dbstatistics:

DB Statistics: locking.tdb
[...]
 Num Hot Keys:     1
     Count:3 Key:6a4128e3ced4681b02a00000000000000000000000000000

This command allows querying additional information for the associated
key to identify the affected file. For now this only adds a subcommand
for the locking.tdb, but could be extended to others:

net tdb locking 6a4128e3ced4681b02a00000000000000000000000000000
Share path:            /test/share
Name:                  testfile
Number of share modes: 2

Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agoctdb: Print key as hex string instead of just the hash in hot record message
Christof Schmitt [Thu, 16 Feb 2017 23:23:39 +0000 (16:23 -0700)]
ctdb: Print key as hex string instead of just the hash in hot record message

Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
6 years agoidmap_rfc2307: Test unix-ids-to-sids with 35 groups
Volker Lendecke [Thu, 6 Apr 2017 10:50:08 +0000 (12:50 +0200)]
idmap_rfc2307: Test unix-ids-to-sids with 35 groups

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agoselftest: Avoid idmap caching when testing idmap_rfc2307
Volker Lendecke [Tue, 4 Apr 2017 15:15:10 +0000 (17:15 +0200)]
selftest: Avoid idmap caching when testing idmap_rfc2307

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agoidmap_rfc2307: "ldap_next_entry" needs the previous entry, not the start
Volker Lendecke [Fri, 31 Mar 2017 15:23:39 +0000 (15:23 +0000)]
idmap_rfc2307: "ldap_next_entry" needs the previous entry, not the start

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agoidmap_rfc2307: Don't stop after 30 entries
Volker Lendecke [Fri, 31 Mar 2017 15:20:07 +0000 (15:20 +0000)]
idmap_rfc2307: Don't stop after 30 entries

We start over again and again, so we need to search in the whole list.
This is a quick hack generating a bad O(n^2). The real fix is to
call idmap_rfc2307_find_map with "maps" starting at the right offset,
but that's an optimization for later when it's restructured

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agotest_idmap_rfc2307: Test wbinfo -r for 35 supplementary group memberships
Volker Lendecke [Tue, 4 Apr 2017 13:28:36 +0000 (15:28 +0200)]
test_idmap_rfc2307: Test wbinfo -r for 35 supplementary group memberships

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agotest_idmap_rfc2307: Do a recursive delete in ou=idmap
Volker Lendecke [Tue, 4 Apr 2017 13:12:02 +0000 (15:12 +0200)]
test_idmap_rfc2307: Do a recursive delete in ou=idmap

We'll create more posix objects soon

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agotest_idmap_rfc2307: Correct usage
Volker Lendecke [Tue, 4 Apr 2017 12:59:45 +0000 (14:59 +0200)]
test_idmap_rfc2307: Correct usage

We already have 13 args at this point, and growing

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agotest_idmap_rfc2307: Avoid a tmpfile
Volker Lendecke [Tue, 4 Apr 2017 12:15:26 +0000 (14:15 +0200)]
test_idmap_rfc2307: Avoid a tmpfile

We can << directly into ldbadd

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agotest_idmap_rfc2307: Remove the correct file
Volker Lendecke [Tue, 4 Apr 2017 12:15:26 +0000 (14:15 +0200)]
test_idmap_rfc2307: Remove the correct file

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agoidmap_rfc2307: "ldap_next_entry" needs the previous entry, not the start
Volker Lendecke [Fri, 31 Mar 2017 15:23:39 +0000 (15:23 +0000)]
idmap_rfc2307: "ldap_next_entry" needs the previous entry, not the start

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agoidmap_rfc2307: Don't stop after 30 entries
Volker Lendecke [Fri, 31 Mar 2017 15:20:07 +0000 (15:20 +0000)]
idmap_rfc2307: Don't stop after 30 entries

We start over again and again, so we need to search in the whole list.
This is a quick hack generating a bad O(n^2). The real fix is to
call idmap_rfc2307_find_map with "maps" starting at the right offset,
but that's an optimization for later when it's restructured

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

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
6 years agoctdb-logging: Initialize DEBUGLEVEL before changing the value
Amitay Isaacs [Fri, 5 May 2017 16:08:50 +0000 (02:08 +1000)]
ctdb-logging: Initialize DEBUGLEVEL before changing the value

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

DEBUGLEVEL is defined as a const array reference at compile time.
debug_init() converts that to an allocated array reference at run time.
Since debug_init() is a static function, initialize DEBUGLEVEL via
setup_logging().

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(master): Volker Lendecke <vl@samba.org>
Autobuild-Date(master): Mon May  8 00:12:25 CEST 2017 on sn-devel-144

6 years agos3/dbwrap_ctdb: free resources in an error code path
Ralph Boehme [Fri, 5 May 2017 20:31:54 +0000 (22:31 +0200)]
s3/dbwrap_ctdb: free resources in an error code path

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Ralph Böhme <slow@samba.org>
Autobuild-Date(master): Sun May  7 16:40:19 CEST 2017 on sn-devel-144

6 years agos3/dbwrap_ctdb: set async_ctx to initialized
Ralph Boehme [Fri, 5 May 2017 20:30:32 +0000 (22:30 +0200)]
s3/dbwrap_ctdb: set async_ctx to initialized

After initializing the async connection to ctdb we must set initialized
to true.

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
6 years agos3-rpcclient: Fix enumdata spoolss display of REG_DWORD
Günther Deschner [Wed, 3 May 2017 09:31:20 +0000 (11:31 +0200)]
s3-rpcclient: Fix enumdata spoolss display of REG_DWORD

One should not assume a REG_DWORD is always 4 byte long.

In an enumdata(ex) context, the first reply will return with content
in a buffer that can fill the largest entry in the array of attributes.
All subsequent queries then reuse this buffer even if the key/value
pair in question then is just a 4 byte DWORD. The remaining buffer
will be just empty and of no interest.

Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Sun May  7 03:28:21 CEST 2017 on sn-devel-144