s4:torture:smb2: fix a nasty double free error.

This error manifested itself in sporadic "talloc_free with references" error.

s4:torture:smb2: fix a comment

s4:torture:smb2: be leass leaky in wrap_simple_2smb2_test()

s3-ctdb: Fix ctdb_read_req

If a complete request has come in already before we consumed it, the
ctdb_packet_fd_read_sync will block indefinitely. So always try packet_handler
first and only if that fails due to insufficient data, read from the socket.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Oct 27 22:12:05 CEST 2011 on sn-devel-104

s3-ctdb: Add debug to ctdb_processes_exist

s3: Use serverids_exist in parse_share_modes

This is the main reason for the preceding commits. We need to reduce the number
of round-trips to ctdb when checking the locking record entries for existence.
Using the plural version of process_exists gets the number of round-trips to
ctdb for process_exists down to 1.

s3: Add serverids_exist

s3: Add processes_exist

s3-ctdb: Make ctdbd_process_exists use ctdbd_processes_exist

Not strictly necessary, but more code exercise is good

s3-ctdb: Add ctdb_processes_exist

This sends out a number of process_exists controls in parallel and collects the
replies as they come in.

s3-ctdb: Allow ctdb_read_req to read any reqid

s3-ctdb: Don't hand out 0 as reqid

0 will be used as a wildcard reqid in ctdb_read_req

s3: Use talloc_tos() in parse_share_modes()

s3: Fix some nonempty blank lines

s3: Fix some type-punned warnings

s4:wscript - install the two missing files "dlz_bind9.so" and "named.conf.dlz"

Signed-off-by: Matthias Dieter Wallnöfer <mdw@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Thu Oct 27 20:27:32 CEST 2011 on sn-devel-104

s4:repl_meta_data LDB module - don't intercept the partition LDB module referrals

Reviewed-by: abartlet

s4:repl_meta_data LDB module - rename operation - do not overwrite error messages

"ldb_error" could overwrite possibly useful error messages.

Reviewed-by: abartlet

s4:repl_meta_data LDB module - remove pointless debug messages

These are displayed when an object just doesn't exist!

Reviewed-by: abartlet

s4:repl_meta_data LDB module - always return the original LDB result codes on failure

And add "ldb_operr()" before the "return ret" to point out the position
where it failed (for "add_time_element" and "add_uint64_element")

Reworked after a suggestion by abartlet.

s4:repl_meta_data LDB module - "dsdb_search_module_dn" already checks if len(res) == 1

No need to perform an additional check here. As a return value we should
always give back the original error code and not generate a new one (to
let the caller know what is going on).

Reviewed-by: abartlet

s4:ldap.py - fix up the dSHeuristics test to check for the right behaviour

Reviewed-by: abartlet

s4:objectclass_attrs LDB module - implement the dSHeuristics length checks correctly

Consider bug #8489

Reviewed-by: abartlet

s4:ldap.py - we test the creation of secrets already in the "systemOnly" testcase

Reviewed-by: abartlet

s4:ldap.py - enhance and fix up the object class test

Also address the problem described in bug #8486.

Reviewed-by: abartlet

s4:objectclass LDB module - objectclass modify op. - remove superflous "talloc_strdup"

We are adding strings embedded in the schema structure which is basically
global and lives longer than the request - hence no duplication needed.

Reviewed-by: abartlet

s4:objectclass LDB module - objectclass modify operations

According to bug #8486 the modification to direct related structural
object classes is possible (equal, child, parent).

Reviewed-by: abartlet

s4:objectclass LDB module - forbid to add unrelated objectclasses

E.g. unsatisfied abstract objectclasses, additional top-most structural
classes

Reviewed-by: abartlet

s4:objectclass LDB module - "objectclass_add" - small optimisation

This saves us from doing one "ldb_msg_find_element".

Reviewed-by: abartlet

s4:objectclass LDB module - "check_rodc_ntdsdsa_add"

For convention use "ldb_attr_cmp()".

Reviewed-by: abartlet

