s3-passdb: split out passdb/pdb_nds.h.

Guenther

s3-passdb: split out passdb/pdb_ldap.h.

Guenther

s3-smbldap: rearrange prototypes a bit.

Guenther

ldb:ldb_autotransaction_request - error string shouldn't be set here

A generic error string should be set independently of the access
mechanism to an LDB operation. Hence it should be handled in "ldb_wait"
and "ldb_request"

Signed-off-by: Andrew Tridgell <tridge@samba.org>
Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Tue Oct 11 10:15:18 CEST 2011 on sn-devel-104

ldb:pyldb.c - py_ldb_* modification calls - error string shouldn't be set here

As discussed with Jelmer, we shouldn't be setting a generic LDB error string
only for the Python bindings alone. This should be done in "ldb_request"
and "ldb_wait" - the common place for all possible LDB access mechanisms.

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

s4:ldap_backend.c/"map_ldb_error" - handle errors similar to "PyErr_SetLdbError"

If the call was done using an error string ("add_err_string"), then use that one
without an additional "ldb_strerror()" for the definitive LDAP output.
Otherwise generate one using "ldb_strerror()".
This omits redundancies in the error string generation (twice the same
information by "ldb_strerror()").

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

ldb:ldb.c/"ldb_wait" - make "ldb_wait" always return an error string

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

ldb:ldb.c/"ldb_wait" - change "ldb_wait" slightly in order to introduce error messages

In this occasion remove a redundant check for "LDB_ASYNC_DONE":

if (handle->state == LDB_ASYNC_DONE ||    if (handle->state == LDB_ASYNC_DONE)
    handle->status != LDB_SUCCESS) {              return handle->status;
        return handle->status;         == if (handle->status != LDB_SUCCESS)
}                                                 return handle->status;
...                                       ...
return LDB_SUCCESS;                       return LDB_SUCCESS;

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

ldb:ldb.c/"ldb_request" - make "ldb_request" always return an error string

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

ldb:ldb.c - "ldb_set_errstring" can be implemented by using "ldb_asprintf_errstring"

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

ldb:ldb_autotransaction_request - fix a typo in a comment

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

gensec: trim header includes back to what is actually required

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Oct 11 06:13:08 CEST 2011 on sn-devel-104

auth/credentials Declare remaining functions are public interfaces and put into credentials.h

This is in preperation for this file being used by s3, and recognises that these are all
reasonable, public interfaces but were not declared as such in the past.

Andrew Bartlett

auth: move gensec_start.c to the top level

This does not change who uses gensec for now, but makes it possible to
write new gensec modules outside source4/

Andrew Bartlett

auth: move credentials layer to the top level

This will allow gensec_start.c to move to the top level.  This does not change
what code uses the cli_credentials code, but allows the gensec code to be
more broadly.

Andrew Bartlett

s4-s3-upgrade: Allow import (just without a uid mapping) where getpwnam fails

This allows the tests to pass on systems without a jelmer user :-)

Andrew Bartlett

lib/param Avoid dyn_ defines that are not provided by the autoconf build

The autoconf build will never use these parameters or load the
smb.conf with these defaults, so the defaults are not important.

Andrew Bartlett

lib/param Use strcasecmp_m rather than strcasecmp as this is banned in the s3 includes.h

lib/param Use talloc_strdup rather than strdup as strdup is banned in the s3 includes.h

lib/param Avoid the name string_set as this is already used in the s3 param code

lib/param Avoid the use of the name service_ok() which is used in the s3 param code

lib/param: Remove unused #include of lib/socket/socket.h

libcli/smb Move CSC_POLICY_* definition to smb_constants.h

This removes the duplicate definition between smb.h and lib/param/loadparm.c
which in turn allows this file to be compiled with the s3 includes.h

Andrew Bartlett

lib/param move source4 param code to the top level

This is done so that the lpcfg_ functions are available across the whole
build, either with the struct loadparm_context loaded from an smb.conf directly
or as a wrapper around the source3 param code.

This is not the final, merged loadparm, but simply one step to make
it easier to solve other problems while we make our slow progress
on this difficult problem.

Andrew Bartlett

pac: Fix wrong memory allocation check

Autobuild-User: Simo Sorce <idra@samba.org>
Autobuild-Date: Tue Oct 11 01:18:22 CEST 2011 on sn-devel-104

ndr.pc: needs samba-util not samba-hostconfig

libcli/cldap: don't pass tevent_context to cldap_socket_init()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Oct 10 23:23:07 CEST 2011 on sn-devel-104

libcli/cldap: make use of explicit tevent_contexts

metze

libcli/cldap: pass tevent_context to cldap_search_send()

metze

libcli/cldap: pass tevent_context to cldap_netlogon_send()

