s3-rpc_server request both sign and seal for clarity

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

s3-auth remove auth_ntlmssp_start(), call auth_generic_start() directly

This makes it clear that this can support more than just NTLMSSP.

Andrew Bartlett

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

s3-auth rename auth_ntlmssp_prepare() -> auth_generic_prepare()

This function handles more than NTLMSSP now, at least when we are an AD DC
and so changing the name may avoid some confusion in the future.

Andrew Bartlett

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

s3-auth rename auth_ntlmssp_state -> auth_generic_state

This structure handles more than NTLMSSP now, at least when we are an AD DC
and so changing the name may avoid some confusion in the future.

Andrew Bartlett

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

s3-rpc_server request the DCE_STYLE feature in ntlmssp_server_auth_start

This is not used or honoured by NTLMSSP, but I hope to make this routine
more generic in the future.

Andrew Bartlett

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

s3-netlogon: use dsgetdcname() instead of get_dc_name()

Sometimes the domain parameter might not contain the NetBIOS name of the remote
domain but the DNS name.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Thu Dec 22 19:21:21 CET 2011 on sn-devel-104

s3-netlogon: Add support to authenticate trusted domains.

s4-torture: Test rpc schannel netr_LogonGetCapability.

metze

s3-rpc_server: Pass in our flags to netlogon_creds_server_init().

metze

s3-netlogon: Add support for LogonGetCapabilities.

This is also needed to support AES.

metze

s4-librpc: Fix netlogon schannel client connect.

As a client we request as much flags as possible. The server checks
which flags it supports and returns the same negotiation flags or less.
So we need to store the negotiate flags from the server. We need them
later if we have to call netr_LogonGetCapabilities.

s3:torture/test_smb2: also try PROTOCOL_SMB2_24

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Dec 22 17:47:17 CET 2011 on sn-devel-104

lib/param: add support for "SMB2_24" in smb.conf options

metze

s3:smb2_negprot: add support for PROTOCOL_SMB2_24

metze

libcli/smb: add PROTOCOL_SMB2_24 support

metze

libcli/smb: add SMB2_DIALECT_REVISION_224

This is specified in the new [MS-SMB2] preview document.

metze

s3-auth Restore shortcut for guest security token

This was lost when the server_info and session_info structures were split.

This helps avoid doing lookups for the guest account to determine the
uid/gid and SID values.

Andrew Bartlett

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Dec 22 15:51:09 CET 2011 on sn-devel-104

runcmd: use set_close_on_exec()

this prevents a fd leak to child processes

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Dec 22 14:00:06 CET 2011 on sn-devel-104

runcmd: use a pipe for stdin to child processes

this allows child processes to detect the exit of the parent by
looking for EOF on stdin

Change the signature of pthreadpool_finished_job() to return 0
on success, errno on fail and return the jobid in a separate variable.

I need this fix for my vfs_aio_pthread.c module.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Thu Dec 22 12:12:33 CET 2011 on sn-devel-104

param: domain_logons and domain_master are of type enum_bool_auto

These parameters should be defined as int and not bool. This fixes
the test failures on big endian machines.

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Thu Dec 22 10:37:42 CET 2011 on sn-devel-104

s3-rpc_server: Add my copyright for my previous work here

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Dec 22 09:02:57 CET 2011 on sn-devel-104

s3-rpc_server: Remove old comment

s3-rpc: added "rpc_server:default" config option

this allows the config to specify a default behaviour (embedded,
external or disabled) for unknown pipes. This is needed to allow the
s3 smbd server to redirect unknown pipes to the s4 RPC server when
using s3 smbd as a file server for a s4 DC. If rpc_server:default is
not specified then this change preserves the old behaviour

s3-loadparm: mark a few parameters as having P_BYTES type

this fixes a problem with "max xmit = 32K" resulting in a maximum SMB
transmit size of 32 bytes, which makes for some very interesting
network traces!

This is what was causing smbtorture to consume gigabytes of memory in
the rpc.schannel test

param: Fix the data type for bAvailable

This causes the copy_service() to not copy bAvailable boolean on
big endian machines causing tests to fail.

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Thu Dec 22 05:30:49 CET 2011 on sn-devel-104

s3: There's no reason not to at least build winbind on darwin

It does not necessarily do nsswitch services, but as a NETLOGON proxy
it should work fine

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Dec 22 00:16:44 CET 2011 on sn-devel-104

s3-dns: prevent from potentially doing wrong SRV DNS lookups.

With an empty sitename we asked for e.g.
_ldap._tcp.._sites.dc._msdcs.AD.EXAMPLE.COM

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Dec 21 17:23:25 CET 2011 on sn-devel-104

idl: Avoid c++ style comments

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Dec 21 13:36:01 CET 2011 on sn-devel-104

tdb: don't free old recovery area when expanding if already at EOF.

We allocate a new recovery area by expanding the file.  But if the
recovery area is already at the end of file (as shown in at least one
client case), we can simply expand the record, rather than freeing it
and creating a new one.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-User: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-Date: Wed Dec 21 06:25:40 CET 2011 on sn-devel-104

tdb: use same expansion factor logic when expanding for new recovery area.

