s3-spoolss: fix some crash bugs and missing error codes in AddDriver paths.

Found by torture test.

Guenther

s4-smbtorture: test both 32bit and 64bit driver uploads in RPC-SPOOLSS-DRIVER.

Guenther

s4-smbtorture: allow to upload and later remove printer driver files in RPC-SPOOLSS-DRIVER.

Guenther

s4-smbtorture: add RPC-SPOOLSS-DRIVER test.

Guenther

s4-smbtorture: rework EnumPrinterDrivers tests a bit.

Guenther

build: give a more useful error when the source dir has moved

you need a distclean if you mv the source directory

build: use 'waf distclean' for make distclean

this cleans up the more than rm -rf bin

s4:provision Remove moduleload for 'hdb' (wrong name).

The backends are not normally modules anyway

spoolss: pretty-print a struct spoolss_Time.

Guenther

s4-cldap: we should set the w2k8 flags even if not the PDC emulator

these two tests are independent

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

s4-drs: validate RODC credentials via the user_sid

This checks whether a replication client is a RODC by inclusion of the
the DOMAIN_RID_ENTERPRISE_READONLY_DCS sid in the users token

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

s4-drs: added new SECURITY_RO_DOMAIN_CONTROLLER level

This is used for allowing operations by RODCs, and denying them
operations that should only be allowed for a full DC

This required a new domain_sid argument to
security_session_user_level()

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

s4-net: fixed two compiler warnings

s4-torture: fixed a initialiser

we were not initialising the whole array

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

s4-libnet: fixed two compiler warnings

s4-drs: removed dsdb_validate_client_flags()

This test is in the wrong place. We end up validating our own flags.

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

s4-drs: only allow replication with the right invocationId

Non-administrator replication checks the invocationId matches
the sid of the user token being used

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

s4-dsdb: removed an unused variable

s4-dsdb: added dsdb_validate_invocation_id()

this validates that a invocationID matches an account sid

This will be used to ensure that we don't allow DRS replication
from someone a non-DC or administrator

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

s4-dsdb: added dsdb_get_extended_dn_sid()

This will be used by the RODC code

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

build: we don't need this makefile magic any more

The waf build now checks for all A=B variables passed via make
and sets the same waf internal variable. This means all waf options
are available via make.

Removing this from the Makefile makes us less reliant on a modern
version of make.

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

s4-dsdb: moved rodc schema validation to samldb.c

This means we are only doing the checks for schema changes

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

s4-drs: Use new samdb_rodc() function in s4 code

This patch fits the calling to the new samdb_rodc() function and
fix a little bug in this function.

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

s4-drs: Do not send RODC filtered attributes to RODCs on GetNCChanges reply

During building an object to send it on a GetNCChanges reply, it checks
the attributes and if any of them is a RODC filtered and the recipient
is a RODC, then such attribute is not sent.

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

s40-drs: Do not send GetNCChanges messages to RODCs

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

s4-drs: dsdb_validate_client_flags() function

This function is intended to check if some client is not lying about
his flags. At this moment, it only checks for RODC flags.

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

s4-drs: samdb_is_rodc() function and new samdb_rodc() function

This patch creates the samdb_is_rodc() function, which looks for
the NTDSDSA object for a DC that has a specific invocationId
and if msDS-isRODC is present on such object and it is TRUE, then
consider the DC as a RODC.
The new samdb_rodc() function uses the samdb_is_rodc() function
for the local server.

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

s4-drs: Do not allow system-critical attributes to be RODC filtered

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

s4:provision Make OpenLDAP backend more robust

With the extra moduleload lines (which succeed if it's already
staticly linked), we now work with OpenLDAP overlays as modules.

Andrew Bartlett

s4:ldap-backend Fix LSA test failures with OpenLDAP backend - convert SIDs

The SIDs in some queries were not being passed as binary, but as
strings in comparison with the securityIdentifer object.  We need to
recognise that these are SIDs in the simple_ldap_map.

Andrew Bartlett

s4:provison Pass nosync in for the OpenLDAP cn=config too

s4:selftest Ensure we don't fsync() all day in the LDAP backend test

Passing this option greatly reduces the time spent in the test.

Andrew Bartlett

s4:OpenLDAP-backend Use the new rdnval module in OpenLDAP

This is rather than rdn_name, which tries to do the job on the client
side.  We need to leave this module in the stack for Fedora DS (and of
course the LDB backend).

Andrew Bartlett