metze

libcli/cldap: pass tevent_context to cldap_set_incoming_handler()

metze

s3:smb2_create: fix allocation size return value when opening existing files

at least on GPFS, using the stat information gave wrong results
example: FileInfo gave 12582912000, Create Response gave 25769803776000

This makes the create part use the same method as fileinfo,
matching up the replies of both calls

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Oct 10 21:49:53 CEST 2011 on sn-devel-104

s3-passdb: split out passdb/pdb_smbpasswd.h.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon Oct 10 20:14:46 CEST 2011 on sn-devel-104

s3-passdb: split out passdb/pdb_wbc_sam.h.

Guenther

s3-passdb: split out passdb/pdb_tdb.h.

Guenther

s3-net: when all DNS servers fail to operate, the overal DNS registration result
must be a failure.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon Oct 10 15:59:20 CEST 2011 on sn-devel-104

ldb: change version to 1.1.3 after pyldb changes

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Oct 10 14:24:25 CEST 2011 on sn-devel-104

Remove pointless exception catching in tests.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sun Oct  9 00:00:26 CEST 2011 on sn-devel-104

netcmd/domain: Remove unused imports.

s3-docs: Fix typo.

Karolin

Autobuild-User: Karolin Seeger <kseeger@samba.org>
Autobuild-Date: Sat Oct  8 22:27:07 CEST 2011 on sn-devel-104

s3-docs: Fix bug #7908.

Fix typo.

Thanks to John Bradshaw for reporting!

Karolin

netcmd: Avoid catching all exceptions, avoid using ';' at the end of lines.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sat Oct  8 15:46:46 CEST 2011 on sn-devel-104

upgrade: Avoid catching all exceptions, just catch the ones we care about.

s4:findprovisionusnranges - the default python path is detected over "env"

Reviewed-by: Jelmer
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sat Oct  8 13:06:20 CEST 2011 on sn-devel-104

s4:upgrading-samba4.txt - fix mistakes reported by bug #8504

Reviewed-by: Jelmer

examples/VFS: add skel_get_dfs_referrals()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sat Oct  8 11:29:18 CEST 2011 on sn-devel-104

examples/VFS: fix names in skel_opaque.c

metze

s4:torture/smb2: avoid '//' comments

The compiler on openindiana doesn't like them.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sat Oct  8 08:56:10 CEST 2011 on sn-devel-104

s4:torture/drs: avoid '//' comments

The compiler on openindiana doesn't like them.

metze

s4:lib/wmi: avoid '//' comments

The compiler on openindiana doesn't like them.

metze

s3:utils: avoid '//' comments

The compiler on openindiana doesn't like them.

metze

s3:lib/netapi: avoid '//' comments

The compiler on openindiana doesn't like them.

metze

dnsserver.idl: avoid '//' comments

The compiler on openindiana doesn't like them.

metze

build: build wbinfo only once in the waf build

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sat Oct  8 04:52:03 CEST 2011 on sn-devel-104

build: remove double-setting of build_public_headers

The top level build already sets this, so we do not need to set it twice.

Andrew Bartlett

build: s3 public headers appear to be correct now

The build now passes without this setting, so remove it to ensure we
do this check in future.

Andrew Bartlett

selftest: run dfs tests on plugin_s4_dc

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sat Oct  8 03:18:54 CEST 2011 on sn-devel-104

s3:modules: add vfs_dfs_samba4

This module overloads the SMB_VFS_GET_DFS_REFERRAL() hooks,
in order to support the domain, dc and sysvol referrals
for a AD DC.

The config would look like this:

[IPC$]
    vfs objects = dfs_samba4

metze

s3:msdfs: removed unused variables

metze

s3:msdfs: implement setup_dfs_referral() on top of SMB_VFS_GET_DFS_REFERRALS()

metze

s3:vfs: add SMB_VFS_GET_DFS_REFERRAL() hooks

metze

s3:param: add lp_host_msdfs to loadparm_s3_context

metze

s4:smb_server: implement dfs referral handing on top of dfs_server_ad_get_referrals()

metze

dfs_server: add generic dfs_server_ad_get_referrals() call

This is a generic function to implement the domain, dc and sysvol
DFS referrals.

metze

examples/VFS: fix a typo in the README

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Sat Oct  8 01:42:33 CEST 2011 on sn-devel-104

Fix bug #8509 - Read-only handles on SAMR allow SAMR_DOMAIN_ACCESS_CREATE_USER.

Not a security issue as we also check inside _samr_CreateUser2.
Thanks to Andreas Schneider <asn@samba.org> for finding and testing this.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Oct  7 21:51:27 CEST 2011 on sn-devel-104

s4:dsdb/password_hash: add DSDB_CONTROL_PASSWORD_BYPASS_LAST_SET_OID