s4:objectclass LDB module - update copyright

Reviewed-by: abartlet

s4:password_hash LDB module - fix compiler warning due to unsatisfied "switch"

Bail out on other LDB request types (only add and modify allowed).

s4:ldap.py - fix up the UTF8 tests

Reviewed-by: abartlet

s4:ldap.py - reactivate some assertions in "test_all"

There should always be one result on both s4 and Windows.

Reviewed-by: abartlet

s4:torture/ldap/basic.c - fix a typo

Reviewed-by: abartlet

ldb:common/ldb_modules.c - fix a typo in comment

Reviewed-by: abartlet

s4-torture: Add a user creation check.

This adds a patch which creates a non-privileged user and then tries to
create a new user as the non-privileged user.

Pretty simple test but it found #8509.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Thu Oct 27 18:51:14 CEST 2011 on sn-devel-104

s4-torture: Add a samr privilege check.

This is the first part which checks if a deleted user is propagated
correctly to all daemons and caches with user information are emtpy.

s4-torture: Add torture_delete_testuser().

s4-torture: Add domain_handle to test_join struct.

s3-build: Add .clang_complete.

s3:rpc_client: remove references to auth_ntlmssp_state

metze

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Thu Oct 27 16:40:15 CEST 2011 on sn-devel-104

s3: Enable uid wrapper.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Thu Oct 27 15:06:58 CEST 2011 on sn-devel-104

replace: Add don't include unistd.h directly and add uid_wrapper.

s3: Include uid_wrapper where it is missing.

Include uid_wrapper correctly.

s3-waf: Don't link uid_wrapper more than once.

The uid_wrapper is linked in globally by default if enabled.

nsswitch: Disable uid_wrapper in libwbclient.

This way we don't have to link against it in the autoconf build.

uid_wrapper: Add uwrap_setresgid().

uid_wrapper: Fix build on UNIX platforms.

uid_wrapper: Handle euid and egid 0.

uid_wrapper: Make sure we have initialized the wrapper.

uid_wrapper: Fix prototypes.

replace: Add checks for setreuid and setregid.

replace: Make sure we compile without uid_wrapper.

s3-netlogon: Fix setting the machinge account password.

This bug has been found with uid wrapper.

Factor out the code checking if a parent should override DELETE_ACCESS into a function.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Oct 26 23:15:05 CEST 2011 on sn-devel-104

Remove another level of indentation - deal with !NT_STATUS_OK individually.

Add early return on stat open without O_CREAT if file doesn't exist.
Reduces one level of indentation.

s3:libsmb: make use of map_nt_error_from_unix_common() in clitrans.c

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Oct 26 17:53:15 CEST 2011 on sn-devel-104

s3:libsmb: avoid using smb_flg2 in cli_trans.c

metze

s3:libsmb: replace  smb_size - 4 with MIN_SMB_SIZE in clitrans.c

metze

s3:libsmb: use 'tevent_' instead of 'event_' in clitrans.c

metze

s3:libsmb: make use of smb_buffer_oob() in clitrans.c

metze

s3:libsmb: use smb_len_nbt() in clitrans.c

metze

s3:include: implement trans_oob() as wrapper to smb_buffer_oob()

metze

libcli/smb: add smb_buffer_oob() helper

A copy of trans_oob().

metze

s3-netapi: remove pointless use_memory_krb5_ccache.

This breaks the ABI.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Oct 26 13:23:58 CEST 2011 on sn-devel-104

libcli/smb: merge NEGOTIATE_SECURITY_* flags to smb_constants.h

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Oct 26 11:27:12 CEST 2011 on sn-devel-104

s3:lib: make_unix_date3() is the same as pull_dos_date3()

Except for a 'void *' vs. 'uint8_t *'.

As a first step let make_unix_date() call pull_dos_date(),
so that we he the logic only once. We can fix the callers
later.

metze

s3:lib: make_unix_date2() is the same as pull_dos_date2()

Except for a 'void *' vs. 'uint8_t *'.

