s3:libsmb: get rid of cli_smb_req_*,cli_smb_wct_ofs,cli_smb_chain_send

Signed-off-by: Luk Claes <luk@debian.org>
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sun Jun  3 23:37:02 CEST 2012 on sn-devel-104

s3:smbd: make conn_close_all() a void function

metze

Revert "s3: Fix bug 8371"

This reverts commit 2642f385887fbd3aecd4286a4d6223a21d981714.

This is not needed anymore, as 035342c11719d1daa647c0b2ae7cec27a969f83a
"Fix bug #8373 - Can't join XP Pro workstations to 3.6.1 DC." is
the more generic fix for the problem.

metze

s3:smbd/close: call del_share_mode() directly before TALLOC_FREE(lck)

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sun Jun  3 21:08:19 CEST 2012 on sn-devel-104

s3:smbd/close: remove unused goto out from close_directory()

metze

s3:smbd/close: do an early return in close_directory()

metze

s3: Fix an assert to trigger in close_remove_share_mode

In case we have a left-over entry in the share mode entry array,
the SMB_ASSERT(got_tokens) is likely to kick in. It happens when
we are about to delete a file with initial delete on close. We don't
have a delete on close token set in the locking.tdb record. We see
the fsp->initial_delete_on_close set, add the delete_on_close token
to lck. Then "delete_file" is being set to true. Then later on we
do the notify_deferred_opens. This walks the list, also checking
for share_mode_stale_pid. We have already deleted our own share
mode entry, share_mode_stale_pid() sees the left-over entry. It not
also deletes that one but also the delete on close token. This leads
to a different view of "delete_file" a.k.a. "got_tokens" further
down in close_remove_share_mode, leading the SMB_ASSERT to fire.

This patch attempts to fix the issue by keeping around our own share
mode entry for almost the whole routine, preventing share_mode_stale_pid()
from removing the delete tokens.

Pair-Programmed-With: Volker Lendecke <vl@samba.org>

s3: Do an early return in close_remove_share_mode

Without "lck" we don't have much to clean up

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

s3: Make close_remove_share_mode a bit more readable

by introducing a variable simplifying a boolean expression

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

s3: Simplify get_delete_on_close_token more

All remaining callers actually want the token

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

s3: Simplify get_delete_on_close_token slightly

Introduce find_delete_on_close_token. Thus is_delete_on_close_set
does not have to call get_delete_on_close_token anymore.

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

s3: Slightly simplify close_remove_share_mode

Remove a level of indentation by more use of "continue;"

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

s3:smbd/close: avoid procid_is_me()

metze

s3:smbd/open: avoid procid_is_me()

metze

s3:smbd/oplock: avoid procid_is_me()

metze

build: Match autoconf build and disable ACLs on MacOS (darwin)

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sun Jun  3 16:03:59 CEST 2012 on sn-devel-104

lib/krb5_wrap: Remove unused smb_krb5_get_creds

Found by callcatcher

This has been unsused since

commit 61f0b247633501d6bf4103ca8345048e537c043d
Author: Günther Deschner <gd@samba.org>
Date:   Thu Nov 12 15:42:03 2009 +0100

    s3-kerberos: remove smb_krb5_get_tkt_from_creds().

    Now that cli_krb5_get_ticket() already handles S4U2SELF impersonation, remove
    smb_krb5_get_tkt_from_creds() which is not required anymore.

    Guenther

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sun Jun  3 13:04:06 CEST 2012 on sn-devel-104

librepace: put #defines after #include "sys/xattr.h"

This avoids redefining the system xattr functions, which should fix MacOS.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sun Jun  3 09:46:44 CEST 2012 on sn-devel-104

lib/replace: Undo change of 0 -> NULL

This reverts part of e9d797e153ae95561dbb10b56a41281b2472f137 as
in the autoconf tests, NULL isn't available!

(it is available in waf, which caused confusion)

Andrew Bartlett

s3-libnetapi: Cope with popt versions without POPT_TABLEEND

libreplace: Fix up MacOS xattr functions

We need undo the rep_ macro to call the real OS function.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sun Jun  3 06:21:21 CEST 2012 on sn-devel-104

s3-build: Remove finddead

This won't work with the new tree structure, and we have the same
output at
https://build.samba.org/lcov/data/coverage/samba_4_0_test/unused-fns.txt

Using the callcatcher framework.

Andrew Bartlett

libreplace: Link libreplace against attr when required

The autoconf Samba build will return to over-linking with -lattr on
systems with both the XFS compat API and native xattrs.

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sun Jun  3 03:56:05 CEST 2012 on sn-devel-104

libreplace: Use true rather than True in xattr.c

This should fix the build on IRIX.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sun Jun  3 02:05:35 CEST 2012 on sn-devel-104

libreplace: Fix build on MacOS where we have the same fn name but more arguments

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sat Jun  2 15:52:51 CEST 2012 on sn-devel-104

libreplace: Solaris needs system/dir.h for that xattr wrapper implementation

build: Remove check for res_ninit (Revert "s3: Check for res_ninit")