Which allows the caller to pass a given 'pwdLastSet' value
(every useful for migrations).

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri Oct  7 15:28:13 CEST 2011 on sn-devel-104

Revert "ldb: support raw OIDs in control string parsing"

This reverts commit ea41860d32d38448e08cefd79d30ee1150317a9e.

This is not needed, because we already have the 'local_oid' magic.

metze

s4:dbchecker: make use of local_oid controls for dsdb.DSDB_CONTROL_DBCHECK

metze

ldb/ldb_controls: allow oid up to 255 chars

We have char oid[256], so allow sscanf() to consume 255 chars.

metze

s4:dsdb: fix the order of DSDB_CONTROL_* defines in samdb.h

This makes clear that struct dsdb_control_password_change
belongs to DSDB_CONTROL_PASSWORD_CHANGE_OID.

metze

uid_wrapper: Add uwrap_setresuid().

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Fri Oct  7 12:04:05 CEST 2011 on sn-devel-104

uid_wrapper: Add uwrap_setregid().

uid_wrapper: Add uwrap_setreuid().

s4-param copy print_parameter and equal_parameter in from source3

This will allow these functions to be put into lib/param shortly.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri Oct  7 10:17:01 CEST 2011 on sn-devel-104

s3-param Remove odd formatting to allow code merge

This code is very old, and has some code styles that we have not used
in Samba for a very long time.  This fixes up a the extra braces around
return values.

Andrew Bartlett

s3-param: inline octal_string into caller to allow code merge

param: Use a bool to hold boolean parameters

All three-state parameters invoking Auto (2) are already declared as integers, not
booleans.

Andrew Bartlett

s3-param use Auto define for lm_announce default

ldb: use ldb directly rather than via a copy

This avoids needing to manually sync the two files, which due to the
top level build must be API compatible at all times anyway.

The most important recent change was:

commit e3b76bd6205acfc1a89fbcab5d9588b32cb47b88
Author: Andrew Tridgell <tridge@samba.org>
Date:   Thu Jul 28 15:51:31 2011 +1000

    ldb: fixed a search expression parse bug

However, as we always control the search expression in the callers to
this code, no backport to other releases is required.

Andrew Bartlett

build: ndrdump is only built with waf

We no longer need an #if (_SAMBA_BUILD >= 4) here.

Andrew Bartlett

build: Remove obsolete --enable/disable-merged-build

All waf builds build everything now.

Andrew Bartlett

build: Remove _SAMBA_WAF_BUILD_

The _SAMBA_BUILD_ macro can pick the difference between autoconf and waf builds now

Andrew Bartlett

autobuild: Remove s3-waf from autobuild

build: Reduce build systems to just top level waf and autoconf

The s3-waf build system is a key component of the top level build, but
with this commit is is no longer available directly.  This reduces the
number of build system combinations in master as we prepare for the
Samba 4.0 release.

Andrew Bartlett

s4-dsdb: special case for deleted objects one way link

we show wellknown links to the deleted objects container

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

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Fri Oct  7 07:58:08 CEST 2011 on sn-devel-104

s4-dsdb: don't display links to deleted objects

unless the user asks for the display of deactivated links, we should
not display DNs that link to deleted objects

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

s4-dsdb: fixed one_way_link calculation

we need to check for the other end of the link, not the current linkID

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

s4-dsdb: fixed behaviour of show_deleted and show_recycled control

to correctly implement the show_deleted and show_recycled control we
need to know if the recyclebin is enabled. When not enabled, the
isRecycled attribute is ignored, and only isDeleted is used.

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

s4-dsdb: fixed the check_optional_feature() call

the dsdb_check_optional_feature() call should look on our own NTDS DN
for the enabled feature. This should work for all features, not just
for forest wide fetaures.

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

s3:lib: remove unused/empty modules.c

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Oct  6 15:23:45 CEST 2011 on sn-devel-104

selftest/Samba3: enable uid wrapper in nmbd, winbindd and smbd

metze

Signed-off-by: Andreas Schneider <asn@samba.org>
Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Thu Oct  6 13:49:05 CEST 2011 on sn-devel-104

s3:configure.developer: add --enable-uid-wrapper

metze

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

s3: Use the uid_wrapper

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

uid_wrapper: We have talloc_array_length, no need for an explicit length

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

lib/util: consolidate module loading

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Oct  6 08:52:30 CEST 2011 on sn-devel-104

modules: standardise on samba_init_module as the hook symbol to resolve

lib/util: Use only init_module_fn typedef in module loading

lib/util: Remove unused module loading functions

lib/util: consolidate module loading into common code

This creates a samba-modules private libary that handles the details.

Andrew Bartlett