s4-socket: allow connect_multi_next_socket to try all the IP for a given host

This fix an incorrect behavior which was that if a host has 2 IP but
was listening on only 1 (and the second one) connect_multi_next_socket
was not able to connect because it used only the first result.

py_passdb: Cannot steal an item pointer from an array

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Fri Nov 11 12:04:33 CET 2011 on sn-devel-104

policy/tests: Fix import of TestCase.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Fri Nov 11 01:10:49 CET 2011 on sn-devel-104

selftesthelper: Fix argument order.

selftest: Make planoldpythontestsuite arguments consistent with planpythontestsuite.

selftesthelpers: Fix syntax error.

selftest: Simplify calling of python tests.

selftest: Allow passing arbitrary number of additional python path values.

ldb: Re-apply pyldb fixes from Tridge and Amity, using unittest rather
than samba.tests.

subunitrun: Add notes about subunitrun being deprecated.

selftest: re-implement directory option for planpythontestsuite.

Revert "test: added directory option to planpythontestsuite()"

This reverts commit b0be72d5080aed3da3f33a45b72f88e90fbe3d13.

I'll add an alternative that uses subunit.run.

Revert "test: added -D option to subunitrun"

This reverts commit ac0bba2ea18e2e9ac578a333a3b260951e4c976e.

"python -m subunit.run" is the preferred way of running tests to
generate subunit. I also don't want to move too far away from
the standard test running infrastructure for python so that
other test runners (nosetests, trial, tribunal, ...) can be used with
Samba code.

Revert "pyldb: fixed ldb API test"

This reverts commit 333248f08c83cc5b18f8e7e84cb19d79a8309d6e.

This commit made the pyldb tests depend on the Samba testsuite.

libcli/cldap/cldap.c - remove outdated comment

Reviewed-by: metze
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Thu Nov 10 20:32:08 CET 2011 on sn-devel-104

s3: server_id.pid has turned 64 (bits, that is)

Fix ctdb_processes_exist protocol. The socket expects pid_t which is 32 bits on
32 bit machines.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Nov 10 18:57:01 CET 2011 on sn-devel-104

s3: Avoid a race with the async echo handler

We can not read from the echo handler socket when we have the main socket
locked. This leads to the echo responder to lock up sitting in the fcntl lock
while the parent wants to read the remainder of a large packet.

libcli/cldap: fix a crash bug in cldap_socket_recv_dgram() (bug #8593)

After a calling any wrapper of tevent_req_notify_callback(),
e.g. tevent_req_nterror(), tevent_req_done(), tevent_req_nomem(),
a function has to return immediately otherwise it is very likely to
crash, unless in calls tevent_req_defer_callback() before.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Nov 10 16:31:59 CET 2011 on sn-devel-104

s4:finddcs_cldap: debug the correct server address instead of NULL

metze

s3:smbd: don't limit the number of open dptrs for smb2 (bug #8592)

This fixes a crash bug that is triggered, when a client has more than
256 directory handles with searches.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Nov 10 14:08:14 CET 2011 on sn-devel-104

s3:smbd: fully construct the dptr before allocating a dnum in the bitmap

metze

s3:smbd: avoid string_set() in dir.c

And do some more error checks.

metze

Fix -Wunused-but-set-variable compiler warnings in tevent_signal.c

The results of some read(2) and write(2) calls are assigned into a
variable that is never used.  Presumably this used to avoid compiler
warnings or similar.

However, from (approximately) GCC 4.6 this produces some warnings:

  [ 609/3910] Compiling lib/tevent/tevent_signal.c
  ../lib/tevent/tevent_signal.c: In function ‘tevent_common_signal_handler’:
  ../lib/tevent/tevent_signal.c:85:10: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
  ../lib/tevent/tevent_signal.c: In function ‘signal_pipe_handler’:
  ../lib/tevent/tevent_signal.c:183:10: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]

The simplest thing to do is remove the variables and cast the function
return to void.  There is already a comment above each call.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Signed-off-by: Stefan Metzmacher <metze@samba.org>

s4: samba-tool time --help documentation improvements

Signed-off-by: Theresa Halloran <thallora@linux.vnet.ibm.com>
Signed-off-by: Andrew Tridgell <tridge@samba.org>
Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Thu Nov 10 06:04:22 CET 2011 on sn-devel-104

s4: samba-tool group --help documentation improvements

Signed-off-by: Theresa Halloran <thallora@linux.vnet.ibm.com>
Signed-off-by: Andrew Tridgell <tridge@samba.org>

test: fixed several tests to use samba.tests

this fixes error checking. Test failures were not being detected
otherwise

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

samba-tool: print samaccountname in samba-tool user list

this is the attribute the admin needs for other commands

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

test: use planpythontestsuite() for ldb.python test

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

test: added directory option to planpythontestsuite()

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

test: added -D option to subunitrun

