s3:smb2_ioctl: Fix Coverity ID 701771 Uninitialized scalar variable

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon May 21 19:27:44 CEST 2012 on sn-devel-104

s4-dsdb: allow modification of some deleted object if the show-deleted control is presented

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Sat May 19 20:28:01 CEST 2012 on sn-devel-104

s4-dsdb: naming context needs to have the extended-dn syntax too

libcli: make it easier to understand that a control was not correctly encoded

Move the set_write_time() call to after get_existing_share_mode_lock() returns with a share mode.

get_existing_share_mode_lock() isn't really the right
call here, as we're being called after
close_remove_share_mode() inside close_normal_file()
so it's quite normal to not have an existing share
mode here. However, get_share_mode_lock() doesn't
work because that will create a new share mode if
one doesn't exist - so stick with this call (just
ignore any error we get if the share mode doesn't
exist.

The previous commit raised the error message debug
level inside get_share_mode_lock_internal() so
we don't always get a level 1 error message if
get_existing_share_mode_lock() fails.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat May 19 06:26:33 CEST 2012 on sn-devel-104

Raise the debug level from 1 to 5 in get_share_mode_lock_internal()

This isn't a fatal condition, there is a valid codepath
that can cause this message.

s4:torture: Add raw.session.reauth2 test

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri May 18 18:25:42 CEST 2012 on sn-devel-104

s4:torture: rename raw.session.reauth => raw.session.reauth1

metze

s3:smbd: allow creating new spnego sessions only with a 0 vuid

Found by the raw.context test.

metze

s3:smbd: SMBtdis should return ERRSRV, ERRinvnid instead of NETWORK_NAME_DELETED

Found by the raw.context test.

metze

selftest/Samba3: add 'smbd:suicide mode = yes'

metze

s3:idmap_cache: improve checks for format of value string in idmap_cache_find_sid2unixid()

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Fri May 18 16:34:27 CEST 2012 on sn-devel-104

s3:idmap_cache: add common exit point to idmap_cache_find_sid2unixid()

s3:idmap_cache: improve debug messages in idmap_cache_find_sid2unixid()

s3: Revert the serverid changes, they need more work

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Fri May 18 13:12:14 CEST 2012 on sn-devel-104

s4-torture: Improve torture test boilerplate, use torture_assert()

This ensures that if this fails, it is reported as a subunit error correctly.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri May 18 09:35:13 CEST 2012 on sn-devel-104

s4-torture: Always use torture_assert() to handle failures

s4-torture: provide correct torture failures for failure to connect

build: Also look for iconv in /usr/local by default

This should help the build find iconv on FreeBSD and similar systems,
and make it possible to operate with 8-bit character sets.

Andrew Bartlett

Add include/lib folders from the commandline

By using opt.add_option(..., match=['Checking for library iconv'], dest='iconvdir'),
all configuration tests displaying 'Checking for library iconv' will get
$(iconvdir)/lib and $(iconvdir)/include

s4-torture: Fix rpc.samr.priv test to use torture_assert() macros

autobuild: Run ABI check on samba4-libs as well.

Because this build is without the developer options (so as to check that we build without those)
we do not do an ABI check by default.

Andrew Bartlett

build: Move generated files out of the normal build tree

This avoids some dual-build-system interactions.

Andrew Bartlett

build: Move generated version.h to a waf-invisible location

Generally, it is better if our generated header files for the autoconf
build do not end up in places that the waf build can see.

Andrew Bartlett

pygensec: Fix init of variable if not specified.

Thanks to Wolfgang Sourdeau for reporting this.

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

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Fri May 18 04:50:17 CEST 2012 on sn-devel-104

Note that "keepalive" only applies to SMB1 connections.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri May 18 02:59:34 CEST 2012 on sn-devel-104

Fix the waf build with the new "cleans up stale processes" test.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri May 18 00:20:36 CEST 2012 on sn-devel-104

s3: Check for serverid_exists in close_directory

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Check for serverid_exists in close_remove_share_mode

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Be less picky on stale share mode entries

If a process died, the share mode entry might be bogus. Ignore those entries.

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Check for serverid_exists in find_oplock_types
Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Test whether get_share_mode_lock cleans up stale processes

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Check for serverid_exists in rename_share_filename

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Do not check the PIDs is parse_share_modes

We do that when conflicts arise

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Check for serverid_exists in smb_posix_unlink

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Check for serverid_exists in open_mode_check

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Check for serverid_exists in notify_deferred_opens

We will remove the check in parse_share_modes soon

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Add "share_mode_stale_server"

This is a helper routine that prunes a dead share mode entry on demand. This
prepares for removing the serverids_exist call in parse_share_modes.

