Revert "s4:dsdb - make the RELAX control private"

This must be available to the OpenLDAP backend, to set the GUID values
in some situations.  We need a proper ACL mechanism to control the use
or abuse of this control.

This reverts commit 10adee89367cee9add993869280542418fb3d370.

heimdal: Attempt to fix path of hdb_asn1.h

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Mon Oct 18 00:15:11 UTC 2010 on sn-devel-104

dsdb: Module aliases require to be prefixed with the subsystem name.

dcerpc_samr: Fix -l arguments for dcerpc_samr and dcerpc_atsvc.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sun Oct 17 19:15:43 UTC 2010 on sn-devel-104

tdb: Bump version to 1.2.7 after addition of pytdb.__version__.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sun Oct 17 18:23:16 UTC 2010 on sn-devel-104

heimdal: Add local simpler copy of to_list().

Revert "s4:remove "util_ldb" submodule and integrate the three gendb_* calls in "dsdb/common/util.c""

This reverts commit 8a2ce5c47cee499f90b125ebde83de5f9f1a9aa0.

Jelmer pointed out that these are also in use by other LDB databases - not only
SAMDB ones.

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sun Oct 17 13:37:16 UTC 2010 on sn-devel-104

s4:remove "util_ldb" submodule and integrate the three gendb_* calls in "dsdb/common/util.c"

They're only in use by SAMDB code.

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sun Oct 17 09:40:13 UTC 2010 on sn-devel-104

s4:lib/util/util_ldb.c - remove unused "gendb_add_ldif" call

s4:ldb-samba/ldb_wrap.c - fix indentation and trailing whitespaces

s4:"wrap_casefold" - move it to "ldb_wrap.c" since it's only used there

s4:dsdb - make the RELAX control private

This makes our LDAP much more secure and less error-prone.

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sat Oct 16 19:43:36 UTC 2010 on sn-devel-104

s4:libcli/ldap/ldap_controls.c - fix up the controls list

- add missing private controls and comments
- use control defines rather than hardcoded values -> easier to comprehend
- reorder controls

ldb:ldb.h - reorder controls/extended operations

This makes it easier to read

heimdal: Remove some unused arguments from HEIMDAL_SUBSYSTEM().

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sat Oct 16 16:14:24 UTC 2010 on sn-devel-104

heimdal: Allow leaving off prefix from asn1 option_file.

heimdal: Leave off ../heimdal prefix to HEIMDAL_BINARY().

heimdal: Allow leaving off prefix to HEIMDAL_ERRTABLE().

heimdal: Allow leaving off prefix for HEIMDAL_ASN1().

heimdal: Allow leaving off ../heimdal prefix for proto headers.

heimdal: Allow leaving off ../heimdal prefix for source files.

heimdal: Add convenience functions for finding heimdal paths.

heimdal: Call HEIMDAL_AUTOPROTO() manually.

heimdal: Call HEIMDAL_AUTOPROTO_PRIVATE() manually.

heimdal: Add variables for source files.

heimdal: Simplify handling of ../heimdal prefixes in HEIMDAL_SUBSYSTEM.

heimdal_build: Fix detection of system roken library.

heimdal_build: Remove ignored argument ldflags from HEIMDAL_BINARY().

ldb:pyldb.c - remove pointless comment

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sat Oct 16 14:38:58 UTC 2010 on sn-devel-104

ldb:"ldb_dn_compare_base" - use "unsigned int" counters

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sat Oct 16 13:54:13 UTC 2010 on sn-devel-104

ldb:"ldb_dn_add_child" - use "unsigned int" counters

ldb:"ldb_dn_remove_base_components" - use an "unsigned int" counter

ldb:"ldb_dn_canonical" - use an "unsigned int" counter

Convert it to use an "unsigned int" counter which represents the exact length
of the DN components.

s4:samldb LDB module - use appropriate fixed-length integer types where needed

- The "systemFlags" we interpret always as signed
- Use "samdb_msg_add_int" where possible (much saver for integer storing than
  ldb_msg_add_fmt)

s4:dsdb/samdb/ldb_modules/util.c - remove now unused functions which set integers

Please do always use the functions which specifiy the appropriate integer
length to not run into platform-specific issues. Therefore I'm removing these
generic calls.

s4:samba3sid LDB module - handle the RID as uint32_t

- This is how we always deal with RIDs
- Use an integer-length safe function for the RID update

s4:objectclass LDB module - implement the "isCriticalSystemObject" subtree delete protection

MS-ADTS 3.1.1.5.5.7.2

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sat Oct 16 11:24:09 UTC 2010 on sn-devel-104

