s3: Fix smbcontrol smbd idmap kill S-1-5-21-...

The calls to sid_to_gid and sid_to_uid create id mapping entries themselves,
which makes it pretty difficult to reliably delete id mapping entries
everywhere just using a SID.

s3:idmap/autorid add a small alloc pool to autorid

this is needed to allocate gids for BUILTIN\Users and
BUILTIN\Administrators

gids are stored at the start of the first domain
as RIDs start with values over 500, we have some room there
so we do not need to allocate a range

s3: Fix bug 8334, do not fork the echo handler for smb2

If a smb1 negprot negotiated smb2 we forked the echo responder. This will
eventually lead to a panic from

[2011/08/30 10:33:29.212578,  0, pid=3846917] smbd/smb2_server.c:243(smbd_smb2_request_create)
  Invalid SMB packet: first request: 0x0009

because from the echo responder we always read using the normal smb1 protocol
handling routine. If that is a bit down the smb2 stream, we get a non-negprot
packet and panic.

BTW, the echo responder is not required for smb2 anyway, Microsoft confirmed
that it probes the server liveness using TCP keepalives and not smb2 echo
requests.

s3:smb2 fix Bug 8428 - wrong reply to DHnC (durable handle reconnect)

According to [MS-SMB2] 3.3.5.9.7
(http://msdn.microsoft.com/en-us/library/cc246784%28v=PROT.13%29.aspx),
smbd must reply with NT_STATUS_OBJECT_NAME_NOT_FOUND as it does not
support durable file-handles yet.

I have seen w2k8r2 running xcopy /C ending up in an endless loop
trying to get back the original file handle from smbd sending the same
requests over and over.

Metze, Jeremy, please check!

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Aug 30 22:20:36 CEST 2011 on sn-devel-104

s3: Use sys_write in fork_domain_child

Counterpart for last checkin. A lot less likely, but not impossible in a child.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Aug 26 13:14:27 CEST 2011 on sn-devel-104

s3: Use sys_read in fork_domain_child

I've seen

[2011/08/26 01:44:10.872057,  1] winbindd/winbindd_dual.c:1336(fork_domain_child)
  fork_domain_child: Could not read child status: nread=-1, error=Interrupted system call

on a customer box. Not good.

s3: Fix a winbind race leading to 100% CPU

This fixes a race condition that leads to the winbindd_children list becoming
corrupted. It happens when on a busy winbind SIGCHLD is a bit late.

Imagine a winbind with multiple requests in the queue for a single child. Child
dies, and before the SIGCHLD handler is called we find the socket to be dead.
wb_child_request_done is called, receiving an error from wb_simple_trans_recv.
It closes the socket. Then immediately the wb_child_request_trigger will do
another fork_domain_child before the signal handler is called. This means that
we do another fork_domain_child, we have child->sock==-1 at this point.
fork_domain_child will do a DLIST_ADD(winbindd_children, child) a second time
where the child is already part of that list. This corrupts the list. Then the
signal handler kicks in, spinning in

for (child = winbindd_children; child != NULL; child = child->next) {

forever. Not good. This patch makes sure that both conditions (sock==-1 and not
part of the list) for a winbindd_child struct match up.

s3:smb2 add some debugs

panic the process instead of returning so we get a backtrace
to help us analyse the intermittent problems

s3: Next step to fix MIT trusts

aa3f10c was not complete in the sense that it did not fully cover some
conditions that led to invalid domains in the winbind parent

s3: Increase a debug level in ads_find_dc

This message can happen with AD trusts that winbind can not cope with. The
message is not really clear and not worth spamming syslog always.

s3:lib change default share security access mask

when there is no share SD set, the default share SD that
is used e.g. for the output of sharesec -v defaults to a
value that is not equivalent to the desired FULL access.

This is a more or less a cosmetical follow-up for the patches
in Bug #8201 that makes them more consumeable by printing
FULL (that is what the user expects) instead of a bitmask
in hexadecimal form.

previous output:
REVISION:1
OWNER:(NULL SID)
GROUP:(NULL SID)
ACL:S-1-1-0:ALLOWED/0/0x101f01ff

with patch:
REVISION:1
OWNER:(NULL SID)
GROUP:(NULL SID)
ACL:S-1-1-0:ALLOWED/0/FULL

Autobuild-User: Christian Ambach <ambi@samba.org>
Autobuild-Date: Tue Jul 26 15:57:55 CEST 2011 on sn-devel-104

s3: Priorize the async echo responder over the client

Without this, an active client connection can starve the echo responder. This
leads to apparently "lost" SMBs.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Jul 28 18:53:38 CEST 2011 on sn-devel-104

tevent: Slightly simplify poll_event_loop_poll

No real code change. Do an early return instead of an if-statement, avoiding
one level of indentation.

s3: Remove unused smbd_echo_reader()

s3: Use smbd_echo_read_send in the async echo handler

s3: Add smbd_echo_read_send/recv

Read a SMB packet in the echo responder, giving the parent one second to step
in

Add wait_for_read_send/recv

Wait for readability of a socket as a tevent_req

s3:winbindd fix a return code check

talloc_traverse_dict will return with -1 in case of an error and
might return positive values that indicate the count of found
entries

s3:smbcontrol: let smbd pass the idmap msg to its children for convenience

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

s3:registry: fix the v2_to_v3 upgrade code so that it does not create value list security records for the INFO/version key ...

s3:registry: fix the v1_to_v2 upgrade code so that it does not normalize the INFO/version key ...

s3:registry: define a constant REGDB_VERSION_KEYNAME for the key INFO/version (and use it)

s3:registry: hand db context in to v2_to_v3 upgrade code via private_data

This also fixes the earlier faulty handing in of a useless stackframe
memory context via private_data.

s3:registry: fix broken use of dbwrap store record in v1_to_v2 upgrade code

This also fixes broken private_data parameter for the traverse function
(making use of it): Originally a memory context was handed in but was not used.

s3:smbcacls fix leaking talloc stackframce on tcon failure

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

s3:smbcacls fix possible SEGFAULT

sddl_encode returns NULL on failure

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

s3:smbcacls add option domain-sid

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

s3:smbcacls get_domain_sid for sddl parsing/formating from lsarpc

get_global_sid panics if we are not root and may give the wrong answer anyway.

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

s3:smbcacls pretty print acl control field

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

packaging(RHEL-CTDB): the ldbtools manpages have been removed. adapt the spec file

packaging(RHEL-CTDB): Revert "add BuildRequires to ctdb-devel >= 1.2.25"

This reverts commit a16520b6939cb6d87f5818db0ac3ded228053cee.

There is also a 1.0.114 based branch that has the required capabilities.
If building against an insufficient version, the configure will fail.

packaging(RHEL-CTDB): align configure.rpm to the spec file

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

packaging(RHEL-CTDB): honour rpm build target options handed in to makerpms.sh

This allows to call e.g. "makerpms.sh -bs" to only build the source rpm.

packaging(RHEL-CTDB): adapt configure.rpm

packaging(RHEL-CTDB): enforce building of libtdb and libtalloc

packaging(RHEL-CTDB): build libtdb and libtalloc packages ourselves

and add appropriate dependencies to the samba-common package.
It should also be possible to run with appropriate system talloc
and tdb packages.

packaging(RHEL-CTDB): update configure.rpm (--disable-smbtorture4)

packaging(RHEL-CTDB): disable smbtorture4 in the rpm build

packaging(RHEL-CTDB): Remove the packaging of the winbind-32bit compat package

packaging(RHEL-CTDB): create the rpm directories

In RHEL6, rpms are built in /root/rpmbuild, and those directories do not
necessarily exist.

s3:registry: enhance debugging of deletekey_recursive

s3:dbwrap_ctdb: improve transaction start/commit/cancel debugging

* also log nesting transaction start/commit/cancel
* unify transaction log messages slightly

s3:lib: fix a comment in tdb_unpack()

s3:registry: fix regdb_key_exists: the record has to contain at least the 4-byte subkey counter

More precisley, we return false if the record does not match the required
structure of a leading 4-byte subkey counter followed by the corresponding
number zero-terminated strings.

s3: avoid reading past the end of buffer in tdb_unpack 'f' if zero termination is missing

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

s3: avoid reading past the end of buffer in tdb_unpack 'P' if zero termination is missing

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

s3:build(autoconf): fix bug #8369: remove irritating "." targets when recent system libs exist

s3:idmap_tdb2: fix bug 8368 : correctly initialize "idmap config * : script" with NULL

this fixes the fallback to the deprecated spelling idmap:script

s3-net: Fixed typo in net conf

There was a missing ']' in net conf addshare usage message.

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

s3:smbconf let drop leave the smbconf key in place to match a virgin registry.tdb

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

s3:registry avoid updating keys which are going to be deleted in
reg_deletekey_recursive

this changes the complexity from O(n^2) to O(n) and reduces the time of
a 'net conf drop' with 10000 shares from 6min to 1.5s

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

s3:registry avoid pruning the sequencenumber while flushing the regsubkey_ctr

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri Jul 15 08:34:47 CEST 2011 on sn-devel-104
(cherry picked from commit 5049e3e142977a4c3d0f5a0fd9c06429f4d85bed)

s3:registry avoid leaking an old regsubkey_ctr on regsubkey_ctr_init

Signed-off-by: Stefan Metzmacher <metze@samba.org>
(cherry picked from commit 7e3d7505ddf2bdcf0d66273d9b96058969e2459b)

s3:registry: update copyright for the registry db implementation
(cherry picked from commit 8b37443c2ddc40e8a077c2d70867a2e82a5e19be)

s3:registry: remove unused function regdb_key_is_base_key()

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Mon Jul  4 21:11:32 CEST 2011 on sn-devel-104
(cherry picked from commit 399c0dba01b4337f5483bf03c99d28a3486e7981)

s3:registry: replace use of deprecated talloc_destroy by talloc_free.
(cherry picked from commit 86f8eb98fe76b4b65f903e1792cdb2eb9b195ee5)

s3:registry: remove superfluous check for success of talloc_stackframe()
(cherry picked from commit 17b091b7c7f57a041c5e90c653f1ee46572d4668)

s3:registry: add debug-level-10 verbosity to regdb_upgrade_v2_v3_fn()
(cherry picked from commit cc384c15969cdc395bfd3fcee10b5c04233df290)

s3:registry: adapt a debug message to be more systematic
(cherry picked from commit 340be35b9b068360b5fec5a0ea43dc8a3b294ba4)

s3:registry: set registry version to 3 and add upgrade code

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit c8c5d8c89e3cf9b71a917de83ad68d4d3c18b117)

s3:registry: add db_context argument to regdb_upgrade_v1_to_v2()
(cherry picked from commit f36ee63df28a2c49e8661a28237b97fe60f872c4)

s3:registry: wrap the whole db upgrade in one transaction

The purpose of this is to prepare for multiple upgrade steps
to be performed in a single transaction, so that no change
at all is made to that database if one step fails.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 1621aab139f0104cb2c7183c28c52cb6605abb09)