If we're expanding because the current recovery area is too small, we
expand only the amount we need.  This can quickly lead to exponential
growth when we have a slowly-expanding record (hence a
slowly-expanding transaction size).

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

Allow an object to be deleted from a directory if the caller has DELETE_CHILD access
even if we don't have access to read the ACL on the object. Fixes bug #8673 - NT ACL issue.
Different fix needed for 3.6.x.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Dec 20 22:13:51 CET 2011 on sn-devel-104

s3: Fix some False/NULL hickups

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Tue Dec 20 13:13:17 CET 2011 on sn-devel-104

samba-tool: Add transaction wrapper for creating GPO

This ensures that if the GPT files are not copied via SMB,
AD changes are rolled back.

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Tue Dec 20 09:12:41 CET 2011 on sn-devel-104

tdb2: Avoid a malloc/memcpy in _tdb1_store

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Dec 19 16:53:40 CET 2011 on sn-devel-104

tdb: Avoid a malloc/memcpy in _tdb_store

s4-dsdb: Relax the conditions where we can't do a subtree delete

If the parent object is a SAM object (as defined in 3.1.1.5.2.3
Special Classes and Attributes of MS-ADTS) then we can use the subtree
delete control even if the object is a critical one.

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Mon Dec 19 14:32:19 CET 2011 on sn-devel-104

s4:drsuapi/getncchanges: return WERR_NOMEM if talloc_array() fails

metze

s4-drs: introduce a timeout in the getncchanges processing to always return something in less than x seconds

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

s4-drs: avoid calling unecesserly ldb_msg_find_attr_as_* as this call in unefficient

Current implementation of ldb_msg_find_attr_as_* iterate on the list of
attributes returned by the search and make a string comparison. As we
sorting the array of messages / guids we tend to call this function many
times. By storing the GUID and the USN in a separate structure we are
sure to call this function only once per attribute and object.

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

s4-becomedc: replicate first with DRS_CRITICAL_ONLY and DRS_GET_ANC objects for the base dn partition

Windows dcpromo do the same: getncchanges with DRS_GET_ANC and
DRS_CRITICAL_ONLY, then it does a getncchanges without those flags for
the rest.

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

s4-join: replicate first with DRS_CRITICAL_ONLY and DRS_GET_ANC objects for the base dn partition

Windows dcpromo do the same: getncchanges with DRS_GET_ANC and
DRS_CRITICAL_ONLY, then it does a getncchanges without those flags for
the rest.

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

tdb: be more careful on 4G files.

I came across a tdb which had wrapped to 4G + 4K, and the contents had been
destroyed by processes which thought it only 4k long.  Fix this by checking
on open, and making tdb_oob() check for wrap itself.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-User: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-Date: Mon Dec 19 07:52:01 CET 2011 on sn-devel-104

patch tdb-oob-fix.patch

s3-net: Fix the return codes. 0 on success, -1 on failure

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Mon Dec 19 01:57:24 CET 2011 on sn-devel-104

s3: Fix the clustering build

ctdb_private.h already defines set_close_on_exec.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat Dec 17 18:41:39 CET 2011 on sn-devel-104

libcli/util: add NT_STATUS_NETWORK_SESSION_EXPIRED and NT_STATUS_ALL_SIDS_FILTERED

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sat Dec 17 14:24:40 CET 2011 on sn-devel-104

s4 dns: Allow updates based on smb.conf setting

Autobuild-User: Kai Blin <kai@samba.org>
Autobuild-Date: Sat Dec 17 04:19:40 CET 2011 on sn-devel-104

s4 dns: Add some debug output

s4 dns: Actually handle the update request

s4 dns: Implement RFC-compatible update prescan

s4 dns: Update prerequisite checking conforming to RFC

s4 dns: Give better test output on RCODE mismatch

s4 dns: Move record lookup to dns_utils.c

s4 dns: Implement dns record comparison function

s4 dns: Implement dns name equality check

s4 dns: Fix some typos in comments and a whitespace

check_name() not needed in mkdir.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Dec 17 02:44:55 CET 2011 on sn-devel-104

Ensure VALID_STAT before accessing fields.

s4:torture/rap/printing: check smbcli_rap_netprintqgetinfo() WERR status

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sat Dec 17 01:09:32 CET 2011 on sn-devel-104

Fix bug #8664 - Renaming a symlink fails if the symlink target is outside of the share.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Dec 16 23:32:26 CET 2011 on sn-devel-104

Restrict UCF_UNIX_NAME_LOOKUP (symlink allowed) to restricted set of calls.

wafsamba: Fix undefined variable name in error message.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Fri Dec 16 21:53:57 CET 2011 on sn-devel-104

s3:idmap_autorid: use less transactions

reduce the amount of transactions that are mostly unnecessary because no
updates were done, only reads

Autobuild-User: Christian Ambach <ambi@samba.org>
Autobuild-Date: Fri Dec 16 20:18:37 CET 2011 on sn-devel-104

s3:dbwrap fix return value of db_tdb_parse

use the TDB ecode to determine the NTSTATUS return value
and not the return code that is just -1

