libcli: use LOCK_ORDER_1 for schannel_store.tdb

I.e. don't allow any other simultaneous locks.

Signed-off-by: Michael Adam <obnox@samba.org>

torture:smb2:durable-open: add test lock2-oplock

open with batch oplock. disconnect. reconnect durable.
take a BRL, unlock.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: use DBWRAP_LOCK_ORDER_VALID() in db_open()

instead of the hand written test.

Signed-off-by: Michael Adam <obnox@samba.org>

dbrwap: completely check validity of lock oder in dbwrap_check_lock_order()

This is currently not strictly necessay, because the
only caller catches the DBWRAP_LOCK_ORDER_NONE case,
and maximum is already checked,  but this seems too dangerous to me.

Use the new DBWRAP_LOCK_ORDER_VALID() macro.

Signed-off-by: Michael Adam <obnox@samba.org>
SQ: add min test (use the ..._VALID() macro)

dbwrap: add a comment explaining the supported lock orders

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: add DBWRAP_LOCK_ORDER_VALID()

Signed-off-by: Michael Adam <obnox@samba.org>

dbrwap: add DBWRAP_LOCK_ORDER_MIN

dbwrap: move definition of DBWRAP_LOCK_ORDER_MAX to the private header.

This is only needed internally.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: explicitly use DBWRAP_LOCK_ORDER_NONE in tdb->ntdb conversion

Make lack of lock order checking more visible.

Signed-off-by: Michael Adam <obnox@samba.org>

s4:cluster: use DBWRAP_LOCK_ODER_NONE for local_db_tmp_open()

Make lack of lock oder checking more visible.

Signed-off-by: Michael Adam <obnox@samba.org>

libcli: use DBWRAP_LOCK_ORDER_NONE when opening schannel_store.tdb

Make lack of lock order checking more visible.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: introduce DBWRAP_LOCK_ORDER_NONE for lock order "0".

Ther are still some databases which are opened (locally) with
lock oder 0, which means don't do lock oder checking,
thereby circumventing this deadlock-prevention mechanism.

Add a symbolic constant for this "0" to make this circumvention
more explicit.

Signed-off-by: Michael Adam <obnox@samba.org>

s3:rpc_client: optimize the netlogon_creds_cli.tdb for read-only access

Usually a record in this DB will be written once and then read
many times by winbindd processes on multiple nodes (when run in
a cluster). In order not to introduce a big performance penalty
with the increased correctness achieved by storing the netlogon
creds, in a cluster setup, we should activate ctdb's read only
record copies on this db.

Signed-off-by: Michael Adam <obnox@samba.org>

dbrwap: add DBRWAP_FLAG_OPTIMIZE_READONLY_ACCESS

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: add dbwrap_flags argument to dbwrap_local_open()

To be consistent with db_open() and prepare for future
possible extensions.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: add a dbwrap_flags argument to db_open_ntdb()

for consistency and to perpare for possible future
flags that the ntdb backend might be aware of.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: add a dbwrap_flags argument to db_open_tdb()

...for consistency and in preparation of future flags
that the tdb backend might be aware of.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: add a dbwrap_flags argument to db_open_ctdb()

This is in preparation of directly supporting ctdb read only
record copies when opening a ctdb database from samba.

Signed-off-by: Michael Adam <obnox@samba.org>

s3:registry: introduce REG_DBWRAP_FLAGS to use for all db_open calls

This is in accordance with the use of REG_TDB_FLAGS.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: add a dbwrap_flags argument to db_open()

This is in preparation to support handing flags to backends,
in particular activating read only record support for ctdb
databases. For a start, this does nothing but adding the
parameter, and all databases use DBWRAP_FLAG_NONE.

Signed-off-by: Michael Adam <obnox@samba.org>

dbwrap: add flags DBWRAP_FLAG_NONE

This is in preparation of adding a dbwrap_flags argument to db_open
and firends.

Signed-off-by: Michael Adam <obnox@samba.org>

s3:smbd: skip empty records in smbXsrv_open_cleanup()

This should avoid scary ndr_pull errors, if there's
a cleanup race.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Michael Adam <obnox@samba.org>

smbd:smb2: fix durable reconnect: set fsp->fnum from the smbXsrv_open->local_id

Originally, fsp->fnum was left at the INVALID fnum value.

Signed-off-by: Michael Adam <obnox@samba.org>

smbd:smb2: add SMB2_HDR_FLAG_REPLAY_OPERATION to the allowed flags

for some calls: create, read, write, ioct and set_info

More AD-voodoo (TM) by Metze  ;-)

witness: hack: make the principle name node-unique by adding a local IP

Only like this, the client sends notify requests after registering.

smbd: don't unconditionally start the witnessd.

Use "rpc_daemon:witnessd = fork" in the config instead.

TMP: witnessd get interface list from faktotum

srv_mgmt: construct princ_name like w2012

smbd:smb2: grant a persistent open without a batch oplock.

persistent handles on CA + scale-out shares are granted
without the request for batch oplocks (or handle leases).

smbd:smb2: treat persistent handles in durable reconnect v2

smbd:smb2: only honour the durable v2 timeout when persistent is requested

This is according to [MS-SMB2], 3.3.5.9.10.