s3:registry: change regdb_store_regdb_version() to take db_context argument.
(cherry picked from commit caf9c9996952e72e61f9c9f0efd27936f38eefda)

s3:registry: fix a comment typo
(cherry picked from commit 2273650ca89c4042d073257383d0e5d28a8a555f)

s3:registry: remove unused regdb_delete_sorted_subkeys()

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit a8360712e1356a347f49a5208c1388dc5ab96d31)

s3:registry: don't delete the unused subkeys cache record any more

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit a0f83ecbb9697d4aa9c47b7b61001996a934ecee)

s3:registry: remove the code to create and search the sorted subkeys cache records

This is not needed any more.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>

s3:registry: adapt the comment explaining the definition of the existence of a key.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 18973fa109132bab8cb22a6757dbe4a206ea1a33)

s3:registry: regdb_key_exists now checks only for the list-of-subkeys-record

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit a25563035823e720a068c3021e01b0556feb7f18)

s3:registry: rewrite init_registry_key_internal

This rewrites init_registry_key_internal() to correctly also create
base keys. It makes use of the existing create functions instead of
duplicating the code.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 9e2b2c1fcc6d0bd852df715fdae7854b3ae77027)

s3:registry: add regdb_create_basekey()

Function to create a base key in the registry db, i.e. one that
has no parent key.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 747c67cf309a5127e78bd78f0d17e3fc094115d2)

