s4:librpc/rpc: convert dcerpc_bind_send/recv to tevent_req

Many thanks to Andrew Bartlett <abartlet@samba.org> for the
debugging, which lead to the following line:

       talloc_steal(state, raw_packet->data);

metze

s4:librpc/rpc: also notify requests which are not shipped about a dead connection

metze

s4:librpc/rpc: ship requests via an immediate event

Deep inside dcerpc_ship_next_request() some code path
could trigger dcerpc_connection_dead(), which means
it's not safe to do any processing after calling dcerpc_ship_next_request().

metze

s4:librpc/rpc: if the connecion is dead we're done

There's no point in doing anymore processing,
it can just lead to crashes.

metze

s4:librpc/rpc: if the connection is dead we don't want to be called recursivly.

metze

s3: Update waf build to include missed dependancy on Lion.

Autobuild-User: Ira Cooper <ira@samba.org>
Autobuild-Date: Thu Mar 15 07:34:43 CET 2012 on sn-devel-104

util: Add --disable-fault-handling.

On some platforms you can not debug coredumps after the default signal handler
gets done dumping core.  This allows waf to have an option to disable our
default signal handler.

libsmbclient-raw: Install smb_composite.h.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Thu Mar 15 02:57:59 CET 2012 on sn-devel-104

build: Do not build with utmp when we do not have utmp.h

This matches the autoconf build, and should partially address bug #8709.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Mar 15 01:12:15 CET 2012 on sn-devel-104

build: param must require cups to get the cups headers path in -I

s3-krb5: Remove GSS_WRAP_IOV conditional

We already confirm that we have this functionality before we set HAVE_KRB5 at
configure time.

Andrew Bartlett

wintest: test wbinfo --krb5auth

This will ensure that this functionality is tested with MIT krb5, as the
make test system only reaches this code with Heimdal.

Andrew Bartlett

selftest: test wbinfo --authenticate and --krb5auth

build: Ensure that we clean build objects from auth/ as well

selftest/flapping: mark ^samba3.rpc.spoolss.*printserver.enum_printers_servername as flakey

I got this:

[370/1371 in 21m20s] samba3.rpc.spoolss.printserver (s3dc)
Testing servername behaviour in EnumPrinters and GetPrinters
Testing EnumPrinters(\\127.0.0.2) level 2
Testing OpenPrinterEx()
WARNING!: ../source4/torture/rpc/spoolss.c:5907: r.out.result was
WERR_INVALID_PARAM, expected WERR_OK: OpenPrinterEx failed
WARNING!: ../source4/torture/rpc/spoolss.c:6417: Expression
`call_OpenPrinterEx(tctx, p, short_name, ((void *)0), &handle)' failed: failed
to open printer
UNEXPECTED(failure): samba3.rpc.spoolss.printserver
.printserver.enum_printers_servername
REASON: _StringException: _StringException:
../source4/torture/rpc/spoolss.c:6499: Expression
`test_OnePrinter_servername(tctx, p, b, servername, info[i].info2.printername)'
failed: failed to check printer

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Mar 14 23:02:46 CET 2012 on sn-devel-104

selftest/flapping: mark ^samba3.rpc.spoolss.*printserver.openprinter_badnamelist as flakey

I saw this a few times today:

[370/528 in 19m28s] samba3.rpc.spoolss.printserver (s3dc)
Using seed 1331739572
Testing OpenPrinter(\\127.0.0.2)
Testing GetPrinterData(Architecture)
Testing OpenPrinter(__INVALID_PRINTER__) with bad name
Testing OpenPrinterEx(__INVALID_PRINTER__) with bad name
Testing OpenPrinter(\\__INVALID_HOST__) with bad name
Testing OpenPrinterEx(\\__INVALID_HOST__) with bad name
Testing OpenPrinter() with bad name
Testing OpenPrinterEx() with bad name
Testing OpenPrinter(\\\) with bad name
Testing OpenPrinterEx(\\\) with bad name
Testing OpenPrinter(\\\__INVALID_PRINTER__) with bad name
Testing OpenPrinterEx(\\\__INVALID_PRINTER__) with bad name
Testing OpenPrinter(\\127.0.0.2\) with bad name
WARNING!: ../source4/torture/rpc/spoolss.c:5765: op.out.result was WERR_OK,
expected WERR_INVALID_PRINTER_NAME: unexpected result
UNEXPECTED(failure): samba3.rpc.spoolss.printserver
.printserver.openprinter_badnamelist
REASON: _StringException: _StringException:
../source4/torture/rpc/spoolss.c:5821: Expression
`test_OpenPrinter_badname(tctx, b, badname)' failed:

metze

selftest/flapping: mark samba4.blackbox.gentest as flakey

https://git.samba.org/autobuild.flakey/2012-03-14-1307/samba4.stdout

    [1228/1371 in 1h19s] samba4.blackbox.gentest(dc)
    UNEXPECTED(failure): samba4.blackbox.gentest(dc).gentest
    REASON: _StringException: _StringException: seed=1
    Connecting to \localdcest1 as Administrator - instance 0
    Connecting to \localdcest1est1 as Administrator - instance 1
    Connecting to \localdcest1est2 as Administrator - instance 0
    Connecting to \localdcest1est2 as Administrator - instance 1
    0
    OPEN num_open_handles=1 h=0 (gentest\foo4)
    Notify count inconsistent 1 0
    Failed at operation 88 - MKDIR
    gentest failed

I saw this more than 10 times in the last year already.

metze

s3:selftest: add support for 'make test FAIL_IMMEDIATELY=1'

This is what autobuild uses in order to let the build abort on the first
failure.

metze

s4:selftest: fix a message-typo in the "samba-tool user" testsuite.

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Wed Mar 14 21:19:11 CET 2012 on sn-devel-104

s4:selftest: add a new test for "samba-tool user list"

testtools/matchers.py: fix a comment typo

s3-spoolss: Check return type of update_dsspooler().

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Wed Mar 14 19:38:45 CET 2012 on sn-devel-104

s3-spoolss: Check return codes in update_dsspooler.

s3-printing: Check for browseable too.

s3-printing: Make printer a const char *.

s3-rpc_client: Add debug message for printer dataex errors.

s3-rpc_server: Increase debug level for policy handle.

samba3util: Add ccan to dependency list now that str_checksum() uses a ccan function.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Wed Mar 14 17:55:51 CET 2012 on sn-devel-104

s3:smb2_server: fix a compiler warning

Autobuild-User: Christian Ambach <ambi@samba.org>
Autobuild-Date: Wed Mar 14 16:06:48 CET 2012 on sn-devel-104

s3:smbd: also create ncalrpc/np directory before forking rpc daemons

After the fixes to directory_create_or_exist(), this should not be
needed anymore, but lets try to make autobuild reliable first.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Mar 14 13:44:41 CET 2012 on sn-devel-104

lib/util: only change umask during mkdir()

metze

lib/util: cope with races between lstat and mkdir in directory_create_or_exist()

metze

lib/util: use a helper variable in directory_create_or_exist()

metze

lib/util: do an early return on error directory_create_or_exist()

metze

lib/util: remove unneeded else branch in directory_create_or_exist()

metze

lib/util: don't start DEBUG output with 'error '

This confused the subunit code.

metze

s4-dsdb: Fix the case for attribute name msDS-hasMasterNCs

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Wed Mar 14 11:59:02 CET 2012 on sn-devel-104

lib/tdb: remove unnecessary XOPEN and FILE_OFFSET_BITS defines in test/

These were relics: they don't need to be defined here as long as we are
careful to include the replace headers before any standard headers (we are).

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-User: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-Date: Wed Mar 14 10:12:26 CET 2012 on sn-devel-104

lib/tdb2: remove unneccessary _FILE_OFFSET_BITS define in test/.

This was a relic from testing; it can interfere with compile.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

autoconf: make autoconf build work on OS X 10.6

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Wed Mar 14 08:31:19 CET 2012 on sn-devel-104

lib/crypto: Detect CommonCrypto and use it if available

CommonCrypto/CommonDigest is available on Mac and there is function in
the libc for MD5 calculation. MD5Final is a C define of CC_MD5_Final.
Under some circumstance we have the symbol defined twice in samba
binaries on Snow Leopard at least.

By detecting CommonCrypto/CommonDigest we end up always using the system
version if available.

lib/tdb2: make summary handle capabilities properly.

Another PPC issue (endian?) revealed that the summary code did not handle
capabilities correctly: in fact, it went into an endless loop.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-User: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-Date: Wed Mar 14 06:51:43 CET 2012 on sn-devel-104

lib/tdb2: fix error string formatting.

This caused a crash on PPC64 when we failed the mmap (found by failtest,
reported by Amitay)

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

lib/tdb2: Add gcc-style format attribute to tdb_logerr.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

Fix bug #8811 - sd_has_inheritable_components segfaults on an SD that se_access_check accepts.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Mar 14 05:08:03 CET 2012 on sn-devel-104

Fix bug #8795 - Samba does not handle the Owner Rights permissions at all

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Mar 14 02:26:34 CET 2012 on sn-devel-104

Second part of fix for bug #7933 - samba fails to honor SEC_STD_WRITE_OWNER bit with the acl_xattr module.

Error found by Andrew Bartlett <abartlet@samba.org> and Ricky Nance
<ricky.nance@weaubleau.k12.mo.us>.

Don't use a pointer when you really mean a bool flag.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Mar 13 21:56:15 CET 2012 on sn-devel-104

Fix bug #8807 - dcerpc_lsa_lookup_sids_noalloc() crashes when groups has more than 1000 groups

Use correct talloc heirarchy.

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

s4-python: Add missing python source file encoding.

See http://www.python.org/dev/peps/pep-0263/

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Tue Mar 13 19:31:09 CET 2012 on sn-devel-104

s4-python: Remove env from non-executable webserver script.

s4-python: Remove env from non-executable samba_tool scripts.

s4-python: Remove env from non-executable dcerpc scripts.

s4-python: Remove execute flag from non-executable rpc_talloc script.

s4-python: Remove env from non-executable blackbox test scripts.

s4-python: Remove execute flag from ndrdump blackbox script.

s4-python: Remove env from non-executable test scripts.

s4-python: Remove env from non-executable netcmd scripts.

s4-python: Remove execute flag from netcmd scripts.

s4-python: Remove env from non-executable samba scripts.

s4-python: Remove execute flag of upgradehelpers.py.

s3-spoolss: Consistently fail OpenPrinterEx with "" printername

samba3.rpc.spoolss.printserver has become a flakey test recently, and this
papers over the real problem.

Andrew Bartlett

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue Mar 13 17:51:00 CET 2012 on sn-devel-104

nbench: Fix typo.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Tue Mar 13 14:53:07 CET 2012 on sn-devel-104

samba.8: Fix typo: deamon -> daemon.

samba_dnsupdate: Mention contents of invalid line when encountering parsing error.

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

samba_dnsupdate: Raise proper exception when getting unexpected DNS reply.

samba_dnsupdate: Use docstrings, which show up nicely in API docs.

s3-rpc: Decrypt with the proper session key in CreateTrustedDomainEx2.

On LSA and SAMR pipes session_key is truncated to 16 byte when doing encryption/decryption.
However, this was not done for trusted domain-related modifying operations.

As result, Samba 4 client libraries do not work against Samba 3 while working
against Windows 2008 r2.

Solved this by introducing "session_extract_session_key()" function that allows to specify
intent of use of the key.

Signed-off-by: Andreas Schneider <asn@samba.org>
Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Tue Mar 13 12:23:44 CET 2012 on sn-devel-104

selftest: samba3.smbtorture_s3.LOCAL-TALLOC-DICT works now

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Mar 13 10:38:20 CET 2012 on sn-devel-104

s3:torture: fix segfault in LOCAL-TALLOC-DICT

talloc_dict_set() expects a pointer to an talloc'ed pointer.

metze

s3:torture: talloc_dict_traverse() returns the number of entries

metze

selftest.run: Use subunit version of iso8601.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Mon Mar 12 01:36:25 CET 2012 on sn-devel-104

selftest.py: Remove some unnecessary checks.

selftest.py: Move some infrastructure for running commands to selftest.run, with tests.

upgradedns: Remove unused import.

samba-tool: Write diagnostics to stderr rather than stdout.

s4:upgradeprovision - fix some output messages

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sun Mar 11 21:54:54 CET 2012 on sn-devel-104

s3: Improve statvfs support

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Sun Mar 11 14:18:04 CET 2012 on sn-devel-104

waf: Do not use standards.h on darwin

We get a nasty warning every time we include that

s4 dns: Test deleting records and fix a small bu

Autobuild-User: Kai Blin <kai@samba.org>
Autobuild-Date: Sun Mar 11 02:27:45 CET 2012 on sn-devel-104

s4 dns: Be less chatty during tests

s4 dns: Clean up tests a bit

s4 dns: Support TXT updates, add tests

s4 dns: Support TXT queries

s3 selftest: Allow DNS updates during testing

tdb_wrap: Move to specific directory.

It's a bit confusing to mix low-level and high-level libraries.  We had
multiple libraries in one directory, and there were have circular
dependencies with other libraries outside that directory (in this case,
samba-hostconfig).

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sat Mar 10 23:13:01 CET 2012 on sn-devel-104

samba.tests.blackbox.samba_tool_drs: Remove unused import.

wscript: Include talloc, tdb and ldb in pydoctor output.

samba.samdb: Fix formatting of arguments in docstring.

pydoctor: Specify --project-name.

python: samba: Use relative import for _glue.

For some reason, using an absolute import breaks pydoctor.

tevent: Install tevent.py into PYTHONARCHDIR rather than PYTHONDIR for consistency with other Samba scripts.

selftest: fix running additional smbclient test for non-spnego session setup

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sat Mar 10 20:46:05 CET 2012 on sn-devel-104

s3: fix build on AIX

Autobuild-User: Björn Jacke <bj@sernet.de>
Autobuild-Date: Sat Mar 10 19:07:20 CET 2012 on sn-devel-104

s3: Add smb_request_done

This is used to enable async chained command sequences. A synchronous
reply_xxx command does not need to take are anymore about and_x
chaining. The async commands (pipe r/w at this moment) must do so
however. When finished, they must inform the main chain engine that
they are finished with a smb_request_done call.

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Sat Mar 10 17:14:05 CET 2012 on sn-devel-104

s3: Remove unused smb_request->done

s3: Remove unused smb_request->chain_outbuf

s3: Remove "req_wct_ofs()"

This is fixed up in construct_reply_chain

s3: Fix the read&x offset within a chain

s3: Remove chain_reply

<insert your favourite tombstone ascii art here>

s3: Replace chain_reply

This is a new implementation of our andx handling code. The old
code was quite involved in that it was called from within the reply_
handlers. This leads to pretty complex faking of smb_request
structures to give them the same environment, independent of whether
they are called directly or from within chain_reply.

chain_reply needs to go because it blocks really async handling of
chained requests.