s3:torture:delete: move the success message for a subtest to the correct place

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: remove an else, reducing indentation

s3:torture:delete: really fail the test in a failure case

s3:torture:delete: fix a comment

s3:torture:delete: fix a message

s3:torture:delete: fix a message

s3:torture:delete: fix a message

s3:torture:delete: fix a message (counting the opens)

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untanlge function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s3:torture:delete: untangle function call from result check

s4-ldb_wrap: Do not vasprintf() the ldb debug messages that will not be shown

This malloc() and free() actually shows up quite high on a call profile of
provision of the AD DC.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Aug  9 13:51:06 CEST 2012 on sn-devel-104

lib/ldb: Do not vasprintf() the tevent debug messages that will not be shown

This malloc() and free() actually shows up quite high on a call profile of
provision of the AD DC.

This allows the debug handler to decide if the argument list should be
printed.

Andrew Bartlett

s4-events: Do not vasprintf() the tevent debug messages that will not be shown

This malloc() and free() actually shows up quite high on a call profile of
provision of the AD DC.

Andrew Bartlett

s3-events: Do not vasprintf() the tevent debug messages that will not be shown

This malloc() and free() actually shows up quite high on a call profile of
provision of the AD DC (and this is the matching patch for source3).

Andrew Bartlett

lib/ldb: Use tdb_exists() rather than tdb_fetch()/talloc_free()

This avoids pulling the record and doing an allocation when we just
want to know if it exists.

Andrew Bartlett

s4-dsdb: Remove strcasecmp() fallback in replmd_ldb_message_element_attid_sort

In all callers, we must already have a attributeID for each of the
values or else we would have already given an error, or could not have
obtained the message over DRS.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Aug  9 11:39:54 CEST 2012 on sn-devel-104

s4-dsdb: Do not reload partition metadata except on transaction start

This ensures that we do not add objects that should go into a partition, but we
simply return that an object is not present if the connection was created
before the partition was loaded.  It is rare to create a new partition.

Andrew Bartlett

s3:smb2_sesssetup: setup global->[en|de]cryption_key

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Thu Aug  9 09:59:02 CEST 2012 on sn-devel-104

s3:smb2_read: don't try sendfile if encryption is used

metze

s3:smb2_server: add smbd_smb2_request->do_encryption

For now it's always false...

metze

s3:smb2_tcon: set global->encryption_required and enforce it

This the account or client doesn't support encryption we should
reject the tree connect.

metze

s3:smb2_sesssetup: set global->encryption_required and enforce it

This the account or client doesn't support encryption we should
reject the session setup.

metze

s3:smbXsrv.idl: add encryption_required to smbXsrv_tcon_global0

metze

s3:smb2_server: check the session before we could response with an error.

metze

s3:smb2_server: do central file_id check if the operation requires it

Note that it's fine to call file_fsp_smb2() twice, the 2nd call
just returns smb2req->compat_chain_fsp without a 2nd lookup.

metze

s4-dsdb: Ensure we have indexing enabled during the provision

Because we set the schema before we connected the ldb to a file, the @INDEX records
were not added until next startup.  This cost 100% more time in running provision on
my laptop.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Aug  9 08:20:36 CEST 2012 on sn-devel-104

s4-pydsdb: Provide control of if we should write index attributes when reloading a schema

This allows us to carefully control the loading of the schema.

Andrew Bartlett

s4-dsdb: Change talloc parent

This matches the rest of the function.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Aug  9 06:26:36 CEST 2012 on sn-devel-104

s4-dsdb: Remove ldb_sequence_type argument from partition_primary_sequence_number

We always want LDB_SEQ_HIGHEST_SEQ here.

Andrew Bartlett

s4-dsdb: simplify migration of old-style seqence numbers to metadata.tdb

This simple operation does not need to be encased in generic ldb extended operations.

Andrew Bartlett

