torture: Reproducer for 64c0367

Signed-off-by: Jeremy Allison <jra@samba.org>

tdb/test: fix build on OSF/1

Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Fri Aug 10 23:33:20 CEST 2012 on sn-devel-104

Revert "s3:smbd: include smbXsrv.h before smbd/proto.h to have the smbXsrv_ structs available"

This reverts commit 98ccca8dca70b87d04a93c8ef5232a071ab7c2af.

Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Fri Aug 10 17:35:38 CEST 2012 on sn-devel-104

Revert "s3:smbd: Include smbXsrv.h before vfs.h (in smbd.h) so that the smbXsrv structures are available"

This reverts commit e332bfaff51e54638bd37cd1fe08e57608e16e86.

Revert "s3:smb: include "smbXsrv.h" before "vfs.h""

This reverts commit db0c233624e633b3cc1a6e0e44dccc09aaa121f2.

s3:smbXsrv.idl: remove smbXsrv_*0 defines

This makes ctags more usable.

metze

s3:param: fix compiler warnings with FN_GLOBAL_CONST_STRING()

metze

build: rename security → samba-security

there is a libsecurity on OSF1 which clasheѕ with our security lib. see bug #9023.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User(master): Björn Jacke <bj@sernet.de>
Autobuild-Date(master): Fri Aug 10 14:22:21 CEST 2012 on sn-devel-104

nsswitch: add ABI checking and symbol versions to libwbclient

This will ensure that we do not unintentionally break the ABI.

Andrew Bartlett

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Fri Aug 10 04:08:54 CEST 2012 on sn-devel-104

s4-dsdb: Explain better what records are written during schema set

This is controlled by setting write_indices_and_attributes.

Andrew Bartlett

lib/ldb: Use tdb_parse_record and a callback rather than tdb_fetch()

This avoid allocation at the tdb layer as we will allocate this
with talloc right away anyway.

Andrew Bartlett

Remove smb_panic() from unix_strlower(). Just rely on error code return.

Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Aug  9 23:52:53 CEST 2012 on sn-devel-104

Correctly check for errors in strlower_m() returns.

Fix strlower_m() to return an error indication.

Check error returns on strnorm().

Check error returns from strupper_m() (in all reasonable places).

Fix missing ads_destroy in error path.

Change strupper_m() to return a value.

Fix bad return in unix_strupper.

Prepare to remove smb_panic() from unix_strlower().

Fix bad return values in unix_strlower/unix_strupper.

s4:torture:basic: check the return status of the last open in deltest16

Autobuild-User(master): Michael Adam <obnox@samba.org>
Autobuild-Date(master): Thu Aug  9 18:01:50 CEST 2012 on sn-devel-104

s4:torture:basic: fix a message typo in the delete17 test

s4:torture:basic: fix abundance of spaces in deltest6

s4:torture:basic:delete: fix 4 vs 8 spc tab formatting in check_delete_on_close()

s3:torture:delete: add a 12th subtest to the delete-on-close tests

test whether second open is possible with initial delete on close
and how setting and unsetting delete on close on the handle affects
the initial delete on close (it does not...)

s3:torture:delete: fix 11th test to work against windows

s3:torture:delete: simplify return code handling, fixing a couple of return codes in error cases

s3:torture:delete: reduce indentation

s3:torture:delete: add a comment

s3:torture:delete: add a comment

s3:torture:delete: add a comment

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