As a first step let make_unix_date() call pull_dos_date(),
so that we he the logic only once. We can fix the callers
later.

metze

s3:lib: make_unix_date() is the same as pull_dos_date()

Except for a 'void *' vs. 'uint8_t *'.

As a first step let make_unix_date() call pull_dos_date(),
so that we he the logic only once. We can fix the callers
later.

metze

s3-util: dbwrap_tool: fix fetch_string function

We do not want truncated strings.

Signed-off-by: Michael Adam <obnox@samba.org>
Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Wed Oct 26 04:17:05 CEST 2011 on sn-devel-104

s3:net registry check: fix a case where automatic and read only mode take the wrong action

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

s3:net registry check: do not silently replace separators

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

s4:torture:smb2: fix broken indentation/formatting in torture_smb2_connection()

this was not readable well

talloc: fix a comment typo

Second attempt to fix bug #8384 - Windows XP clients seem to crash smbd process every once in a while.

Don't just use the first entry in back_channels as a talloc context
to allocate a long-lived chan entry on - must be NULL. It's already
correctly deleted when the last reference goes away.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Oct 26 02:42:35 CEST 2011 on sn-devel-104

s3:smb2cli_base: make use of [_]smb_[set]len_tcp()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Oct 25 19:30:43 CEST 2011 on sn-devel-104

s3:smb2cli_base: make use of map_nt_error_from_unix_common()

metze

s3:lib/errmap_unix: map EPIPE to NT_STATUS_CONNECTION_DISCONNECTED

This lets map_nt_error_from_unix() return the same value as
map_nt_error_from_unix_common().

There are no callers relying on NT_STATUS_PIPE_BROKEN.

metze

s3:libsmb: s/event_context/tevent_context/ in async_smb.[ch]

metze

s3:lib/addrchange.h: include libreplace headers before others

Also we typically don't use an explicit "lib/replace/" prefix.

metze

build: always include tevent with <tevent.h>

This ensures we do the right thing if we are built against the system
tevent.

Andrew Bartlett

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

build: always include talloc with <talloc.h>

This ensures we do the right thing if we are built against the system
talloc.

Andrew Bartlett

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

torture: Remove unused smbiconv

smbiconv is not used in any test script, and has only had changes
since being introduced by Jelmer to keep it building.

It is also not installed on the system. I have also checked with
Jelmer, who approved the removal.

Andrew Bartlett

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

s3-libsmbclient: make some functions static in testsuite code.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue Oct 25 17:57:25 CEST 2011 on sn-devel-104

s3:libsmb: make use of map_nt_error_from_unix_common() in async_smb.c

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Oct 25 03:21:06 CEST 2011 on sn-devel-104

s3:libsmb: use HDR_* defines in cli_state_dispatch_smb1()

metze

s3:libsmb: make use of smb_[set]len_nbt() in async_smb.c

metze

s3:libsmb: use HDR_* defines in cli_pull_raw_error()

metze

libcli/smb: move smb_signing.[ch] to the toplevel

metze

s3:libsmb/smb_signing: make use of HDR_* defines

metze

s3:libsmb/smb_signing: make use of smb_len_nbt()

metze

s3:libsmb/smb_signing: avoid usage of smb_ss_field

metze

s4:libcli: rename smb_signing.h to smb_raw_signing.h in the install directory

metze

libcli/smb: remove unused includes from read_smb.h

metze

Try and fix bug #8384 - Windows XP clients seem to crash smbd process every once in a while.

Remove the copy of the binding handle from struct notify_back_channel, use
the direct pointer in struct rpc_pipe_client instead. Ensure we can't call
the functions with a NULL binding handle.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Oct 24 22:56:40 CEST 2011 on sn-devel-104

s3-docs: Remove -Y option from winbind manpage.

This option no longer exists.

Guenther

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

idl: Improve MS-PAC IDL

Change some misleading variable names to reflect the actual function.
Add missing field name/types previously marked as unkown.

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon Oct 24 19:19:28 CEST 2011 on sn-devel-104