s4:ldif_handlers.c - fix a typo

s4:dsdb - fix unsigned integer save problems using the "%u" specifier

The issue here is that we have not yet first cast to int32_t explicitly,
before we cast to an signed int to printf() into the %d or cast to a
int64_t before we then cast to a long long to printf into a %lld.

There are *no* unsigned integers in Active Directory LDAP, even the RID
allocations and ms-DS-Secondary-KrbTgt-Number are *signed* quantities.
(See the schema, and the syntax definitions in schema_syntax.c).

The failure has been detected by Matthieu Patou on the buildfarm host "tridge"
due to a malformed "groupType" attribute.

The solution is to use the "%d" specifier. Either to use it directly - or better
(when possible) use the call "samdb_msg_add_uint" (which encapsulates it).

This patch changes such problematic situations.

s4:dsdb/common/util.c - describe the behaviour of the functions "samdb_msg_add_uint", "samdb_msg_add_uint64" and "samdb_msg_set_uint" a bit more

Unsigned int data in AD is a bit problematic to handle. Problem described by
abartlet.

Don't arbitrarily clean all parametric options in add_a_service(),
that is called from many places, not just smb.conf processing. Only
clean parametric options when doing actual smb.conf reading (or
registry equivalent).

Michael Adams, Volker, Metze, please check.

Jeremy.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Oct 16 06:46:19 UTC 2010 on sn-devel-104

Ensure we have correct parameters to use Windows ACL modules.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Oct 16 03:36:04 UTC 2010 on sn-devel-104

heimdal_build: Support using system asn1 library.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sat Oct 16 02:14:32 UTC 2010 on sn-devel-104

heimdal_build: Skip some hx509 code when using system hx509.

heimdal_build: Add comment with stub for finding the system libtommath.

heimdal_build: Initial work on supporting an external heimdal library.

heimdal_build: Support using system hx509 library when explicitly
requested.

heimdal_build: Fix build with system roken.

heimdal_build: Put version objects in a separate subsystem.

heimdal: Support using system roken.

heimdal_build: Use existing functions for finding system libraries.

Add acl_xattr:ignore system acls boolean (normally false) to allow
Samba ACL module to ignore mapping to lower POSIX layer. With this
fix Samba 3.6.x now passes RAW-ACLs (with certain smb.conf parameters
set).

Jeremy.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Oct 16 01:26:31 UTC 2010 on sn-devel-104

Add make_default_filesystem_acl() function to be used in following change to acl_xattr and acl_tdb module.

Fix handling of "NULL" DACL. Map to u/g/w - rwx.

Jeremy.

Fix "force unknown ACL user" to strip out foreign SIDs from POSIX ACLs if they can't be mapped.

Add debug message to get_nt_acl_internal() to see what we got.

Fix valgrind "uninitialized read" error on "info" when returning !NT_STATUS_OK.

Jeremy.

Fix bug #7734 - When creating files with "inherit ACLs" set to true, we neglect to apply appropriate create masks.

Jeremy.

Fix bug #7733 - Invalid client DOS attributes on create can cause incorrect unix mode_t to be generated.

It turns out a client can send an NTCreateX call for a new file, but specify
FILE_ATTRIBUTE_DIRECTORY in the attribute list. Windows silently strips this,
but we don't - causing the unix_mode() function to go through the "mode bits
for new directory" codepath, instead of the "mode bits for new file" codepath.

Jeremy.

heimdal Add missing dependencies on wind.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Fri Oct 15 23:20:22 UTC 2010 on sn-devel-104

heimdal_build: Allow using system wind.

Please note that to use the system wind library the user explicitly has
to specify --bundled-library=!wind . Without that option we will always
use the included wind library.

s4:torture/local/dbspeed.c - use LDB result constant

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Fri Oct 15 19:11:32 UTC 2010 on sn-devel-104

s3-rpc_server: Normalize rpc_pipe_open_interface pipe name.

Autobuild-User: Andreas Schneider <asn@samba.org>
Autobuild-Date: Fri Oct 15 12:15:45 UTC 2010 on sn-devel-104

s3-rpc_server: Make auth_serversupplied_info const.

s3-winbind: Fixed the build of idmap_rid.

s4-test: fixed a typo in test_kinit.sh

too many Ts

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Fri Oct 15 10:14:27 UTC 2010 on sn-devel-104

s4-test: fixed test_kinit.sh time command test

passing -W breaks -k yes

s4-net: exit with a failure when a command is unknown

this ensures we don't pass bad tests in 'make test' if they call
unknown net commands

script: the --tests option has been replaced by --test-command

