s4:provision.c - "provsion_store_self_join" - don't set the realm if it's NULL

The realm can be NULL when this function is called by "libnet_Join_primary_domain".
This seems to be a valid value since "libnet_JoinDomain" is allowed to return an
empty realm if we are not in a ADS domain.

s4-drs: replace manual checks with dsdb_modify_permissive()

Much simpler to use the permissive control instead of manually munging
the SPN list.

s4-drs: use a permissive modify in addentry

It is not an error if entries already exist.

s4-dsdb: added dsdb_modify_permissive()

This will be used in the drsuapi server

s3: eventlogadm.c: add -s option to use alternative config file.

Update the manpage accordingly.

s3: Fix initgroups return check

A return code of 1 from initgroups() is OK since apparently it means
the gid has already been set. The man page doesn't mention this.

s4:dcesrv_lsa.c - remove a superfluous empty line

One empty line is enough for code part divisions.

s4-rpcserver: teach the rpc server to cope with bad sig_size estimates

a4-dcerpc: another attempt at dcerpc auth padding

The last change broke net vampire against w2k8r2

util: update three other copies of our dlinklist.h macros

ldb and tevent have their own copies of these macros. This brings them
in sync with the master copy of dlinklist.h

s3: last part of TYPESAFE_QSORT() conversion

convert smbcacls, sharesec and web/

s3-smbd: convert lanman and notify code to TYPESAFE_QSORT()

s3-rpc: convert wkssvc to use TYPESAFE_QSORT()

s3-locking: convert brlock to TYPESAFE_QSORT()

s3: convert registry and printing code to TYPESAFE_QSORT()

s3-nmbd: note TODO item for qsort

This uses another char* cast hack. Left alone for now.

s3-vfs: use TYPESAFE_QSORT() in s3 VFS modules

s3-lib: use TYPESAFE_QSORT() in remaining s3 library code

the sort_query_replies() in nmblib.c is a TODO. It uses a hack that
treats a char* as a structure. I've left that one alone for now.

s3-libsmb: use TYPESAFE_QSORT() in namequery code

This one was a bit trickier. I'd appreciate it if someone else can
look over this.

s3-lib: use TYPESAFE_QSORT() in s3 interfaces code

lib: use TYPESAFE_QSORT() in lib/ and libcli/

s3-includes: enable TYPESAFE_QSORT() in s3

s4-ldb: use TYPESAFE_QSORT() in the rest of the ldb code

s4:loadparm.c - prevent memory leaks

If a "logfile" was already set, free the content up before setting a new
location. This can happen on a loadparm reload.

s4:loadparm.c - the logfile parameter value needs to be duplicated

Otherwise the "logfile" pointer tracks all changes of "pszParmValue" which
content is only temporal. This was the cause of bug #6212.

s4:popt_common.c - fix intendation

s4:util.c - Corrected the location of the "Directory Service" object

I wonder why nobody noticed this since for sure this "tombstone" functionality
was broken till now.

s4:server.c - use always "return" instead of "exit" in the "binary_smbd_main" function

Just to be consistent - no functional change

libwbclient: Fix s4 build failure by fixing up includes.

Sorry about this, folks.

s3: Fix Coverity ID 629: DEADCODE

We have already returned if (argc < 1) above

s3: Fix bug 5198 -- parse chfn(1)-change gecos field

s3: Fix typo found by jhell <jhell@DataIX.net>

libwbclient: Separate out the async functions

s3: Remove unused comparison fn from "struct sorted_tree"

s3: Make adt_tree data definitions private to adt_tree.c

s3: SORTED_TREE -> struct sorted_tree

s3: TREE_NODE -> struct tree_node

s3: Fix some nonempty blank lines

use ZERO_STRUCT

Use ZERO_STRUCTP

s3: Fix a typo

s3: Fix a C++ warning

s4-smbd: fix crash in notify code on client termination

We need to free the lp_ctx after we free the event context, otherwise
the teardown code in the notify backend dies when it tries to use the
iconv_convenience ptr

Fixes bug 7053

s4-rpc: be more careful about DCERPC auth padding

Cope with a wider range of auth padding in dcerpc bind_ack and
alter_context packets. We now use a helper function that calculates
the right auth padding.

s4: use LDB_TYPESAFE_QSORT() instead of ldb_qsort()

s4-ldb: added LDB_TYPESAFE_QSORT()

Like TYPESAFE_QSORT() but for the ldb_qsort() function

s4-wrepl: use TYPESAFE_QSORT() in wins repl code

s4-torture: use TYPESAFE_QSORT() in smbtorture

s4-rpcserver: use TYPESAFE_QSORT() in rpc servers

s4-ntvfs: use TYPESAFE_QSORT() in notify code

s4-socket: use TYPESAFE_QSORT() in netif code

s4-auth: use TYPESAFE_QSORT() in gensec

s4-dsdb: use TYPESAFE_QSORT() in dsdb code

tdb: use fdatasync() instead of fsync() in transactions

This might help on some filesystems

libreplace: add fdatasync() if not available

util: added TYPESAFE_QSORT() macro

This makes it much harder to get the type of a qsort comparison
function wrong.

