s4-rpc: paranoid check for auth_length

This is not strictly needed as the ndr_pull_advance() checks it a few
lines further down, but I want to save Jeremy getting more grey hairs :-)

testprogs: add rather simple device mode tests to spoolss test.

Guenther

s3: Fix timeout calculation if g_lock_lock is given a timeout < 60s

Detected while showing this code to obnox :-)

s3: Slightly increase parallelism in g_lock

There's no need to still hold the g_lock tdb-level lock while telling the
waiters to retry

s3: Avoid starving locks when many processes die at the same time

In g_lock_unlock we have a little race between the process_exists and
messaging_send call: We only send to 5 waiters now, they all might have died
between us checking their existence and sending the message. This change makes
g_lock_lock retry at least once every minute.

s3: Avoid a thundering herd in g_lock_unlock

Only notify the first 5 pending lock waiters. This avoids a thundering herd
problem that is really nasty in a cluster. It also makes acquiring a lock a bit
more FIFO, lock waiters are added to the end of the array.

s3: Optimize g_lock_lock for a heavily contended case

Only check the existence of the lock owner in g_lock_parse, check the rest of
the records only when we got the lock successfully. This reduces the load on
process_exists which can involve a network roundtrip in the clustered case.

s3: Fix handling of processes that died in g_lock

g_lock_parse might have thrown away entries from the locks array because the
processes were not around anymore. Don't store the orphaned entries.

s4-kcc: remove a qsort() that snuck into the new topology code

s4-rpc: don't use auth padding in rpc bind requests as it breaks s3

The s3 RPC server returns a bind_nak if it gets a rpc bind with auth
padding. This change forces a padding length of zero to maximimise
compatibility with s3 servers.

I've left the padding code in as a #if 0 to make it easier for us to
test/fix the s3 server code, which should be changed to correctly
handle arbitrary auth padding in all rpc requests with auth trailers.

s4-dcerpc: fixed auth padding to be relative to the stub, not packet

The recent dcerpc padding changes made our padding relative to the
packet header, instead of the start of the stub. Surprisingly, this
broke w2k8r2 doing a dcpromo join to a s4 server. It seems that w2k8r2
is very fussy about the padding it gets in some circumstances.

s4-dsdb: return LDB_ERR_CONSTRAINT_VIOLATION on num_recs != 1

In a single record search, LDB_ERR_CONSTRAINT_VIOLATION is more useful
than the generic LDB_ERR_OPERATIONS_ERROR

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-samdb: use dsdb_search() in cracknames

greatly simplifies some of the cracknames code

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-kcc: remove search_onelevel_with_deleted() in kcc

Use dsdb_search() instead

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-dsdb: removed gendb_search_single_extended_dn()

Use dsdb_search_one() instead, which allows for arbitrary controls

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-dsdb: added dsdb_search_one() and cleanup dsdb_find_dn_by_guid()

dsdb_find_dn_by_guid() now takes a struct GUID instead of a
guid_string. All the callers in fact wanted a struct GUID, so we now
avoid the extra conversion.

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-dsdb: replace dsdb_find_dn_by_guid() with a dsdb_search() call

much simpler code by using dsdb_flags

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-dsdb: change dsdb_search_dn_with_deleted() to dsdb_search_dn() with dsdb_flags

Allows for arbitrary controls

s4-dsdb: change samdb_replace() to dsdb_replace() and allow for dsdb_flags

This allows for controls to be added easily where they are needed.

s4-dsdb: replace dsdb_modify_permissive() with dsdb_modify() and dsdb_flags

s4-dsdb: move dsdb_request_add_controls() into dsdb/common/util.c

This will be used to allow the flag based ldb functions to work on
both a ldb or a module, thus saving a lot of specialist functions.

s3-docs: Document 'smbclient -C'.

s4-smbtorture: fix spoolss SD test against w2k8r2.

Guenther

s4-smbtorture: more work on test_PrinterInfo_DevMode RPC-SPOOLSS-PRINTER.

Guenther

s4-smbtorture: allow to call OpenPrinterEx with devicemode.

Guenther

testprogs: add "samba3" and "architecture=" options to spoolss tester.

Guenther

s4-drs: DsGetReplInfo() refactoring

s4-drs: DsReplGetInfo() for DS_REPL_INFO_REPSTO infoType

Implements the DS_REPL_INFO_REPSTO infoType of DsReplGetInfo().

s4-smbtorture: more work on SD tests for RPC-SPOOLSS.

Guenther

s4-smbtorture: add rather simple spoolss device mode test.

Guenther

s4-smbtorture: use test_SetPrinter call in spoolss security descriptor tests.

Guenther

s4-smbtorture: w2k8r2 allows to add printers w/o defining printprocessor.

Guenther

testprogs: fix EnumJobs debug output in spoolss test.

Found by Kai, thanks!

Guenther

s3: Fix a typo

s4-rootdse: we don't need DSDB_FLAG_OWN_MODULE here

s4-drs: enable the recyclebin optional feature

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4-kcc: remove C++ comment

s4-kcc: initial code for the topology algorithm

this patch contains the data structures declarations and the functions to
setup the graph.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4-dsdb: don't change replPropertyMetaData if the value hasn't changed

When updating replPropertyMetaData, check if the value being stored is
the same as the current value, and skip the update if it is.

This is based on a patch by Fernando J V da Silva <fernandojvsilva@yahoo.com.br>

s4-heimdal: Fix typos in comment.

Karolin

s4-test: use local ldb for urgent_replication test

This test only currently works on a local ldb, as it needs system
access.

s4-ldb: fixed permissions on urgent_replication.py

s4-test: minor fixes to urgent_replication.py

- fix usage name
- remove unnecessary python functions
- remote unused gc ldb

s4-drs: Test situations for Urgent Replication

Checks if the partition's uSNUrgent is updated or not, depending
on the class of the object which is created, modified or deleted.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4-drs: Function for accessing dsdb_load_partition_usn from Python

Returns both uSNHighest and uSNUrgent as a dict object in Python
from a given partition dn.

Signed-off-by: Andrew Tridgell <tridge@samba.org>

s4-drs: Fixes bugs regarding Urgent Replication on wrong situations

It fixes the bug which causes an urgent replication to be enabled
incorrectly when an object is modified, but it should happen only
when it was created. This patch also fixes the bug that enable an
urgent replication when an object is deleted, but it should happen
only when it was modified and fixes the bug that does not enable
an urgent replication when an object is deleted and it should happen
only when it is deleted (not when it is modified).

Signed-off-by: Andrew Tridgell <tridge@samba.org>

spoolss: disable GetPrinterDriver level 101 (as called by XP).

We still dont get the marshalling right, disable and XP will just fall back to
level 6.

Guenther

s4-smbtorture: add some security descriptor tests to RPC-SPOOLSS and RPC-SPOOLSS-PRINTER.

Guenther

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