s3:locking/proto.h: add _LOCKING_PROTO_H_ guard

s3:locking/proto.h: add GPL/Copyright header

s3:auth/proto.h: add _AUTH_PROTO_H_ guard

s3:auth/proto.h: add GPL/Copyright header

s3:passdb/proto.h: add _PASSDB_PROTO_H_ guard

s3:passdb/proto.h: add GPL/copyright comment header

s3-build Specify more of the smbconf dependencies

This brings more functions into util_names.c, and util_names.c into
PARAM_WITHOUT_REG_SRC.

This is not yet a full list, that would formalise the implicit
dependency loop.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue May 31 01:43:37 CEST 2011 on sn-devel-104

s3-build Make smbregistry depend on more of the subsystems it needs

This is not the full list (that can be seen by setting
allow_undefined_symbols=True).

Andrew Bartlett

s3-build Add util_sec subsystem

s3-build Move dbwrap_rbt into dbwrap_util subsystem

s3-build Create dbwrap_util subsystem

This contains the functions from dbwrap that don't require lp_
functions, and can therefore be put into a library (without dependency
loops).

Andrew Bartlett

s3-build: Rearrange build system to seperate out simple libraries

This will slowly allow us to develop a proper dependency tree without
interlibrary loops and unresolved symbols.

Andrew Bartlett

s3-build Add dependency on 'ldap' and 'ber'

These external libraries are required for the hooks in lib/ldap_debug_handler.c

s3-lib Move free_namearray() into it's own file

This makes it easier to have conn_smbd strictly depend on all it's
dependencies.

Andrew Bartlett

s3-lib Move realloc based string substitution functions out of util_str.c

This makes the dependency set for source3/lib/util_str.c simpiler,
which in turn makes it easier to build a dependency tree.

Andrew Bartlett

s3-lib Move sstring_sub() to it's only user and make static

This should not be used more generally, as it is specifically not for
multibyte strings, and uses malloc rather than talloc.

Andrew Bartlett

s3-param Move init_iconv() to loadparm.c

This assists with some dependency loops

Andrew Bartlett

libcli/security: move secdesc.c to the top level libcli/security

This code does not rely on lp_ or other source3 only functions, so can
be part of the common library.

Andrew Bartlett

s3-globals Remove smbd_event_context() (use server_event_context())

This has been a wrapper around server_event_context() for some time
now, and removing this from dummmysmbd.c assists with library
dependencies.

Andrew Bartlett

heimdal: Remove getprogname and setprogname from the heimdal import

heimdal_build: Don't use heimdal's getprogname() and setprogname()

Writing into an __progname variable spooks me, and if we use the local
variable, then we duplciate the system one, which fails SYMBOLCHECK

Andrew Bartlett

s3-param split service.c into param and smbd components

The dependency chain of find_service can't be satisfied sensibly
outside smbd, so don't include this in the main 'param' subsystem.

Also remove the duplicate find_service() and conn_snum_used() from
dummysmbd.c: The WAF build does not need these dummies any more, but
file.

Andrew Bartlett

s3-smbd Split conn.c into 3 files

The idea with this split is to make it easier to handle dependencies,
avoiding having the loadparm code depend on the global server
variables, without resorting to dummy functions and linker tricks.

conn_clear_vuid_cache() is brought in from uid.c to make it static

Andrew Bartlett

s3-build: Move user_util.c into it's own subsystem

s3-lib Move string_init functions into their own file

These have not been moved in common, as they are not talloc-based, but
it helps with dependencies if these are seperated from the rest of
util_str.c

Andrew Bartlett

s3-smbd provide struct smbd_server_connection * to conn_snum_used

This provides the 'sconn' parameter to this key functions, that
is currently duplicated in dummysmbd.c, which causes duplicate symbol
issues in the waf build.

This has natrually caused a number of consequential changes across the
codebase, includning not passing a messaging context into initial
reload_services():

This causes problems because the global smbd_server_connection isn't
yet set up, as there isn't a connection here, just the initial
process.

Andrew Bartlett

s3: Safely mark our sconn as smb2 if we have that protocol

Otherwise smbd will crash at an unclean exit. Without this conn_close_all will
do a close_cnum() on all connection_struct's. In smb2, those are talloc
children of the smbd_smb2_tcon's. sconn is talloc_free'ed after the
conn_close_all, but the smbd_smb2_tcon destructor will still reference
tcon->compat_conn, referencing then free'ed (and null'ed out) memory.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon May 30 22:49:53 CEST 2011 on sn-devel-104

Improve documentation for net rpc trust

Add man pages entries and fix usage output.

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon May 30 20:05:34 CEST 2011 on sn-devel-104

s3:net: registry export: close key after recursion returns

Signed-off-by: Michael Adam <obnox@samba.org>
Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Mon May 30 18:53:28 CEST 2011 on sn-devel-104

s3:net registry: polish output of net registry enumerate[_recursive]

so that net registry enumerate output is as before, and
net registry enumerate_recursive is formatted more nicely

s3:net: registry: use recursive implementation for enumerate

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