s4-dsdb: Reduce calls to the ldb layer by reloading less often

We do not need to reload the partition list to get the global sequence
number, as that number is stored in the metadata.tdb, not the ldb files.

Andrew Bartlett

s3:nmbd: log a failure in get_domain_master_name_node_status_success() as level 1

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Wed Aug  8 20:14:29 CEST 2012 on sn-devel-104

s3:nmbd: don't log get_domain_master_name_node_status_fail at level 0

metze

s3:smb2_server: s/i/idx in smbd_smb2_request_pending_queue()

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Wed Aug  8 17:32:26 CEST 2012 on sn-devel-104

s3:smb2_server: make use of SMBD_SMB2_OUT_HDR_PTR() in smbd_smb2_request_pending_queue()

metze

s3:smb2_server: remove useless variable 'i'

metze

s3:smb2_server: rewrite dup_smb2_vec3() using SMBD_SMB2_*_IOV_OFS and helper variables

metze

s3:smb2_server: make use of SMBD_SMB2_* macros in smbd_smb2_request_done_ex()

metze

s3:smb2_server: make use of SMBD_SMB2_* macros in smbd_smb2_request_verify_sizes()

metze

s3:smb2_server: use the common buffer layout for smbd_smb2_request_pending*

metze

s3:smb2_read: use SMBD_SMB2_NUM_IOV_PER_REQ when checking for sendfile() support

metze

s3:smb2_read: fix indentation in schedule_smb2_sendfile_read()

metze

s3:smb2_server: don't try to update req->in.vector[0] in smbd_smb2_request_pending_queue()

req->in.vector[0] is reserved for the transport and might be removed in future.
This is currently always { NULL, 0 }, as it's not used, by our
transport layer code. The SMB2 layer should never touch this!

metze

selftest: Rename samba4.blackbox.pdbtest to samba.blackbox.pdbtest

This test covers s3dc as well.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Wed Aug  8 09:57:14 CEST 2012 on sn-devel-104

s3-torture: Extend pdbtest to also run an authentication unit-test

This tests both the builtin auth_sam against passdb directly and the
configured auth module.

Andrew Bartlett

build: Remove pdbtest from the autoconf build

pdbtest is internal test utility that is not installed.  It is only
run from the full (waf) make test, and does not need to be built in
the autoconf build.

Removing it from the autoconf build makes it easier to expand this
test to depend on more parts of Samba.

Andrew Bartlett

libcli/smb: do not set SMB2_TF_MSG_SIZE in the caller

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Wed Aug  8 07:32:55 CEST 2012 on sn-devel-104

libcli/smb: smb2_signing_[en|de]crypt_pdu() check and set SMB2_TF_MSG_SIZE

metze

s3:smb2_sesssetup: remove unused code in smbd_smb2_reauth_generic_return()

A reauth exchange is already signed, with the channel signing key.

metze

s3:smb2_sesssetup: remove TALLOC_FREE(session) from smbd_smb2_[re]auth_generic_return

The caller does this via the smbd_smb2_session_setup_state_destructor()

metze

s3:smb2_server: sign the last request at the start of smbd_smb2_request_reply()

This means we correctly sign all responses in a compound chain.

metze

s3: Fix a crash in reply_lockingX_error

A timed brlock with 2 locks comes in and the second one blocks,
file is closed. smbd_cancel_pending_lock_requests_by_fid sets
blr->fsp to NULL. reply_lockingX_error (called via
MSG_SMB_BLOCKING_LOCK_CANCEL) deferences blr->fsp because
blr->lock_num==1 (the second one blocked).

This patch fixes the bug by only undoing the locks if fsp!=NULL.
fsp==NULL is the close case where everything is undone anyway.

Thanks to Peter Somogyi, somogyi@hu.ibm.com for this bug report.

Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Aug  8 04:12:04 CEST 2012 on sn-devel-104

