s4:torture: add smb2.durable_open test reopen1

s4:libcli:smb2: don't allocate the smb2_tree struct earlier than necessary in smb2_connect()

Pair-Programmed-With: Stefan Metzmacher <metze@samba.org>

s4:libcli/torture:smb2: change smb2_tree_connect to take a smb2_session instead of smb2_tree arguement.

Only the tree->session is needed. This way it is less confusing.

Pair-Programmed-With: Stefan Metzmacher <metze@samba.org>

s4:torture:smb2: talloc_free the right tree structure in secondary_tcon()

Pair-Programmed-With: Stefan Metzmacher <metze@samba.org>

s4:torture:smb2: check for success of talloc in secondary_tcon()

Pair-Programmed-With: Stefan Metzmacher <metze@samba.org>

s4:torture:smb2:notify: remove an unused variable

s4:torture:smb2:notify: remove an unused variable

s4:torture:smb2: remove an unused variable from torture_smb2_setinfo()

loadparm: fix a load of uninitialized memory errors found by valgrind.

s3-nmbd: don't discard DGRAM frames from local addresses when we are a DC

Winbind use netbios to locate the DC, if samba is also a DC then nmbd
receive request from Winbind that are from a local address, those
requests must not be discarded otherwise winbind can't find the DC and
won't try another method.

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Fri Feb  3 11:09:29 CET 2012 on sn-devel-104

s3-pid: Catch with pid filename's change when config file is not smb.conf

This is what is done in pidfile_create

selftest: Fix the exit code of make test when run in buildfarm

When make test is run in buildfarm environment (RUN_FROM_BUILD_FARM=1),
last process in make test pipeline is subunit-filter. This subunit-filter
is responsible for converting failure test result to xfail for known
failures. While converting failures to xfail, it sets exitcode=1. If
this is the last process in the pipeline make test fails, which is
the observed behavior in buildfarm.

In autobuild, there is additional process in the pipeline format-subunit
which processes the output of subunit-filter and correctly terminates
with exit code=0. The buildfarm make test failure is fixed by adding
extra filter-subunit in the pipeline, as the extra filter-subunit
exits correctly with exit code=0.

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Fri Feb  3 07:00:04 CET 2012 on sn-devel-104

Only ask for specific permissions required when setting an ACL.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Feb  3 03:07:33 CET 2012 on sn-devel-104

Allow a SACL to be sent in cli_set_secdesc().

s3: Test for faulty xattr_tdb listxattr

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Feb  2 22:10:49 CET 2012 on sn-devel-104

s3-xattr_tdb: Fix listxattr

We have to tell the caller how many bytes we actually want

s3-vfstest: Add removexattr

s3-vfstest: Add setxattr

s3-vfstest: Add listxattr

s3-vfstest: Add getxattr

s3: Move stream_depot test to script/tests

s3: Use ARRAY_SIZE in bsd_attr_list

s3: Simplify streams_depot a bit

b0c0b949 is a bit involved. Passing down booleans is almost never
very clear

s3-vfstest: Fix an uninitialized read in close

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Feb  2 18:00:58 CET 2012 on sn-devel-104

s3-vfstest: Fix a double-free on closing a file

"mem_ctx" will be gone after we leave the routine