This reverts commit 4f6b1e8c81776cf1fa386574e60e54fe6b4fe82e because I
cannot find any use of this configure check, and it is one of the
differences between the WAF and autoconf builds.

Andrew Bartlett

build: Show extra configure results found by WAF as well

This may assist in debugging a difference in a configure test on the
build farm.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sat Jun  2 09:46:09 CEST 2012 on sn-devel-104

ccan: fix autoconf test for isblank()

The define in the C code is HAVE_ISBLANK

Andrew Bartlett

libreplace: Add missing tests for HAVE_MREMAP and HAVE_SHARED_MMAP

libreplace: Fix autoconf build on platforms needing xattrs

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sat Jun  2 07:23:32 CEST 2012 on sn-devel-104

build: Check correctly for sendfile (missing .h in header)

This will hopefully fix the build on SLES8, which does not support 64-bit sendfile.

Andrew Bartlett

lib/replace: Relicence xattr.c to LGPLv3

By the kind consent of the copyright holders.  (There wasn't any code from tridge
in the code brought in from source3/lib/system.c).

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sat Jun  2 04:00:42 CEST 2012 on sn-devel-104

build: Always attempt to build posix ACLs

These are on more systems than just linux.  If the configure test passes
then assume it is available.

Andrew Bartlett

lib/replace: Merge remaining xattr test details from lib/util

I prefer the longer XATTR_ADDITIONAL_OPTIONS define and the NULL
rather than 0 values in the getxattr test.

Andrew Bartlett

s4-xattr: Use libreplace xattr functions directly

lib/replace: xattr wrappers in lib/replace rather than source3/lib/system.c

This also moves all the still-used configure tests etc.  The unused OSF API
is also removed at this time.

Andrew Bartlett

lib/replace: We cannot use strchr_m in lib/replace

In any case, it is always safe to search for . even in a multibyte string.

Andrew Bartlett

lib/replace: DEBUG is not acceptable here, as this may not be linked into Samba

lib/replace: Copy lib/system.c xattr wrappers to lib/replace

s3: Same fix as 8576256, this time for fgetxattr

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Jun  2 02:12:31 CEST 2012 on sn-devel-104

s3: Avoid a lot of calls to serverid_exists()

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri Jun  1 18:58:30 CEST 2012 on sn-devel-104

Revert "waf-mitkrb5: enable dcerpc_server library to support OpenChange client code"

This reverts commit f8c447b1a48eaf12dcf70b92fd7525c4ad26c246.

After discussing with Julien (Openchange) and Metze, I decided to revert this code.
Instead I made a patch to Openchange which allows to build client side only.

Openchange server code requires working s4 member DC and --without-ad-dc build
does not provide working provisioning even if we enable dcerpc_server and end point mapper.

Autobuild-User: Alexander Bokovoy <ab@samba.org>
Autobuild-Date: Fri Jun  1 16:46:08 CEST 2012 on sn-devel-104

waf: check for krb5_create_checksum and krb5_creds.flags for some Heimdal versions

Signed-off-by: Andreas Schneider <asn@samba.org>
Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Fri Jun  1 11:23:21 CEST 2012 on sn-devel-104

waf-mitkrb5: enable dcerpc_server library to support OpenChange client code

Signed-off-by: Andreas Schneider <asn@samba.org>

.gitignore: remove already-gone gen-8bit-gap.sh

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri Jun  1 04:53:53 CEST 2012 on sn-devel-104

build: Add automatic compare of config.h files to recursive waf build

This uses the fact that we have both build systems running at the same time.

The krb5 checks are skipped because we typically are comparing internal Heimdal
with the system krb5, so they do not make sense.

The required checks for krb5 are pretty well understood in any case, as
we have a limited set of supported libraries.

Andrew Bartlett

Remove an unused variable.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Jun  1 03:03:12 CEST 2012 on sn-devel-104

Forward port of Richard Sharpe's <realrichardsharpe@gmail.com> fix for bug #8970 - Possible memory leaks in the samba master process.

We are triggering the cleanup_timeout_fn() too often, on exiting when an smbd is idle.

Calls to exit_server_cleanly() should be treated as a "clean" shutdown,
and not trigger the master smbd to call cleanup_timeout_fn.

s3:smbd: use server_messaging_context() instead of sconn->msg_ctx in exit_server_common()

sconn is not available in the parent anymore.

Thanks to Volker Lendecke <vl@samba.org> for finding this!

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu May 31 18:08:06 CEST 2012 on sn-devel-104

s3:vfs_hpuxacl: remove unused file_find_fd() calls

metze

wintest: use (renamed) smbclient4 tool as smbclient

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu May 31 13:23:34 CEST 2012 on sn-devel-104

wintest: introduce a variable to contain the (relative) path of smbclient

s3: Fix some 64-bit warnings

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu May 31 10:06:56 CEST 2012 on sn-devel-104

build: rename build targets smbclient -> smbclient4 and smbclient3 -> smbclient

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Thu May 31 06:36:55 CEST 2012 on sn-devel-104