this allows for:

 subunitrun -D lib/ldb/tests/python api

which makes this easier for developers

pyldb: fixed ldb API test

it was failing previously, but the failure wasn't being noticed

build: only run symbol checking if running all tests

this makes it much faster to run 1 test

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

pyldb: enhanced get() method on msg object

get() now takes a default value and an idx, allowing for much easier
usage

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

samba-tool: Added new "user disable" command

samba-tool: Added "user list" command

samba-tool: Change internal argument name in domain machinepassword

samba-tool: Fix domain machinepassword secret db search

pair programmed with Jelmer Vernooij
Add search to include secrets.ldb

samba-tool: Fix domain machinepassword name search

Append '$' to name provided by user to search for a machine account
when necessary

s4-test: improved error checking in samba-tool time test

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

s4-socket: detect NULL server in socket connection code

return NT_STATUS_NOT_FOUND instead of a segfault

Pair-Programmed-With: Amitay Isaacs <amitay@gmail.com>

test: enable time and user samba-tool tests

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

test: add samba-tool user tests

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

test: Add samba-tool time tests

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

test: add directory and base class for samba-tool tests

this allows for more sophisticated testing of samba-tool

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

s4-setup Remove servicePrincipalName: DNS/${DNSDOMAIN} from new installations

This servicePrincipalName is incorrect (windows does not use that
servicePrincipalName, as it targets the server it is updating, not the
root of the DNS tree), and now that we have multiple DNS backends that
use the internal database, it is quite incorrect (as it cannot exist
on more than one account).

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Nov 10 01:11:46 CET 2011 on sn-devel-104

s3-py_passdb Ensure that group mapping list input is initailised

This may help to ensure we consistantly crash on an incorrect de-reference.

Andrew Bartlett

s3-passdb Initialise the correct level of pointer dereference

*pp_rmap may be NULL or un-initialised data.

This was introduced by 995d1567265be178b4e45f79ea4562a7041ffa52.

Andrew Bartlett

s4:join.py - fix up "DsAddEntry" against Windows Server 2000

It is important to consider the result level regarding error
information. Windows 2000 seems to send us back v2, all other
implementations v3. Consider post "Joining W2k AD domain"

Reviewed-by: abartlet
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Wed Nov  9 23:37:08 CET 2011 on sn-devel-104

docs: Make clear to use the workgroup name not the realm.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Wed Nov  9 21:17:09 CET 2011 on sn-devel-104

s4: Install samba-tool and samba_kcc to bin/ rather than sbin/.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Wed Nov  9 19:43:07 CET 2011 on sn-devel-104

samba_kcc: Remove unnecessary return statements.

util/time: Make some functions explicitly public.

util/time: Remove unnecessary _PUBLIC_ - that's implicit for things in header files.

s3:registry: prevent unnecessary transactions in regdb_init

Skip the transaction for the upgrade code, if the database
is already at code-level.

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Wed Nov  9 15:06:59 CET 2011 on sn-devel-104

s3: Use autotools to set the winbind socket directory.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Wed Nov  9 13:31:27 CET 2011 on sn-devel-104