s4-finddcs: better debug messages to help track down DNS problems

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-net: fix the dependence on command line ordering

this fixes python net commands where you use a '-' option before the
command name

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4:samdb_msg_add_int* - use "ldb_msg_add_string" rather than "samdb_msg_add_string"

"ldb_msg_add_string" is safe here since the integer has already been converted
to a string which is "talloc"ed on "mem_ctx".

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Fri Oct 15 09:11:49 UTC 2010 on sn-devel-104

s4 param: change messaging directory name from messaging to msg

This saves 6 chars and as in some installation users tends to install
samba in directory with very long name they have a problem with the
sockets.
It will also ease the pain on buildfarms as we tend to have very long
environnement name (ie. vampire_dc) in path that are already quite long
leading also to errors on unix socket manipulation.

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Fri Oct 15 08:15:56 UTC 2010 on sn-devel-104

s4 dsdb: fix sign problem on PPC and x86

In LDAP we used signed intege and groups have the highest bit set (ie.
0x80000002). So it will result with values that are > 2^31 when these
value are used on some plateforms (x86 and PPC 64bits in this case) it
causes problem with strtol.

s4:lib/registry/util.c - cosmetic - fix indentation, trailing whitespaces

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Fri Oct 15 07:27:07 UTC 2010 on sn-devel-104

s4:lib/registry/util.c - strtoll call - better say explicitly that data is in hex format

Template is commit deebc934edb9a0a70e7615b1161d7a60fcb1d78f from Wilco.

s4:ntptr/ntptr_simple_ldb.c - use LDB result constants

s4:libnet_samsync_ldb.c - use LDB result constants

s4:lsa RPC server - use LDB result constant

s4:samldb LDB module - cosmetic - use "ldb" variable rather than "ldb_module_get_ctx"

s4:samdb_create_foreign_security_principal - proof error code of "samdb_msg_add_string"

s4:dsdb/common/util.c - samdb_msg_add_* calls - proof for more OOM conditions

s4:dsdb/common/util.c - samdb_msg_add_string - the attribute name doesn't need to be duplicated

This is done internally by the LDB library - look at "ldb_msg_add_empty".

s4:libds/common/flags.h - fix typo

s4:dsdb - remove "samdb_msg_add_value"

This can be substituted by "ldb_msg_add_value".

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri Oct 15 00:21:53 UTC 2010 on sn-devel-104

s4:dsdb - remove "samdb_result_uint", "samdb_result_int64", "samdb_result_uint64" and "samdb_result_string"

We have ldb_msg_find_attr_as_* calls which do exactly the same. Therefore this
reduces only code redundancies.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s3: Remove smbd_server_conn from construct_reply

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Oct 14 12:35:07 UTC 2010 on sn-devel-104

s3: Lift smbd_server_conn from receive_smb_talloc

s3: Lift smbd_server_conn from receive_smb_raw_talloc

s3: Lift smbd_server_conn from receive_smb_raw_talloc_partial_read

s3: Remove some explicit smbd_server_conn refs in process_smb()

s3: Rename "conn" to the more used "sconn" in process_smb()

s4:samldb LDB module - attempt to fix integer handling on big-endian platforms

And beside this it's also nicer to use standard LDB functions for type
conversions.

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Thu Oct 14 08:26:53 UTC 2010 on sn-devel-104

s4-waf: automatically remove stale C and header files

this prevents stale .c and .h files in bin/ from causing build
problems

Thanks to Thomas Nagy for the example implementation!

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Thu Oct 14 07:44:13 UTC 2010 on sn-devel-104

s4-heimdal: fixed some trailing commas in heimdal build

security: ensure the merge of libcli/security doesn't change s3 behaviour

Jeremy, you put a #if 0 around this logic in this commit:

  8344e945 (Jeremy Allison    2008-10-31 10:51:45 -0700 181)

is this still needed?

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Thu Oct 14 03:16:41 UTC 2010 on sn-devel-104

libcli/security Use static SIDs rather than parsing from strings

This should make the security_token_is_*() calls a little faster.

Andrew Bartlett

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

libcli/auth Merge source4/libcli/security and util_sid.c into the common code

This should ensure we only have one copy of these core functions
in the tree.

Andrew Bartlett

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

libcli/security Define traditional constants in terms of IDL macros

The source3/ code uses these constants in a lot of places, and it will
take time and care to rename them, if that is desired.  Linking the
macros here will at least allow common code to use the IDL based macros,
and preserve a documentary link between the constants (other than just their value)

Andrew Bartlett

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

libcli/security Move source3/lib/util_seaccess.c into the common code

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