tdb: Apply some const, just for clarity

Use sec_initial_uid() in the places where being root doesn't matter,
and 0 in the places where it does.

Jeremy

Simplify the logic in make_connection_snum(), and make it match Windows behavior.

Cause all exit paths to go through one place, where all cleanup is
done. change_to_root_user() for pathname operations that should succeed if
the path exists, even if the connecting user has no access.

For example, a share can now be defined with a path of /root/only/access
(where /root/only/access is a directory path with all components only
accessible to root e.g. root owned, permissions 700 on every component).
Non-root users will now correctly connect, but get ACCESS_DENIED on
all activities (which matches Windows behavior). Previously, non-root
users would get NT_STATUS_BAD_NETWORK_NAME on doing a TConX to this
share, even though it's a perfectly valid share path (just not accessible
to them).

This change was inspired by the research I did for bug #7126, which
was reported by bepi@adria.it.

As this is a change in a core function, I'm proposing to leave
this only in master for 3.6.0, not back-port to any existing releases.
This should give us enough time to decide if this is the way we want this to
behave (as Windows) or if we prefer the previous behavior.

Jeremy.

Fix warning messages on compile in g_lock.c Volker & Michael please check.

Jeremy.

Fix error in docs. Should be milliseconds, not microseconds.

s3/smbd: change locking behavior when "lock spin time = 0".

The "lock spin time" parameter mimics the following Windows
setting which by default is 250ms in Windows and 200ms in Samba.

HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\LanmanServer\Parameters\LockViolationDelay

When a client sends repeated, non-blocking, contending BRL requests
to a Windows server, after the first Windows starts treating these
requests as timed blocking locks with the above timeout.

As an efficiency, I've changed the behavior when this setting is 0,
to skip this logic and treat all requests as non-blocking locks.

This gives the smbd server behavior similar to the 3.0 release with
the do_spin_lock() implementation.

I've also changed the blocking lock parameter in the call from
push_blocking_lock_request() to true as all requests made in this
path are blocking by definition.

Remove #if SOFTLINK_OPTIMIZATION code.

This hasn't been turned on or been capable of doing so for
many years now. Makes this jumbo function smaller...
Jeremy.

Revert "Fix bug #7126 - [SMBD] With access denied error smbd return wrong NT_STATUS_OBJECT_PATH_INVALID error"

This reverts commit 2fdd8b10c6abadd27c579e772c0482214d2363a5.

This fix is incorrect. The original code works as desired,
I made a mistake here.

Jeremy.

Fix bug #7126 - [SMBD] With access denied error smbd return wrong NT_STATUS_OBJECT_PATH_INVALID error

As tridge's comment says, we should be ignoring ACCESS_DENIED
on the share path in a TconX call, instead allowing the mount
and having individual SMB calls fail (as Windows does). The
original code erroneously caught SMB_VFS_STAT != 0 and errored
out on that.

Jeremy.

packaging(RHEL-CTDB): adapt configure.rpm to match the spec-file configure call

Michael

v3-4-ctdb: Do not do any logrotation

s3:registry: eliminate race condition in creating/scanning sorted subkeys

Called, from key_exists, scan_sorted_subkeys re-creates the sorted
subkeys record of the given key and then searches through it.
The race is that between creation and parsing of the sorted subkey
record, another process that stores some other subkey of the same
parent key will delete the sorted subkey record, resulting in an
WERR_BADFILE of an operation that should actually succeed.

This patch fixes the issue by wrapping the creation and parsing
into a transaction.

Michael

s3:make "net conf addshare" atomic by wrapping all writes in one transaction

Michael

s3:g_lock: remove a nested event loop, replacing the inner loop by select

This made smbd crash in g_lock_lock() when trying to start a
transaction on a db with an already started transaction,
e.g. in a tcon_and_X where the share_info.tdb was not yet
initialized but share_info.tdb was already locked by another
process or writing acces to the winreg rpc pipe where the
registry tdb was already locked by another process.

What we really _want_ to do here by design is to react to
MSG_DBWRAP_G_LOCK_RETRY messages that are either sent
by a client doing g_lock_unlock or by ourselves when
we receive a CTDB_SRVID_SAMBA_NOTIFY or
CTDB_SRVID_RECONFIGURE message from ctdbd, i.e. when
either a client holding a lock or a complete node
has died.

Doing this properly involves calling tevent_loop_once(),
but doing this here with the main ctdbd messaging context
creates a nested event loop when g_lock_lock() is called
from the main event loop.

So as a quick fix, we act a little corasely here: we do
a select on the ctdb connection fd and when it is readable
or we get EINTR, then we retry without actually parsing
any ctdb packages or dispatching messages. This means that
we retry more often than necessary and intended by design,
but this does not harm and it is unobtrusive. When we have
finished, the main loop will pick up all the messages and
ctdb packets. The only extra twist is that we cannot use
timed events here but have to handcode a timeout for select.

Michael

s3:ctdb_conn: add ctdbd_conn_get_fd() to get the fd out of the ctdb connection

Michael

s3:g_lock: remove an unreached code path.

Michael

s3:dbwrap_ctdb: fix reading/storing of special key __db_sequence_number__