s3:printing/spoolssd: avoid using server_messaging_context()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri Dec 16 16:29:41 CET 2011 on sn-devel-104

s3:smbd: remove unused sconn_server_id()

metze

s3:smbd: avoid using sconn_server_id()

metze

s3:locking: avoid using sconn_server_id()

metze

s3:lib/conn_tdb: avoid using sconn_server_id()

metze

s4:libcli/rap: the caller should get talloc children from call->ndr_pull_{param,data}

This should fix the flakey 'rap.printing.rap_printq_getinfo' test,
see https://git.samba.org/autobuild.flakey/2011-12-16-0022/samba4.stdout

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri Dec 16 13:11:42 CET 2011 on sn-devel-104

s4:libcli/rap: call->ndr_push_{param,data} can be talloc childs of 'call'

metze

s4:libcli/rap: use talloc_zero() and initialize everything

metze

s4:libcli/rap: remove unused members from struct rap_call

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri Dec 16 10:53:42 CET 2011 on sn-devel-104

s4:libcli/rap: initialize call->rcv_datalen

metze

s3: Add some const to find_oplock_types

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Dec 16 09:10:07 CET 2011 on sn-devel-104

s3: Make open_file_ntcreate a bit easier to read

Move the declaration of "id" closer to its use

s3: Make open_file_ntcreate a bit easier to read

Move the calculation of "new_file_created" closer to its use

s3: Make open_file_ntcreate a bit easier to read

Remove a negation where it's not necessary

Add support for TCP_DEFER_ACCEPT

"man tcp" on Linux says:

TCP_DEFER_ACCEPT

Allows a listener to be awakened only when data arrives on the  socket.  Takes
an integer value (seconds), this can bound the maximum number of attempts TCP
will make to complete the connection. This option should not be used in code
intended to be portable.

This might reduce the 139/445 forks a bit on high-load servers

s3: Slightly simplify delay_for_exclusive_oplocks

s3: Slightly simplify delay_for_batch_oplocks

Third part of fix for bug #8663 - deleting a symlink fails if the symlink target is outside of the share.

smbd_check_access_rights() - we can always delete a symlink.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Dec 16 03:32:15 CET 2011 on sn-devel-104

Second part of fix for bug #8663 - deleting a symlink fails if the symlink target is outside of the share.

Ensure we use UCF_UNIX_NAME_LOOKUP flags on filename_convert()
when doing a UNIX infolevel in trans2setfilepathinfo().

First part of fix for bug #8663 - deleting a symlink fails if the symlink target is outside of the share.

Remove two unneeded check_name() calls. They have already been done
in order to get here.

lib/param: Set s4 "host msdfs = true" by default

This matches the s3 loadparm, and makes this feature available
by default for our users in a DC environment.  (This is needed
for the correct operation of GPOs).

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri Dec 16 01:08:34 CET 2011 on sn-devel-104

loadparm: fixed service list handling

when you have:

  server services = +smb -s3fs

and 'smb' is already in the list, then this should not be an
error. This ensures that a config that specifically sets the services
it wants doesn't generate an error if the service list being set
happens to be the default

pidfile: use set_close_on_exec()

this prevents a fd leak to child processes

s4-socket: use set_close_on_exec()

this prevents a fd leak to child processes

genrand: use set_close_on_exec()

this prevents a fd leak to child processes

lib/util: added set_close_on_exec()

this was already in tevent_util.c, but library layering prevented us
from using it in some other libraries

dynconfig: added SBINDIR and BINDIR as updated dynconfig variables

this allows these to work correctly in a build environment, pointing
at bin/

s3: Fix a typo

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Dec 15 23:35:28 CET 2011 on sn-devel-104

s3:vfs fix up vfs_gpfs after function pointer renames

In 422494a8e630e2ca89386344eaa5346388698a32, vfs_gpfs.c was forgotten

Autobuild-User: Christian Ambach <ambi@samba.org>
Autobuild-Date: Thu Dec 15 21:08:02 CET 2011 on sn-devel-104

s3-libsmb: Don't duplicate kerberos service tickets.

This fixes bug #8628.

Each time we do a client connection. Each time we call to function to
get the service ticket from the cache we duplicate it. So with each
connection we end up with one or three duplicated tickets.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Thu Dec 15 19:30:42 CET 2011 on sn-devel-104

s3-dbwrap: All relevant backends provide parse_record(). Remove the fallback.

Signed-off-by: Michael Adam <obnox@samba.org>
Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Thu Dec 15 17:41:53 CET 2011 on sn-devel-104

s3-dbwrap: Remove the "fetch" db_context callback

Signed-off-by: Michael Adam <obnox@samba.org>

s3-dbwrap: Rewrite dbwrap_fetch in terms of dbwrap_parse_record

Signed-off-by: Michael Adam <obnox@samba.org>

s3-dbwrap: Rewrite dbwrap_fallback_parse_record based on dbwrap_fetch_locked

This is in preparation to remove the db_context->fetch function pointer

Signed-off-by: Michael Adam <obnox@samba.org>

s3-dbwrap: For nostalgic reasons, make dbwrap_file.c compile at least

Signed-off-by: Michael Adam <obnox@samba.org>