Signed-off-by: Jeremy Allison <jra@samba.org>

s4:torture: add smb2.session.expire1

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu May 17 21:53:11 CEST 2012 on sn-devel-104

s4:torture: add smb2.session.reconnect2 test

This uses just one transport connection
and demonstrates the implicit logoff of the previous session.

metze

s4:torture: rename smb2.session.reconnect to smb2.session.reconnect1

metze

s4:torture: remove unused vars in smb2.session.reauth*

metze

s4:auth/gensec_gssapi: add "gensec_gssapi:requested_life_time" option

metze

s3:gse: implement gensec_gse_expire_time()

metze

s4:auth/gensec: implement gensec_gssapi_expire_time()

metze

auth/gensec: implement gensec_spnego_expire_time()

metze

auth/gensec: add gensec_expire_time()

metze

s4:auth/gensec_gssapi: add missing 'break' statements

metze

s4:auth/gensec_gssapi: remember the expire time

metze

s3:gse: remember the expire time

metze

s3:smb2_sesssetup: make use of nt_status_squash() in smbd_smb2_session_setup_recv()

metze

Check the return from create_acl_blob

Autobuild-User: Richard Sharpe <sharpe@samba.org>
Autobuild-Date: Thu May 17 07:17:29 CEST 2012 on sn-devel-104

s3-smbd: Avoid creating a UID ACL entry for SIDs that are mapped as ID_TYPE_BOTH The GID ACL entry is what will be mapped in most cases, and so is sufficient.

Andrew Bartlett

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu May 17 05:08:44 CEST 2012 on sn-devel-104

s3-smbd: Consider a group with the same SID as sufficient duplication

This code is to ensure that the user does not loose rights when their file
ownership is taken away.  If the owner (an IDMAP_BOTH SID) appears as a group
then a duplicate user is not required.

Signed-off-by: Jeremy Allison <jra@samba.org>

s3-smbd: Handle ID_TYPE_BOTH by mapping to both a group ACL entry and file ownership This will allow groups, such as domain administrators, to own files while correctly handling the rest of the ACL permissions.

Andrew Bartlett

Signed-off-by: Jeremy Allison <jra@samba.org>

We need to split things up into a new helper function add_current_ace_to_acl() in order for there to be more posix ACL elements than NT ACL elements (so a group SID can own a file, but also get the group permissions that will be honoured)

Andrew Bartlett

Slightly modified by Jeremy to reduce diff size.

Signed-off-by: Jeremy Allison <jra@samba.org>

This covers a case where an ID_TYPE_BOTH mapping creates group permissions, but must own the file. Based on an original patch by Andrew Bartlett.

s3-smbd: Do not merge UID ACE values with GID ACE values for posix ACL

This might happen when we get a SID mapped to IDMAP_BOTH.

Andrew Bartlett

Signed-off-by: Jeremy Allison <jra@samba.org>

s3:onefs: remove all onefs related code as it not maintained anymore

See https://lists.samba.org/archive/samba-technical/2012-May/083631.html
for the discussion.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed May 16 11:23:05 CEST 2012 on sn-devel-104

Fix the overwriting of errno before use in a DEBUG statement and use the return value from store_acl_blob_fsp rather than ignoring it.

Autobuild-User: Richard Sharpe <sharpe@samba.org>
Autobuild-Date: Wed May 16 03:43:41 CEST 2012 on sn-devel-104

build: Add explicit mention of --abi-check-disable to ABI checker

This information has always been in the linked wiki page, but put it
in the build to unblock developers using platforms with slightly
different GDB output.

We can also assist this by improving the string normalisiation in the
ABI checker when example errors are provided.

It is better to build with the waf build and the full testsuite than
to avoid the waf build or not to use the developer options simply to
skip the ABI checker.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed May 16 01:53:42 CEST 2012 on sn-devel-104

libwbclient: Fix an invalid free()

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Wed May 16 00:00:00 CEST 2012 on sn-devel-104

wbinfo: Fix Coverity ID 242684 Resource leak

wbinfo: Fix Coverity ID 242685 Resource leak

s3: Fix Coverity ID 242710 Untrusted pointer read

According to susv3 we have to make sure that we call isupper with
values only in the range of an unsigned char. This is best achieved
by automatic narrowing through assignment.

dbwrap: Fix Coverity ID 242750 Incorrect sizeof expression

Taking the size of "db" is correct, but a bit fishy. Silence Coverity.

libcli/smb: move smb2cli_ioctl.c from source3 to the toplevel

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue May 15 20:17:01 CEST 2012 on sn-devel-104