The key for reading and writing was inconsistent due to a
off by one data length.

Michael

s3:dbwrap_ctdb: exit early when nothing has been written in transaction_commit.

This skips update of the __db_sequence_number__ record when nothing else has
been written. There are transactions that are just openend and then nothing
is written until transaction_commit is called. This is for instance the case
with registry initialization routines: They start a transaction and only
write somthing when the registry has not been initialized yet.
So this change will skip many db_seqnum bumps and TRANS3_COMMIT roundtrips.

Michael

s3:dbwrap_ctdb: fix brown paperbag bug in ctdb_transaction_commit.

I carefully prepared the return value only to "return 0;" at the bottom. :-(
This may well have hit us for instance in the nested cancel case
and produced random errors.

Michael

s3:dbwrap_ctdb: fix logic error in pull_newest_from_marshall_buffer().

The logic bug was that if a record was found in the marshall buffer,
then always the ctdb header of tha last record in the marshall buffer
was returned, and not the ctdb header of the last occurrence of the
requested record.

This is fixed by introducing an additional temporary variable.

Michael

s3:dbwrap_ctdb: fix an uninitialized variable.

Michael

s3:dbwrap_ctdb: fix two "may be used uninitialized" warnings

Michael

s3:dbwrap_ctdb: fix db_ctdb_fetch_db_seqnum_from_db() when NT_STATUS_NOT_FOUND.

Don't treat this as an error but return seqnum 0 instead.

Michael

s3:dbwrap: If "-n" is given to dbwrap_torture, open db with CLEAR_IF_FIRST

s3:build: remove checks for deprecated ctdb controls.

Michael

s3:dbwrap_ctdb: maintain a database sequence number that bumps in transactions

For persistent databases, 64bit integer is kept in a special record
__db_sequence_number__. This record is incremented with each completed
transaction.

The retry mechanism for failing TRANS3_COMMIT controls inside the
db_ctdb_transaction_commit() function now relies one a modified
behaviour of ctdbd's treatment of persistent databases in recoveries.
Recently, a special treatment for persistent databases had been
introduced in ctdb (1.0.108) to work around the problems with the
orinal design of persistent transactions.
Now with the rewrite we need to revert to the old behaviour that
ctdb always takes the newest copies of all records.

This change also paves the way for a next step, which will make
recovery use the db seqnum to tell which node has the newest copy
of a persistent db and use that node's copy. This will greatly
reduce the amount of data transferred with each recovery.

Michael

s3:dbwrap_ctdb: change db_ctdb_transaction_store() to return NTSTATUS.

The return values calculated by the callers were wrong anyways since
the new marshalling code does not set the local tdbs tdb error code.

Michael

s3:dbwrap_ctdb: update (C)

Michael

build: Add a configure check for CTDB_CONTROL_TRANS3_COMMIT.

This is the new implementation of ctdb transactions using the
global lock feature. It is needed by the current dbwrap_ctdb code.

Michael

s3: Add tdb_data_equal

s3:torture: add a test LOCAL-DBTRANS to torture dbwrap with transactions.

s3:dbwrap_ctdb: start rewrite of transactions using the global lock (g_lock)

This simplifies the transaction code a lot:

* transaction_start essentially consists of acquiring a global lock.

* No write operations at all are performed on the local database
  until the transaction is committed: Every store operation is just
  going into the marshall buffer.

* The commit operation calls a new simplified TRANS3_COMMIT control
  in ctdb which rolls out thae changes to all nodes including the
  node that is performing the transaction.

Michael

s3: setup debug for smbtorture

s3: Add ctdb_conn_msg_ctx()

s3: Implement global locks in a g_lock tdb

This is the basis to implement global locks in ctdb without depending on a
shared file system. The initial goal is to make ctdb persistent transactions
deterministic without too many timeouts.

s4:kdc Fill in created_by principal field

s4:kdc Fix double free and uninitialized memory.

In samba_kdc_trust_message2entry() on error, hdb_free_entry()
may end up trying to access uninitialized memory or double
free the hdb_entry.

testprogs: print all printerinfo levels in spoolss test.

Guenther

s4:auth/credentials/credentials.c - Initialise the "lm_response" and "nt_response" structures

In some cases those structures are not initialised and the whole authentication
system crashes with a SIGSEGV. Bug discovered by Matthieu Patou in bug #6755.

s4:getopt.py - set the password callback only when no password has been provided

Previously the "no_pass" and "no_pass2" variables weren't handled correctly.
Since at the initialisation of the "CredentialsOptions" we don't have any
password at all. Only afterwards we could get one through "set_password".

If a password is specified, use it. If no password is specified, consider the
use fo an input mask on STDOUT. But if the loadparm context contains one prefer
it over the input.

s4:net_drs_bind: make some more stuff static

metze

s4:net/drs: use the full path to "utils/net/drs/net_drs*.h"

This fixes the build with automatic dependecies.

metze

s3: notify_onelevel does not use seqnums, so don't open asking for it

spoolss: regenerate hand-marshalled ndr_pull_spoolss_DriverInfo101 after ndr64 changes.

Guenther