HACK: set epmd rpc service mode to external

(needed for registering services since another hack commit
starts the external daemon unconditionally)

rpc: print error code in debug message when registering end point fails

TMP: add debug messages to dcerpc_ep

witness: fix copy and paste error in debug msg.

hack persistent:remove unused

smb2:persistent hack fix typo

smbd: announce caps - fix typos

HACK:smbd:smb2: fake support for persistent handles...

smbd:smb2: Add persistent flag to global open structure

adapt run_smbtorture_open_all.sh script for another cluster

Commit metze's AD voodoo

TEST Results

FIXUP: s4:torture:smb2: add test open-lease-all

s4:torture:smb2: add test open-lease-all

s4:torture:smb2: add test open-oplock-all

s4:torture:smb2: make test pattern tables static

s4:torture:smb2: display share capabilities

s4:torture:smb2: remove some unused labels

s4:libcli/smb2: honor option "client max protocol"

srvsvc.idl: add STYPE_CLUSTER_XXX

foo

Revert "s4:torture:smb2: durable_v2_open_oplock be verbose on failure"

This reverts commit 07dfb247be19b7fefd271d987932475e10ba903d.

Revert "s4:torture:smb2: durable_v2_open_lease be verbose on failure"

This reverts commit 6988302759855ab21dbfc6333983b0141f50081d.

s4:torture:smb2: durable_v2_open_lease be verbose on failure

s4:torture:smb2: durable_v2_open_oplock be verbose on failure

s4:torture:smb2: fix superfluous newline

run_smbtorture.sh

s4:torture:smb2:durable_v2 let print test_persistent_open_xxx if the share is CA

s4:torture:smb2:durable_v2 test_durable_v2_open_xxx_table do not stop at first failure

s4:torture:smb2:durable_v2 make open-lease more verbose

s4:torture:smb2:durable_v2 make open-oplock more verbose

rpcclient witness: use popt in Register()

rpcclient witness: print witness version in getInterfaceList()

FIXUP: rpcclient witness: RegisterEx()

rpcclient witness: RegisterEx()

WIP: add function AsyncNotify_Move()

WIP: add function AsyncNotify_Move()

WIP: add function AsyncNotify_Move()

librpc: add witness_RegisterEx for V2

rpcclient: add witness_notifyResponse_type's for V2

librpc: add witness_notifyResponse_type's for V2

FIXUP: librpc: enum witness_version

rpcclient witness: use WITNESS_V1

librpc: enum witness_version

rpcclient witness: print change notify

rpcclient witness: AsyncNotify()

rpcclient witness: hack to use only one rpc connection

rpcclient witness: fix warning

rpcclient witness: WITNESS_VERSION

rpcclient witness: UnRegister()

rpcclient witness: Register()

rpcclient witness: GetInterfaceList fix win_errstr

rpcclient witness: GetInterfaceList

torture:winreg: enable sd testing

rpcclient witness: check ntstatus and werror

WIP: witness rpcclient

librpc: enums and bitfields for witness protocol

TODO witness

TODO librpc/idl: add basic witness.idl

See [MS-SWN] for the details.

TODO smb2_tcon: add some share flags/capabilities options (Default:off)

TODO: s3:smb2_negprot: announce persistent handle support (default: no)

add "smbd:announce persistent handles" option (default: false)

metze

samba-tool classicupgrade: Remove unsued upgrade_smbconf

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-By: Jelmer Vernooij <jelmer@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Jan 23 23:51:56 CET 2014 on sn-devel-104

samba-tool classicupgrade: Remove unsued reference to samba3sam

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-By: Jelmer Vernooij <jelmer@samba.org>

s3: Don't open the dir for stat/attr/acl related mask

Signed-off-by: Abhidnya Joshi <achirmul@in.ibm.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jan 23 03:56:35 CET 2014 on sn-devel-104

s3: ldap client can return NT_STATUS_OK when an error occurs in a paged search.

"Inside ads_do_search_all_args(), if the first call to ads_do_paged_search_args()
fails, the proper error status is returned.

But, if the execution is already inside the loop to get all the accounts doing
several calls to ads_do_paged_search_args(), and one of these calls times out,
the status returned is from the *first* call, so success. This causes
net_ads_search() to interpret the return from ads_do_search_retry() as
success and print all the accounts returned, but it’s only a subset."

Also ensure we free previously returned results on error
in subsequent fetches.

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

Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jan 23 01:40:54 CET 2014 on sn-devel-104

log winbind version (when requested) in winbindd log

winbindd currently only logs the INTERFACE version request,
it would be useful to additionally have the version returned
in the log also.

Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: Jim McDonough <jmcd@samba.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Autobuild-User(master): David Disseldorp <ddiss@samba.org>
Autobuild-Date(master): Wed Jan 22 21:57:04 CET 2014 on sn-devel-104

s4:torture/rpc: add invalidAuthenticate2

This add 'rpc.netlogon.netlogon.invalidAuthenticate2' as new test
it demonstrates the STATUS_BUFFER_OVERFLOW on computer names
larger than 15 characters.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Wed Jan 22 19:07:12 CET 2014 on sn-devel-104

s4:rpc_server/netlogon: return a zero return_authenticator and rid on error

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>