s4:selftest: change the blackbox.samba_tool_demote test to use a binary mapping for smbclient

s4:selftest: change the blackbox.passwords test to use a binary mapping for smbclient

s4:selftest: change the blackbox.pkinit test to use a binary mapping for smbclient

s4:selftest: change the blackbox.kinit test to use a binary mapping for smbclient

s4:selftest: change the blackbox.export.keytab test to use a binary mapping for smbclient

s4:selftest: change the blackbox.chgdcpass test to use a binary mapping for smbclient

s4:selftest: change the blackbox.samba_tool test to use a binary mapping for smbclient

s4:selftets: change the blackbox.bogusdomain test to use binary mapping for smbclient

s4:selftest: change the blackbox.smbclient test to use binary mapping for smbclient

s3:selftest: add a binary mapping for smbclient4

selftest: add a binary mapping for smbclient4

build: rename build targets nmblookup -> nmblookup4 and nmblookup3 -> nmblookup

build: add a build suffix ("4") for s4-binaries

s4:selftest: determine nmblookup via binary mapping for blackbox test in tests.py

selftest:Samba4: use the nmblookup4 binary mapping

s3:selftest: add a binary mapping for nmblookup4

selftest: add a binary mapping for nmblookup4

s3:selftest: improve strange linebreaks for blackbox tests in tests.py for readability

s3:selftest: introduce a variable for binpath('dbwrap_tool') in tests.py

s3:selftest: introduce a variable for binpath('ntlm_auth3') in tests.py

s3:selftest: introduce a variable for binpath('smbtorture3') in tests.py

s3:selftest: introduce a variable for binpath('net') in tests.py

s3:selftest: introduce a variable for binpath('wbinfo') in tests.py

to remove explicit calls to binpath('wbinfo')

s3:selftest: introduce a variable for binpath('smbclient3') to test.py

to remove manual calls to binpath("smbclient3")

s3:selftest: introduce a variable for binpath('nmblookup3') in tests.py

s3:build: fix some spacing in wscript_build

Stop spamming the logs with "Could not remove pid XX from serverid.tdb" messages and initiating the cleanup function on every process death.

We now have many sub-processes from smbd that don't serve SMB1/SMB2 requests and
don't register themselves in the serverid.tdb. Only initiate the cleanup
from processes that were explicitly in the child list.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Thu May 31 04:44:09 CEST 2012 on sn-devel-104

lib/dbwrap: make it possible to delete/store the current record during traverse

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu May 31 02:50:09 CEST 2012 on sn-devel-104

lib/dbwrap: don't alter the record on failure in db_rbt_store()

metze

lib/dbwrap: fix db_rbt_store and update the per record node pointer

metze

lib/dbwrap: remove unused per db_record pointer in dbwrap_rbt

metze

Fix bad bugfix for bug #8910 - resolve_ads() code can return zero addresses and miss valid DC IP addresses

Original code incorrectly used a while() instead of a for() loop.
We need to iterate over the entire array here.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed May 30 23:29:03 CEST 2012 on sn-devel-104

Fix metze's complaint about the bugfix for bug #8953 - winbind can hang as nbt_getdc() has no timeout.

This code explicitly isn't needed as the tevent code will take
care of this.

s3:libsmb: use nb_connect_send() directly instead of doing a 0 timer for port 139

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed May 30 20:51:51 CEST 2012 on sn-devel-104

selftest: bug #8373 is fixed and we should always test this now.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed May 30 18:56:38 CEST 2012 on sn-devel-104

dsdb: Fix error checking conditions in partition_metadata module

Thanks to Matthieu Patou <mat@matws.net> for pointing it out.

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Wed May 30 17:00:01 CEST 2012 on sn-devel-104

librpc: Fix an incompatible pointer type warning

Signed-off-by: Amitay Isaacs <amitay@gmail.com>

s4-provision: Use the s3fs file server by default in migrations

This covers both migrations from s3 and joining a domain as a new DC.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed May 30 14:57:22 CEST 2012 on sn-devel-104

s3-build: Fix configure checks for Heimdal

A series of small errors meant that all Heimdal versions became
unsupported, when for the autoconf build, some versions have what we
need.

Andrew Bartlett

build: Make gss_wrap_iov mandatory for krb5 build

This isn't in Heimdal 1.1, so we still fail on that version, but at
least we fail for the right reasons.

Andrew Bartlett

lib/krb5_wrap: Move krb5_princ_size helper to source4 as it is only used there

This is also where the related krb5_princ_component is declared.

Also fix the configure check to use the correct name

This helps the autoconf build on Heimdal.

Andrew Bartlett

s3:utils: use cli_tree_connect() instead of cli_tcon_andx()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed May 30 12:54:05 CEST 2012 on sn-devel-104

s3:libsmb: use cli_tree_connect() instead of cli_tcon_andx()

cli_tree_connect() is more generic and uses what the server supports

metze

s3:libsmb: fallback to SMBtcon for old servers

metze