s4:dsdb Revert accidentilly commited change for LDAP backends

In the future, LDAP backends will be resposible for maintaining the
'name' attributes.

Andrew Bartlett

s4:provision Use more reasonable values for DB_CONFIG

With the OpenLDAP backend, the old DB_CONFIG caused OpenLDAP to abort
on startup, and was very inefficient.  This new one, kindly supplied
by Matthew Backes <mbackes@symas.com> uses a more reasonable set of
buffer sizes.

Andrew Bartlett

build: added --enable-auto-reconfigure

this is off by default until some issues are resolved. See my mail to
samba-technical for details.

s4:netlogon RPC server - fix a counter variable type

build: recalculate project deps when NONSHARED_BINARIES changes

build: added --nonshared-binary=LIST option

This allows you to specify some binaries that should be built without
shared libs. A non-shared smbtorture will make testing s3 in the build
farm easier

s4-waf: python devel headers are mandatory for the source4 build

build: make python development headers not mandatory in standalone libs

This needed an update to the python tool in waf

thanks to Kai for spotting this

waftest: updated the cross compilation environment I test with

s4-server: show build host in samba -b output

build: fixed uname output to be on target machine when cross compiling

this also makes the output of define_ret configure tests show up
in the configure output

s4-upgradeprovision: fixed --realm option duplicate in upgrade_from_s3

s4-drs: accept zero revision in drs selftest

Kamen, please have a look at this. We need to accept revision zero as
w2k8r2 sends it during initial schema replication

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

s4-provision: cope with --realm being in getopt.py

we still need to allow for interactive querying of the realm

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

s4-waf: create the smbd.tmp/messaging directory

this prevents a warning when we run net vampire from the install dir
when samba has never been run previously

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

s4-python: added --realm option to python scripts

this is needed for net vampire

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

s4-provision: set "setup_dir" to the right path

This needs to cope with both running from the build tree or running
from the install tree. We use the provision.smb.conf.dc as a sentinal
to detect if we are in the build tree.

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

s4-schema: allow revision numbers of zero

w2k8r2 sends a revision of zero in the initial schema replication
during a net vampire

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

s4-python: accept --option arguments in python cmdline parsing

also fixed the -d option to use lp.set() which calls lp_set_cmdline()

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

s4-devel: allow extra net command line options and gdb

This allows you to run:

  GDB="gdb --args" vampire_ad.sh

and also to add higher debug levels like this:

  vampire_ad.sh -d100

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

s4-pynet: accept None for target_dir in vampire

Fix i18n of net conf import error message.

Thanks gd

Display an error on net conf import failures.

When something goes wrong, such as a typo in a parameter
name, we'll now display the failure instead of just returning
with -1 and no message.

s4-smbtorture: add spoolss DriverInfo and winreg consistency test.

Guenther

s4-smbtorture: add function to get a printserver's environment.

Guenther

s4-smbtorture: simplify macros used in PrinterInfo winreg consistency teste.

Guenther

s4-smbtorture: refactor test_GetPrinterDriver2().

Guenther

s4:dynconfig: fix the autoconf build and pass -DPYTHONDIR=\"$(pythondir)\"

metze

s4:rpc_server/netlogon: add no memory checks

metze

s4-netlogon: fixed dc_unc and dc_address_type

These are needed for dcpromo from w2k8r2

build: added uname display and SYSTEM_UNAME define

suggestion from Metze

util-runcmd: ignore spurious ECHILD errors

when we get ECHILD in samba_runcmd it is because the parent has set
SIGCHLD to SIG_IGN. In that case the child status information is
lost. We then have to fallback on the logging of child error messages
for any useful information on what happened to the child.

A longer term fix is to stop using SIG_IGN for SIGCHLD in the standard
process model of s4.

s4-smbtorture: add test for csetprinter field behaviour in printer info level 0.

Suprisingly, that value is always 0 (at least on w2k8r2).

Guenther

s4-net: don't show a full python exception when you can't open sam.ldb

s4-net: show a list of commands when someone runs "net" with no arguments

s4-python: added PYTHONDIR to python search path

we put it after the scripting/python dir, so we look in the build
directory (if applicable) first.

s4-dynconfig: added dyn_PYTHONDIR

s3-spoolss: fix winreg spoolss helper call documentation.

Guenther

s3-spoolss: avoid passing down full "struct pipes_struct".

Guenther

s4:netlogon RPC - "fill_one_domain_info" - use "lp_workgroup" for the DC short domainname discovery