s3:net: registry: add new command enumerate_recursive

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

s4:torture: fix SEGFAULT in raw.acl.dynamic

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

nfs4_acls: pass ACE_FLAG_INHERITED_ACE down from the client

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

nfs4_acls: pass ACE_FLAG_INHERITED_ACE up to the client

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

s3:smbcacls: fix parsing of multiple flags

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

s3: fix some -Wunused-but-set-variable build warnings.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon May 30 17:41:18 CEST 2011 on sn-devel-104

s3: Add ioctl code to smbd_smb2_ioctl_send debug

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon May 30 12:47:27 CEST 2011 on sn-devel-104

replace: fix build issues on GNU Hurd (#7998)

Patch from Samuel Thibault <sthibault@debian.org> to fix Debian Bug 610678
resp. BSO #7998. IOV_MAX and UIO_MAXIOV are not defined on GNU Hurd.

Autobuild-User: Björn Jacke <bj@sernet.de>
Autobuild-Date: Mon May 30 00:53:59 CEST 2011 on sn-devel-104

s3: Remove unused cli_session_request

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun May 29 22:25:55 CEST 2011 on sn-devel-104

s3: Remove unused cli_connect

s3: Use cli_connect_nb in SMBC_server_internal

s3: Run BAD-NBT-SESSION in make test

s3: Reply correctly to a bad nbt session request

s3: Fix cli_bad_session_request

We expect a negative session response with a 0x82 error (called name not
present), not a 0x82 message

s3: Simplify run_bad_nbt_session

s3: Simplify cli_bad_session_request

For this purpose we don't need the full cli_state

s3: Add set_socket_options to cli_connect_sock()

A leftover of stuff that cli_connect() does

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun May 29 15:47:17 CEST 2011 on sn-devel-104

s3: Use cli_connect_nb in nmbd sync_child

s3: Remove unused attempt_netbios_session_request

s3: Use cli_connect_nb in do_connect

s3: Use the correct guest_login field in auth_server

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun May 29 13:57:21 CEST 2011 on sn-devel-104

s3: Extract the guest_login field in sesssetup

s3: Fix wct check in cli_sesssetup_blob_done

s3: Use cli_connect_nb in auth_server

s3: Fix a type-punned warning

s3: Check password server loop earlier

We do that in the loop for the ip address anyway

s3: Use cli_connect_nb in do_message_op

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun May 29 10:11:54 CEST 2011 on sn-devel-104

s3: Use cli_connect_nb in smbtorture open_nbt_connection

s3: Use cli_connect_nb in net_rpc_check

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat May 28 21:57:04 CEST 2011 on sn-devel-104

s3: Use cli_connect_nb in smbd_running()

s3: Use cli_connect_nb in cli_servertime

s3: Use cli_connect_nb in remote_password_change

s3: Add called name_type param to cli_connect_nb

s3: Fix a type-punned warning

s3: Use cli_connect_nb in masktest

s3: Fix some nonemtpy blank lines

s3: Use cli_connect_nb in locktest

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat May 28 18:06:49 CEST 2011 on sn-devel-104

s3: Fix smbsock_connect

If all connection attempts fail, return immediately. Plain bug.

s3: Fix a tiny memleak in copy_unix_token

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat May 28 11:47:11 CEST 2011 on sn-devel-104

s3: Use talloc_memdup in copy_unix_token

s3: Fix some nonempty blank lines

s3: Use cli_connect_nb in cli_start_connection

s3: Add cli_connect_nb

This builds up a cli_state until after the netbios session setup. It makes use
of smbsock_connect, so it connects to 139 and 445 simultaneously. This improves
the connection to Windows 2008 which does not listen on *SMBSERVER anymore.

packaging(RHEL-CTDB): align configure.rpm to the spec file

Signed-off-by: Michael Adam <obnox@samba.org>
Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Fri May 27 13:27:03 CEST 2011 on sn-devel-104

s3: Fix a typo

Thanks to Samba-JP oota <ribbon@samba.gr.jp>

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri May 27 10:12:12 CEST 2011 on sn-devel-104

s3: Fix a typo

Thanks to Samba-JP oota <ribbon@samba.gr.jp>

Fix bug #6911 - Kerberos authentication from vista to samba fails when security blob size is greater than 16 kB

We were not correctly checking the output of asn1_start_tag().
asn1_start_tag() returns -1 and sets data->has_error if the
remaining blob size is too short to contain the tag length.
We were checking data->has_error and returning NT_STATUS_OK
(to allow the second asn.1 parse to fail in that case). We
should not be checking data->has_error in this case, but
falling through to the code that already checks the length.

Thanks to Jim for reproducing this for me. We don't get bitten
by this as we announce a max buffer size of 16k, greater than
Windows's 4k, which means that most krb5 spnego packets already
fit.

Jeremy.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri May 27 02:57:27 CEST 2011 on sn-devel-104

s3: Document "async smb echo handler"

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu May 26 12:50:55 CEST 2011 on sn-devel-104

Fix bug #8165 - Inverted WITH_SYSLOG condition in vfs_full_audit.c.

Found by freeserif@yahoo.com.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed May 25 22:37:23 CEST 2011 on sn-devel-104

Fix numerous missing dependencies in WAF build scripts

With the recent consolidation of code between s3 and s4, a number of new
dependencies have been implicitly introduced.  For example, previous s3
code gained an implicit dependency on talloc after the charset related
consolidation (lib/util/charset/charset.h now includes talloc.h).  When
building against the embedded version of talloc this isn't a problem
since the paths are automatically added to the search path, but when
building against the external libraries build failures will occur for
all components that don't directly or indirectly include talloc as
a dependency.

Since charset.h is included from util.h, which in turn is included from
includes.h, this means most of the codebase (s3 and s4) has such an
undeclared dependency.

Therefore, samba-util-common and samba-util have been added as
dependencies to the s3 and s4 code respectively, for all cases where
the source would otherwise fail to build.  Additionally, a few other
dependencies are added in specific wscript_build files to address
similar dependency-related problems.

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

Signed-off-by: Sean Finney <seanius@seanius.net>
Signed-off-by: Matthias Dieter Wallnöfer <mdw@samba.org>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Wed May 25 19:22:13 CEST 2011 on sn-devel-104

s3-winbind: BUG 8166 - Don't lockout users when offline.

Windows does not track bad password attempts when offline.  We were locking users out but not honoring the lockout duration.

Autobuild-User: Jim McDonough <jmcd@samba.org>
Autobuild-Date: Wed May 25 18:11:10 CEST 2011 on sn-devel-104

s4:samldb LDB module - check if the RODC group exists if creating an RODC

Older AD deployments simply don't have it and hence there is no RODC
support.

Reviewed-by: abartlet
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Wed May 25 10:26:37 CEST 2011 on sn-devel-104

s4:samldb LDB module - better to call "samldb_prim_group_trigger"

"samldb_prim_group_trigger" which as a wrapper calls "samldb_prim_group_change"
for a LDB modify operation.

Reviewed-by: abartlet

s4:samldb LDB module - convert a "dsdb_module_search" into "dsdb_module_search_dn"

It saves us from checking the number of returned entries.

Reviewed-by: abartlet

s4:sam.py - uncomment/enhance some account type tests

Reviewed-by: abartlet

s4:samldb LDB modules - only objectClass "computer" is allowed to embed all types of account

Reviewed-by: abartlet

s4:sam.py - tests for "isCriticalSystemObject" attribute

Reviewed-by: abartlet

s4:samldb LDB module - fix "isCriticalSystemObject" behaviour

Tests against Windows Server show that it gets set to "FALSE" (not
deleted) if we change the account type to a domain member.

Reviewed-by: abartlet

s4:sam.py - unchanged "primaryGroupID" when account type remains the same

Enhance the testcase with a workstation example.

Reviewed-by: abartlet

s4:samldb LDB module - fix the behaviour when changing the "userAccountControl"

Ekacnet was not quite right yet but his patch made me think further.
This primary group changing is only needed if the account type changes.
With this patch we do one more search if the "userAccountControl"
changes but we save us from doing these unneeded and wrong modify replace
operations most of the time.

Reviewed-by: abartlet

s3-testparm Warn about incorrect use of 'password server'

This merges master with v3-6-test

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed May 25 05:20:57 CEST 2011 on sn-devel-104

s3-param Depricate 'password server = foo:12389' syntax

This was originally intended to allow the LDAP port on a DC to be
varied, but makes little sense to change one port when in an
environment where krb5, ldap, smb and potentially DCE/RPC over TCP are
involved.

Andrew Bartlett

docs: Rewrite 'password server' documentation

I think this new version is more clear.

Andrew Bartlett

s4-provision Use correct tkey-gssapi-credential

We changed to ${DNSNAME} (the fully qualified domain name) a while
back, and while it's usually functionally idential to the previous
setting, this breaks down if there is more than one DNS server.

Andrew Bartlett

docs: Clarify the 'security=server' fails for NTLMv2

selftest: Make knowfail/skip files consistent, always use ^prefix

Except in one case (where we mark printing tests as knownfail), this
has all our regular expressions start with ^, which ensures we don't
accidentially mix up the samba3/samba4 prefix here.

Because of the particular values in these files at the moment, this
should not change the set of tests, but it will help to ensure that
future edits follow the correct pattern.

Andrew Bartlett

Fix our asn.1 parser to handle negative numbers.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue May 24 22:57:16 CEST 2011 on sn-devel-104

lib/util Add Volker's asn1_Integer test into code that runs in 'make test'

The comfychair test harness isn't hooked up, and with the current
infrustructure C code is better tested directly here.

Andrew Bartlett

Actually make use of the SMBTA_SUBRELEASE define in smb_traffic_analyzer.h. This will allow to introduce new features or fixes into the protocol after the 3.6.0 release. The client software is designed to take care for the subrelease number.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue May 24 21:23:22 CEST 2011 on sn-devel-104