vfs_dirsort: Remove unnecessary return; statement

Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Wed Aug  8 02:17:13 CEST 2012 on sn-devel-104

vfs_afsacl.c: Remove some unnecessary return; statements

vfs_full_audit: Remove some unnecessary return; statements

vfs_time_audit: Remove unnecessary return; statement

vfs_time_audit: Remove some unnecessary return; statements

vfs-mediaharmony: Remove some unnecessary return; statements

vfs_media_harmony: fix return of void

caught by the Studio Compiler

Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Tue Aug  7 22:22:48 CEST 2012 on sn-devel-104

s3:smb2_server: fix SMB2 signing of compound responses

We need to defer the signing until we know the response
doesn't change anymore before it goes over the wire.

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Tue Aug  7 20:29:30 CEST 2012 on sn-devel-104

s3:smb2_server: there's no need to copy req->out.vector when we just keep the last request

metze

s3:smb2_server: use memmove instead of copying single vector elements

metze

s3:smb2_server: make use of SMBD_SMB2_OUT_HDR_PTR() smbd_smb2_request_pending_queue()

metze

s3:smb2_server: check for compound based on SMBD_SMB2_NUM_IOV_PER_REQ

metze

s3:smb2_server: make use of SMBD_SMB2_OUT_*_IOV smbd_smb2_request_reply()

metze

s3:smb2_server: check for compound based on SMBD_SMB2_NUM_IOV_PER_REQ

metze

s3:smb2_server: make use of SMBD_SMB2_*_IOV_OFS

metze

s3:smb2_server: make use of helper macros in smb2_calculate_credits()

metze

s3:smb2_server: make use of helper macros in smbd_smb2_request_validate()

metze

s3:smb2_server: make use of SMBD_SMB2_NUM_IOV_PER_REQ

metze

s3:smb2_server: add some more SMBD_SMB2_* defines/macros

metze

s3-param: Remove never-reached condition for popts == NULL

All the callers provide a parametric options pointer to fill in.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue Aug  7 17:16:38 CEST 2012 on sn-devel-104

s3-param: Remove never-reached condition for opt_list == NULL

All the callers provide a parametric options pointer to fill in.

Andrew Bartlett

source3/loadparm.c: Move string_set/string_free inside.

The only user, so make them static inside loadparm.c

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>

source3/smbd/conn.c: wean off string_set/string_free

Use straight talloc strings.  This is the only user outside loadparm.c.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>

source3/loadparm: make struct loadparm_service a talloc object.

This gives us a place to allocate members from.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>

loadparm: Add ctx member to struct loadparm_global.

Rather than tallocing global parameters off NULL, keep it neat by having
a Global.ctx member.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>

media_harmony VFS module: Add and build by default.

 - Supersedes previous patch.
 - Added various fixes for fake mtime functionality.
 - Now requires lp_cache_locked_write_times patch (bug 8912).
 - Removed various xattr functions to comply with recent VFS changes.
 - Changed SMB_STRUCT_DIR to DIR and SMB_STRUCT_DIRENT to struct dirent to comply with recent VFS changes.
 - Added manpage.
 - Added sample trigger_avid_update.py script.

Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Tue Aug  7 15:16:39 CEST 2012 on sn-devel-104

s3:smb2_server: do one central as_root check if the operation requires it

metze

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Tue Aug  7 13:14:38 CEST 2012 on sn-devel-104

s3:smb2_server: do one central tcon check if the operation requires it

metze

s3:smb2_server: do one central session check if the operation requires it

metze

s3:smb2_server: add and use smbd_smb2_call()

metze

s3:smb2_server: add .as_root to smbd_smb2_dispatch_table

metze

s3:smb2_server: add .need_tcon to smbd_smb2_dispatch_table

metze

s3:smb2_server: add .need_session to smbd_smb2_dispatch_table

metze

s3:smb2_server: introduce a smbd_smb2_dispatch_table (for now just with names)

metze