libcli/smb: move smb2cli_query_directory.c from source3 to the toplevel

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

libcli/smb: move smb2cli_query_info.c from source3 to the toplevel

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

libcli/smb: move smb2cli_set_info.c from source3 to the toplevel

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

libcli/smb: move smb2cli_flush.c from source3 to the toplevel

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

libcli/smb: move smb2cli_write.c from source3 to the toplevel

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

libcli/smb: move smb2cli_read.c from source3 to the toplevel

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

libcli/smb: move smb2cli_close.c from source3 to the toplevel

metze

libcli/smb: move smb2cli_create.c from source3 to the toplevel

metze

s3:libsmb: avoid interpret_long_date() in smb2cli_create

metze

s3:cli_np_tstream: include smbXcli_base.h, because we'll use functions from there

metze

s3: Remove some unused code

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Tue May 15 18:24:10 CEST 2012 on sn-devel-104

s3: Fix a likely cut&paste error

lib/ccan: add a missing dependency to 'execinfo' for 'backtrace()'

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue May 15 16:12:54 CEST 2012 on sn-devel-104

s3-auth: Rename to init_system_session_info().

Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Tue May 15 13:56:00 CEST 2012 on sn-devel-104

s3-auth: Don't lookup the system user in pdb.

This fixes bug #8944, ldapsam:trusted and ipasam. It is an additional
fix for bug #8567 (0528cb5f3a15b72dcb34ece21a3ffb3e7b8d6eb9).

Use ENC_ values for proto_tree_add_item() calls.

It appears that the integral value containing the bits in a bitmap should be
dissected according to the data representation, as the individual fields are
fetched by dissect_ndr_XXX, which is passed the data representation and
extracts the value appropriately for the data representation, so the patch
replaces TRUE (which meant "always little-endian" when the last argument to
proto_tree_add_item() was a byte-order Boolean) with DREP_ENC_INTEGER(drep)
(which means "ENC_BIG_ENDIAN or ENC_LITTLE_ENDIAN, depending on the data
representation").

For structures, I'm just using ENC_NA.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=8942
Signed-off-by: Jelmer Vernooij <jelmer@samba.org>
Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Tue May 15 12:07:06 CEST 2012 on sn-devel-104

s4:ntvfs/smb2: remove misleading comment regarding security=server

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue May 15 10:12:53 CEST 2012 on sn-devel-104

s4:ntvfs/cifs: remove misleading comment regarding security=server

metze

s4:auth: remove unused auth_server.c

metze

s3:auth: remove unused auth_server.c

metze

docs-xml: remove documentation of "SECURITY = SERVER"

metze

s3-auth: remove "security=server" (depricated since 3.6)

"security=server" has a lot of problems in the world with
modern security (ntlmv2 and krb5). It was also not very
reliable, as it needed a stable connection to the password
server for the lifetime of the whole client connection!

Please use "security=domain" or "security=ads" is you
authentication against remote servers (domain controllers).

metze
                       --------------
                      /              \
                     /      REST      \
                    /        IN        \
                   /       PEACE        \
                  /                      \
                  |      SEC_SERVER      |
                  |    security=server   |
                  |                      |
                  |                      |
                  |       12 May         |
                  |                      |
                  |        2012          |
                 *|     *  *  *          | *
        _________)/\\_//(\/(/\)/\//\/\///|_)_______

s4:param/tests: remove "security=server" test

metze

selftest: Remove tests for security=server

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

samba-upgradedns: Use the correct magic incantation of sys.path.insert()

This replaces "bin/python" with the correct path for python libraries. The
pattern requires double quotes (") instead of single quotes (').

Autobuild-User: Amitay Isaacs <amitay@samba.org>
Autobuild-Date: Tue May 15 05:19:46 CEST 2012 on sn-devel-104

s3: talloc_zero is not needed for structs we ndr_pull into

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Mon May 14 18:59:07 CEST 2012 on sn-devel-104

libcli/smb: move smb2cli_logoff_* from source3 to the top level

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon May 14 17:02:46 CEST 2012 on sn-devel-104

libcli/smb: git mv smb2_session.c smb2cli_session.c

metze

s3:libsmb: s/\<event_context/tevent_context/gc in smb2cli_write.c

metze

s3:libsmb: s/\<event_context/tevent_context/gc in smb2cli_session.c

metze

s3:libsmb: s/\<event_context/tevent_context/gc in smb2cli_read.c

metze

s3:libsmb: s/\<event_context/tevent_context/gc in smb2cli_query_directory.c

metze

s3:libsmb: s/\<event_context/tevent_context/gc in smb2cli_ioctl.c

metze