s3:registry: add regdb_create_subkey_internal() taking additional db context argument

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 144c8d77d8dcde0b85cef28b6515360aa0c55b85)

s3:registry: add new mode to regdb_store_subkey_list() for handing in NULL parent key

Standard behaviour is to concatenate parent and key path (parent\key).
This new mode allows for storing subkey lists for base keys using a NULL
parent argument.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 15a50ef8f856d83508ece3a8294dfaffbadc3b41)

s3:registry: create the empty list of subkeys of the new key in regdb_create_subkey

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit fc4d79d41a18ed3c2a2f8a831e5dcc3002423ab1)

s3:registry: refactor creation of subkey list out into regdb_store_subkey_list()

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 322eaf1e9ef927bdec7fca947e894470196c1049)

s3:registry: remove redundant write of zero-lentgh subkey list record

In this case, the zero-lenght list has already been written in step #2.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 394ca1446e712c9cb3649c8a2c4fd2ebe638d8cd)

s3:registry: don't update the sorted subkeys cache any more in regdb_store_keys_internal2()

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit eb797bd2060732bca06cda3dfd5eabec5e7c8ab0)

s3:registry: don't skip base key in existence check in regdb_store_keys_internal()

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 5b01a34ad734a772e52c50ccd8ffbb461f986af7)