s3:auth: fill the sids array of the info3 in wbcAuthUserInfo_to_netr_SamInfo3() (bug #8739)

Originally, only the rid array was filled and foreign domain sids were omitted.

Pair-Programmed-With: Michael Adam <obnox@samba.org>

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Feb  2 12:59:32 CET 2012 on sn-devel-104

s3:auth: fix potential gap creation in wbcsids_to_samr_RidWithAttributeArray()

Pair-Programmed-With: Michael Adam <obnox@samba.org>

metze

s3-libsmb: Remove dead prototype.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Thu Feb  2 10:33:13 CET 2012 on sn-devel-104

build: fixed a link order problem

this fixes a problem found by obnox where the -L path for CUPS was put
before the path to internal libraries. The install path for CUPS
happened to be the same as for a old system libtevent, which meant we
linked against the old tevent instead of the correct one from our
private library paths.

The problem was that we were adding the -L paths directly to the
ldflags. The waf core code (in ccroot.py) only adds more paths if they
are not there already. So by adding it in ldflags it was not added at
the end of the list. The fix is just to not do the -L processing in
wafsamba and let the waf core do it in the right order

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Thu Feb  2 06:54:42 CET 2012 on sn-devel-104

replace: Only add bsd dependency when bsd library was found.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Thu Feb  2 05:18:45 CET 2012 on sn-devel-104

tdb2: add --disable-tdb2

I thought you could --disable-tdb2=false, apparently not!  Thanks
Michael Adam...

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-User: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-Date: Thu Feb  2 03:43:08 CET 2012 on sn-devel-104

smbtorture: Do not leak child processes when exiting

When some child processes could not be created, smbtorture would exit
and leave the successfully started child processes running.  Add a loop
to kill the existing child processes for this case.

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Thu Feb  2 01:35:27 CET 2012 on sn-devel-104

s3-libsmb: Remove dead gssapi code.

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Wed Feb  1 23:29:44 CET 2012 on sn-devel-104

s3-lib: Fix util_cmdline which doesn't use popt.

This creates its own header file for util_cmdline so it doesn't need to
link against popt.
This should fix linking on FreeBSD.

s3: Test for bug 8733

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Feb  1 21:53:50 CET 2012 on sn-devel-104

s3-vfstest: Correctly initialize the connection path

s3-vfstest: Fix some uninitialized vars

s3-vfstest: Fix some pointless statics

s3-vfstest: Fix a typo

s3-vfstest: Fix some nonblank line endings

s3:smbd:smb2: fix an assignment-instead-of-check bug conn_snum_used()

Accidential "=" instead of "==".

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Wed Feb  1 17:10:15 CET 2012 on sn-devel-104

Revert "s4-drs: do not try to contact for replication servers that are not anymore in reps*"

This reverts commit 5bfd6251eb22ff701184a95649822a73cf4d157b.

This change has been causing regular segfaults in the build farm since
it was applied. I also think it may be unnecessary as
dreplsrv_refresh_partitions() should already be achieving the same
thing (removing stale replication targets).

I think the segfaults were caused by freeing an in-flight DSA, but I
have been unable to reproduce it outside of the build farm

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Wed Feb  1 07:49:42 CET 2012 on sn-devel-104

s4-heimdal: the ASN1 compiler now depends on libreplace

it uses strlcat(), which we now get from libbsd if available

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Wed Feb  1 05:51:56 CET 2012 on sn-devel-104

s3-auth Remove unused nt_status_squash from auth_context

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed Feb  1 04:18:50 CET 2012 on sn-devel-104

selftest: Remove unused support for --exeext

s3/wscript: Fix dependency on gssapi, necessary when there is no system Kerberos.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Wed Feb  1 02:43:11 CET 2012 on sn-devel-104

source3/wscript: Expand tabs, to prevent confusion (whitespace is significant in python/waf).

replace: Add include for bsd/string.h.

s3: Add rmdir operation to streams_depot

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Feb  1 01:05:57 CET 2012 on sn-devel-104

s3: Delete streams on directories

s3: Simplify an if-statement

is_ntfs_default_stream_smb_fname already contains all checks that
the variable "stream_name" gives us here.

Fix bug #8729 - getpass regressions on Solaris/Illumos - 3.6 and 

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Jan 31 23:28:09 CET 2012 on sn-devel-104

s3:client: ignore SMBecho errors (the server may not support it) (bug #8139)

metze

Revert "Fix bug #8139 - smbclient fails if server does not support Echo request."

This reverts commit 1fdc96ecaff8ca12e9aa0082527468ad4242a8a9.

This is wrong in master as the error codes are different compared to
3.5.x. The correct way to handle this is to call cli_state_is_connected().

metze

s3:smbd: inline code in reply_sesssetup_and_X_spnego()

This makes the logic much easier to follow.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Jan 31 21:52:45 CET 2012 on sn-devel-104

s3:smbd: the spnego session setup don't need to copy the in blob

metze

s3:smbd: rework reply_spnego_ntlmssp to reply_spnego_generic

This removes the unused spnego_gen_auth_response() wrapping.

metze

s3:smbd: remove unused code from sesssetup.c

metze

s3:smbd: remove pending_auth_data logic

This is handled by the gensec_spnego module.

metze

s3:smbd: always use the gensec code path in sesssetup.c

The other code pathes are unused, because we always have
the spnego gensec module.

metze

s3:smbd: rework smbd_smb2_*_ntlmssp_auth* to smbd_smb2_auth_generic*

metze

s3:smbd: always use the gensec code path in smb2_sesssetup.c

The other code pathes are unused, because we always have
the spnego gensec module.

metze

libcli/smb: Convert struct smb_trans_enc_state to talloc

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

s3-libsmb: Remove unused enum smb_trans_enc_type

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

s3-libsmb: Use gensec_spnego in smb seal client

This is intead of the inline, manual spnego code currently
in use.

Andrew Bartlett

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

s3-smbd: Use gensec_spnego in smb seal server

This is instead of the inline, manual spnego code currently
in use.

Andrew Bartlett

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

s3:libsmb/auth_generic: make use of gensec_spnego in the client

metze

s3:auth/auth_generic: make use of gensec_spnego in the server

metze

s3: Unify stream testing in open_directory

The second if-statement could never have kicked in, assuming the
SMB_ASSERT above was right.

Jeremy, please check!

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Tue Jan 31 20:15:36 CET 2012 on sn-devel-104

s3: Re-indent an && sequence for better understanding

s3: Fix "smbcontrol smbd close-share"

s3:net ads join: remove a useless empty comment block

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Tue Jan 31 18:37:45 CET 2012 on sn-devel-104

s3:net ads join: add a comment for the call to _net_ads_join_dns_update()

s3:net ads join: reduce indentation in _net_ads_join_dns_updates()

s3:net ads join: move dns update code out to new function _net_ads_join_dns_updates()

s3:net ads join: improve comment for dns update block

s3:net ads join: improve status evaluation for call to net_update_dns()

untangle assignment from check and log error code in message if failed.

s3:net ads join: interpret return code of ads_kinit_password() in dns update block

If failed, print according error message and skip the attempt to do dns update.

s3:net ads join: check for malloc success and react accordingly in dns update block

s3:net ads join: check for success of fetching machine password in dns update block

log and cleanup accordingly if failed

s3:net ads join: untangle assignment from check, fix return code and improve error logging

only the dns update failed, not the join.
Also do proper memory cleanup

s3:net ads join: reduce indentation and improve logging in the dns update code block

by doing an early goto done upon error condition

s3:net ads join: untangle assignment from check.

s3:net registry: fix a copy and paste error in a help text

tevent: Fix python documentation strings

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Tue Jan 31 16:59:29 CET 2012 on sn-devel-104

replace: Only include C files from which there are actually functions used.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Tue Jan 31 15:21:04 CET 2012 on sn-devel-104

replace: use libbsd for strlcat/strlcpy when available.

s4-s3-upgrade: Check if there are duplicate sids for users and groups

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Tue Jan 31 02:23:17 CET 2012 on sn-devel-104

s4-s3-upgrade: Use lowercase hostname as hostname for provision

Fix bug #8139 - smbclient fails if server does not support Echo request.

Based on work by Matthias Scheler <tron@NetBSD.org>

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Jan 31 00:47:19 CET 2012 on sn-devel-104

s3-winbind: don't try to do clever thing if the username is not found while authenticating through winbind

This could cause that we authenticate a user with a bogus domain to
winbind's domain if the password supplied for the PAM_AUTH match.

The problem was reported by Jeff Venable (jvenable@juniper.net).
Patch from Andrew Bartlett (abartlett@samba.org).

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Mon Jan 30 18:58:12 CET 2012 on sn-devel-104

s3: check that a user in a bogus domain name is mapped to the localnetbios name of a domain member

This means that if we authentify for BOGUS\administrator in AD domain
FOREST with samba being domain member with the netbiosname MEMBER then
BOGUS\administrator will be mapped to MEMBER\administrator if the
password match.

s3-auth: Remove duplicate check for NT_STATUS_IS_OK(nt_status)

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Mon Jan 30 09:38:47 CET 2012 on sn-devel-104

gensec: inline gensec_generate_session_info() into only caller

This avoids casting to and from the struct auth_user_info_dc *user_info_dc

to to this, the

if (user_info_dc->info->authenticated)

is moved into auth_generate_session_info_wrapper(), which is the
function that gensec_security->auth_context->generate_session_info
points to.

Andrew Bartlett

s4-auth: Return NT_STATUS_NOT_IMPLEMENTED if the challenge cannot be obtained

auth: Make check_password and generate_session_info hook generic

gensec_ntlmssp does not need to know the internal form of the
struct user_info_dc or auth_serversupplied_info.  This will allow the
calling logic to be put in common.

Andrew Bartlett

tdb2: make --enable-tdb2 the default.

We still use the tdb1 on-disk format, but we do so via the tdb2 library.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-User: Rusty Russell <rusty@rustcorp.com.au>
Autobuild-Date: Mon Jan 30 08:02:43 CET 2012 on sn-devel-104

tdb2: add -1 and -2 options to tdbtorture

(For now, -1 is the default).

tdb2:tdbtorture: use TEST_DATA_PREFIX for files

TDB2 version of commit b83672b36c1ea8c35833c40c3919b63809f16624.

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

tdb2: name tools the same as TDB1 tools.

Otherwise, when we switch everyone's scripts will break (including our
own tests!).

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