Here we don't need to use "lp_sam_name" since in this function we are always a
DC.

s4:torture/rpc/netlogon.c - fix typo

pytalloc: ensure talloc_ctx is directly after PyObject_HEAD

the talloc python interface for tp_alloc and tp_dealloc relies on a
cast to a py_talloc_Object to find the talloc_ctx (see
py_talloc_dealloc). This means we rely on the talloc_ctx for the
object being directly after the PyObject_HEAD

This fixes the talloc free with references bug in samba_dnsupdate

The actual problem was the tp_alloc() call in
PyCredentialCacheContainer_from_ccache_container() which used a cast
from a py_talloc_Object to a PyCredentialCacheContainerObject. That
case effectively changed the parent/child relationship between the
talloc_ctx and the ccc ptr.

This patch changes all the structures that follow this pattern to put
the TALLOC_CTX directly after the PyObject_HEAD, to ensure that if
anyone else decides to do a dangerous cast like this that it won't
cause the same sort of subtle breakage.

Pair-Programmed-With: Rusty Russell <rusty@samba.org>

talloc: there is no ambiguity when freeing a ptr with a null parent

when a ptr has a single reference and a NULL parent, then
talloc_free(ptr) is not ambiguous, as the caller could not have done a
talloc_free(NULL) to free the memory

Pair-Programmed-With: Rusty Russell <rusty@samba.org>

tdb: update tdb ABI to use hide_symbols=True

We now use -fvisibilty=hidden to hide symbols from outside the tdb
shared library.

This also moved tdb_transaction_recover() into the tdb_private.h
header, as it should never have been a public API. For that reason we
are changing the version number. We're only doing a minor version
increment as it is extremely unlikely that anyone was actually using
tdb_transaction_recover() as its locking requirements were rather
unusual.

Pair-Programmed-With: Rusty Russell <rusty@samba.org>

build: include uninitialised data in the ABI symbols

This is needed for symbols like tdb_null in tdb, which are part
of the public ABI

Pair-Programmed-With: Rusty Russell <rusty@samba.org>

build: quote cross-answer strings

This allows for spaces and special characters in cross-answers

build: allow "waf --abi-check" to force a re-check of the ABI

s4:provisionbackend Print the command we failed to start slapd with

This makes it easier to put failed startups into a debugger.

Andrew Bartlett

s4:provision Pass in the invoication ID and NTDS Settings DN to Schema()

By putting these values into the cache on the LDB, this reduces some
of the noise in provision, particularly with the LDAP backend.

Andrew Bartlett

Now SMB2 error messages are correctly being returned with the 1 byte data area, smbd_smb2_request_error_ex() must call smbd_smb2_request_done_ex() in order to do the padding correctly on compound replies.

Jeremy.

Ensure vectors are always allocated with consistent size. Removes one byte alloc on SMB2 error packet. Always use talloc_zero_array on out vectors - fixes valgrind errors in tevent writes.

Jeremy.

Fix valgrind error where a strdup of name reads one byte beyond the end. Ensure buffer returned from inotify is null terminated.

Jeremy.

Removed more excess looping and fixed problem with incorrect IO flag handling.

Remove an unused auto variable.

Jeremy.

libwbclient: wbcFreeMemory deals fine with a NULL pointer

libwbclient: Fix wbcListGroups against too small num_entries

Thanks for the s4 winbind sending 0 here and Tridge to point it out to me :-)

libwbclient: Fix wbcListUsers against too small num_entries

Thanks for the s4 winbind sending 0 here and Tridge to point it out to me :-)

s4:winbind: fill response.data.num_entries for WINBINDD_LIST_USERS

metze

s4:winbind: fill response.data.num_entries for WINBINDD_LIST_GROUPS

metze

s4:winbind: fill response.data.num_entries for WINBINDD_LIST_TRUSTDOM

metze

s3: Remove a leftover of my lua experiments

s3: Move the in-memory ccache to the parent

None of this blocks, so there is no reason to keep this in
a winbind child process

nsswitch: Add wbinfo --pam-logon

This does a wbcLogonUser with credential caching

s3: Test for wb ccache access by smbclient

libwbclient: remove async libwbclient and talloc from libwbclient.so

Except for tests there is right now no active user of this. We can easily
re-add this when smbd makes more use of it.

libwbclient does not need talloc_free anymore

libwbclient: Make wbcListTrusts not use talloc