s3:registry: don't skip base keys from existence check in regdb_create_subkey()

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit da2e64b63f6bbb2faa93f621e14992e38aa37354)

s3:registry: don't skip base keys from existence check in regdb_delete_subkey()

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>
(cherry picked from commit 6b99aa496fe653f6bb7dcd7720f5a2c736779af6)

s3:registry: when deleting the records for a key, also delete the sorted subkeys cache

This prevents orphaned empty sorted subkeys cache records from filling the database.

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>

Signed-off-by: Michael Adam <obnox@samba.org>
(cherry picked from commit 1f72088633c2215a23e086c8627f35621b47ffec)

s3:registry: add helper function regdb_delete_sorted_subkeys()

Pair-Programmed-With: Gregor Beck <gbeck@sernet.de>

Signed-off-by: Michael Adam <obnox@samba.org>
(cherry picked from commit 2b162db6e9cc3e098f339516bbda719618664a83)

s3: Fix MIT trusts

Winbind can't really cope with trusts that don't have a SID associated. This
happens with external MIT trusts for example. This filters them out when
sending the trust list from child to parent.

s3:testparm: avoid spurious warnings about dos mode mapping and create mask if store dos attributes is set

s3: Fix the build, NAME_MAX not universally available

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Jun 20 13:45:21 CEST 2011 on sn-devel-104

s3: Replace shadow_copy2 with a new implementation

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Jun 20 11:17:47 CEST 2011 on sn-devel-104

s3: Fix the return of make_new_server_info_guest

s3: Add debug level 0 warnings to make_new_server_info_guest

I've got a backtrace where this must have failed, but it is not clear why. If
this fails, we should really complain because we can't start up.

s3: Fix connecting to ctdb as non-root

This is at least one instance which I could identify

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Tue Jun 14 19:06:39 CEST 2011 on sn-devel-104

s3: Fix count_current_connections

"False" is not really a valid return value for "int"..

s3:net: registry export: close key after recursion returns

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

s3: Fork the echo handler only after SMB1 negprot is done

This enables activating the echo responder also if SMB2 is enabled, albeit it
will only be used for SMB1 at this moment.

s3-torture: Add a test for notify upon read&x

This makes sure that when a file is brought online by a read call
we notify the client for FILE_NOTIFY_CHANGE_ATTRIBUTES.

s3:docs: document the smbclient "tid" command

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Mon Jul 25 13:14:57 CEST 2011 on sn-devel-104
(cherry picked from commit 233ba37416551a0b64dcfeb5cd405aadede5418c)

s3:docs: document the smbclient "tdis" command (cherry picked from commit a8d0e4ddb8730dc176e0589674ec4eaea307119a)

s3:docs: document the smbclient "tcon" command (cherry picked from commit 5ecfcf6cb998f5670895f998c2c917afc86aacfd)

s3:docs: document the smbclient "logoff" command (cherry picked from commit 9724fe303d5f9a3e37a881d025937285668e2dbe)

s3:smbclient: add a "tid" command to view and change the tree id (tid).

This low level command can be used to switch between tree connects in one
session.

s3:smbclient: add a "tdis" command to do a tree disconnect (close a connection to a share) (cherry picked from commit 7327bde81acecf4ae0c09f80cebe21a1312aeb3a)

s3:smbclient: add a "tcon" command to do a tree connect (connect to a share)