s3:smb2_server: grant credits in async interim responses (bug #8357)

The first fix for bug #8357 intruduced a regression, so that we no
longer grant credits for real async interim responses with
STATUS_PENDING.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Nov  9 11:56:29 CET 2011 on sn-devel-104

s3:libsmb: consistently use state->size in cli_write_andx_create() (bug #5326)

Otherwise we may get unexpected results.

This is a fix that was missing in commit 95595dd93fd04999fcf56ecaab7c29b064d021f8
(s3:libsmb: fix cli_write_and_x() against OS/2 print shares (bug #5326))

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Nov  9 10:13:32 CET 2011 on sn-devel-104

s4-provision: Create ldap records for DNS partitions similar to windows

this allows easy comparison between windows and samba ldap trees

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Wed Nov  9 08:27:46 CET 2011 on sn-devel-104

s4-provision: Set security descriptor for DNS records

s4:torture:smb2: rewrite durable-open.open test to use smb2_*_create functions

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Wed Nov  9 02:36:36 CET 2011 on sn-devel-104

s4:torture:smb2: rewrite the durable-open.lock test to use smb2_lease_create()

s4:torture:smb2: rewrite the durable-open.lease test to use smb2_lease_create()

s4:torture:smb2: rewrite the durable-open.oplock test to use smb2_oplock_create()

s4:torture:smb2: rewrite the durable-open.file-position test to use smb2_oplock_create()

s4:torture:smb2: rewrite the durable-open.open2 test to use smb2_lease_create_share()

s4:torture:smb2: rewrite the durable-open.open1 test to use smb2_oplock_create_share()

s4:torture:smb2: add smb2_oplock_create_share() that takes sharemode as parameter

s4:torture:smb2: add smb2_lease_create_share() that takes a sharemode as parameter

s4:torture:smb2: add smb2_generic_create_share() that also takes the sharemode as parameter

s4:torture:smb2: move the smb2_create_*() utility functions from lease.c to util.c for re-use

s4:torture:smb2: use smb2_util_oplock_level() in the durable-open test for convenience

s4:torture:smb2: lease: oplock_level is uint8_t, not uint32_t or int

s4:torture:smb2: move oplock() from lease.c to smb2_util_oplock_level() in util.c for re-use

s4:torture:smb2: extract map_sharemode() to smb2_util_share_access() in util.c

s4:torture:smb2: use common smb2_util_lease_state() in the lease tests

s4:torture:smb2: extract map_lease() to util.c as smb2_util_lease_state().

s4:torture:smb2: fix illegal write/uninitialized data bug in the lease.multibreak test

s4:torture:smb2: lease.update2: some more reasonable struct names and a comment

s4:torture:smb2: change the test file name in the durable-open.oplock test

to a more reasonable name. This one must have been copied
from the durable-open.lease test.

samr: filterModuleName is a lsa_String in userPwdChangeFailureInformation.

The entire marshalling of samr_ChangePasswordUser3 broke with c2685cdedb430ae75a94e86f34484292b4269363.

Matthias, the bad effect of this change was that actually all failed password
change attempts will always return NT_STATUS_OK because the last 4 bytes (the
resulting status code) were not marshalled anymore.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Nov  9 00:41:13 CET 2011 on sn-devel-104

s4-smbtorture: demonstrate how broken our samr_ChangePasswordUser3 marshalling is...

Guenther

s4:samba-tool user command - always initialise the boolean arguments properly

It they haven't been specified.

Reviewed-by: Jelmer
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Tue Nov  8 22:28:38 CET 2011 on sn-devel-104

s4:samba-tool - other fix for Gémes Géza patch regarding parameter handling

The condition has not been specified correctly since we are expecting a boolean.
Pointed out by Jelmer.

Reviewed-by: Jelmer

s3:libsmb: fix cli_write_and_x() against OS/2 print shares (bug #5326)

Print shares doesn't support CAP_LARGE_WRITEX, while it's negotiated
by the file server part.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Nov  8 17:01:36 CET 2011 on sn-devel-104

s3:libsmb: correctly parse the LANMAN2.1 negprot response from OS/2 (bug #8584)

metze

s3:libsmb: key_len is 8bit only in the NT1 case

metze

s3: Remove two unused variables

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Tue Nov  8 10:14:36 CET 2011 on sn-devel-104

s4-provision: site is passed as an argument to setup_ad_dns()

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Tue Nov  8 06:07:46 CET 2011 on sn-devel-104

s4-provision Remove options for LDAP backend to reduce user confusion

We do not support the LDAP backend any more, but keep the code in case someone
comes up with an interesting use case that could leverage this in a very
particular situation.  In order to keep the code, we must test it, so
we keep just this much of the support around.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Nov  8 04:33:49 CET 2011 on sn-devel-104

s4-s3-upgrade Add my copyright

param: Remove duplicate initialization of 'share backend' parameter

s4-smb_server No longer follow the security=share smb.conf directive

By ignoring the value of security= from the smb.conf, we can allow this
to instead set the value of 'server role' in a manner compatible
with the Samba 3.x release stream.

Andrew Bartlett

selftest: Remove the 'all' environment as it is just too slow to start up

Instead we start the 'dc' environment, and other environments are available as:
make testenv SELFTEST_TESTENV=fl2003dc

Andrew Bartlett

s4-dnsserver: Test forward zones are not listed in reverse zone search

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Tue Nov  8 01:26:43 CET 2011 on sn-devel-104

s4-dnsserver: Fix enumeration of zones in ComplexOperation RPC call

zone_request_flags are interpreted in different groups rather than
a single group. This correctly returns 0 zones when there are no
reverse zones and DNS_ZONE_REQUEST_REVERSE is set in zone_request_flags.

Add the SEC_DIR_LIST check to dptr_create().

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Nov  7 21:11:03 CET 2011 on sn-devel-104

s4:samba-tool - fix Gémes Géza patch regarding parameter handling

The new "random-password" parameter has not been evaluated correctly.

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Mon Nov  7 19:35:05 CET 2011 on sn-devel-104

Add a --random-password option to user create command.

Signed-Off-By: Jelmer Vernooij <jelmer@samba.org>
Signed-off-by: Matthias Dieter Wallnöfer <mdw@samba.org>

s3-wafbuild: Fix inotify detection (bug 8580)

Enable inotify if sys or kernel inotify is available.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Nov  7 16:28:38 CET 2011 on sn-devel-104

s3-build: Fix inotify detection (bug 8580)

Enable inotify if sys or kernel inotify is available.

Signed-off-by: Stefan Metzmacher <metze@samba.org>