s4:registry/ldb.c - if "name" isn't set we should return WERR_INVALID_PARAM

nsswitch: build libnss_winbind.so with SOVERSION = 2

metze

nsswitch: don't use socket_wrapper for winbind client code

metze

nsswitch: build libnss_winbindd.so only for the non merged build

metze

s4:build/m4/public.m4: add realname argument to SMB_BUILD_LIBRARY()

metze

s4:rpc_server/wkssvc: don't reset [out,ref] pointer to 0

r->out.total_entries = 0; should be *r->out.total_entries = 0;

Otherwise we fail to marshall the reponse or crash if we run with
log level = 10 and trigger the ndr_print functions.

All out elements are already setup and initialized by the pidl generated code.

metze

s4:rpc_server/wkssvc: remove unneeded talloc_reference()

metze

s4:rpc_server/wkssvc: avoid ZERO_STRUCT(r->out) and use already allocated [out,ref] memory

metze

s4:rpc_server/srvsvc: remove unused ZERO_STRUCT(r->out);

Here it doesn't matter, but it's a bad example for other code
as it might reset the [out,ref] pointers which are already
generated by the pidl generated code.

metze

idl: Regenerate DRSUAPI idl files

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

s4/idl: drsuapi_QuerySitesByCostCtr1 'unknown' member is now known

and this it is a reserved fields for future flags - 'flags_reserver'
Must be 0 and is currently ignored.

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

s4/idl: QuerySitesByCost - switch types should be unsigned

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

s4/idl: DsGetMemberships2 - switch types should be unsigned

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

s4/drs: DsRemoveDSServer - Propagating IDL changes to source code

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

s4/idl: DsRemoveDSServer - switch types should be unsigned

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

s4/drs: DsWriteAccountSpn - Propagating IDL changes to source code

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

s4/idl: DsWriteAccountSpn - switch types should be unsigned

Changes also propagated into source code as they are
too minor to be done in separate commit

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

s3/drs: DsCrackNames - Propagating IDL changes to source code

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

s4/drs: DsCrackNames - Propagating IDL changes to source code

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

s4/idl: DsCrackNames - switch types should be unsigned

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

s4/idl: DsGetMemberships - switch types should be unsigned

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

s4/idl: DsReplicaMod - switch types should be unsigned

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

s4/idl: DsReplicaDel - switch types should be unsigned

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

s4/idl: DsReplicaAdd - switch types should be unsigned

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

s4/idl: DsReplicaUpdateRefs - switch types should be unsigned

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

s3/drs: DsGetNCChanges - Propagating IDL changes to source code

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

s4/drs: DsGetNCChanges - Propagating IDL changes to source code

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

s4/idl: DsGetNCChanges - switch types should be unsigned

Converting those types to unsigned should help in
creating DRSUAPI dissector for Wireshark also.

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

s4/idl: drsuapi_DsReplicaSyncOptions flags are no more used

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

doxygen: talloc_get_type name fixed in Doxygen documentation

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

s3-winreg: add some debug statements to _winreg_QueryValue().

Guenther

s4-smbtorture: do full winreg QueryValue test for values we created.

Guenther

s4-smbtorture: check for non existing value under version key in RPC-WINREG.

Guenther

s3: Fix a long-standing problem with recycled PIDs

When a samba server process dies hard, it has no chance to clean up its entries
in locking.tdb, brlock.tdb, connections.tdb and sessionid.tdb.

For locking.tdb and brlock.tdb Samba is robust by checking every time we read
an entry from the database if the corresponding process still exists. If it
does not exist anymore, the entry is deleted. This is not 100% failsafe though:
On systems with a limited PID space there is a non-zero chance that between the
smbd's death and the fresh access, the PID is recycled by another long-running
process. This renders all files that had been locked by the killed smbd
potentially unusable until the new process also dies.

This patch is supposed to fix the problem the following way: Every process ID
in every database is augmented by a random 64-bit number that is stored in a
serverid.tdb. Whenever we need to check if a process still exists we know its
PID and the 64-bit number. We look up the PID in serverid.tdb and compare the
64-bit number. If it's the same, the process still is a valid smbd holding the
lock. If it is different, a new smbd has taken over.

I believe this is safe against an smbd that has died hard and the PID has been
taken over by a non-samba process. This process would not have registered
itself with a fresh 64-bit number in serverid.tdb, so the old one still exists
in serverid.tdb. We protect against this case by the parent smbd taking care of
deregistering PIDs from serverid.tdb and the fact that serverid.tdb is
CLEAR_IF_FIRST.

CLEAR_IF_FIRST does not work in a cluster, so the automatic cleanup does not
work when all smbds are restarted. For this, "net serverid wipe" has to be run
before smbd starts up. As a convenience, "net serverid wipedbs" also cleans up
sessionid.tdb and connections.tdb.

While there, this also cleans up overloading connections.tdb with all the
process entries just for messaging_send_all().

Volker

s3: Make TLDAP_IS_ALPHA and TLDAP_IS_ADH static functions

s3-passdb: Fix typo in debug message.

Karolin

s4-spoolss: remove unsed iconv handle from dcesrv_spoolss_GetPrinterData().

Guenther

s4:winreg RPC - don't crash when incoming data wasn't correctly specified

Also found by the WINREG torture test enhancements by gd.

s4:lib/registry/ldb.c - fix up registry backend to be more robust

This should let the new WINREG tests written by gd at least pass against us.

Fix the shell script in the root case. When run as root, make test now detects CAP_DAC_OVERRIDE being left on in error.

Jeremy.

s4-smbtorture: disable winreg QueryValue test for today.

wow, both s3 and s4 crash on full coverage winreg QueryValue testing.

Guenther

s4-smbtorture: fix some build warnings in RPC-SPOOLSS test.

Guenther

Add tests which, when run as root, will ensure we can't write
into a read-only directory, or read a owner-read-only file.

Jeremy.

s4-smbtorture: on HKLM hive test the well known CurrentVersion value.

Guenther

s4-smbtorture: add full coverage test for winreg QueryValue calls.

Guenther

s4-smbtorture: rework test_winreg_QueryValue in RPC-SPOOLSS-PRINTER once again.

Guenther

s4-smbtorture: add tests for set and delete value in RPC-WINREG.

Guenther

Fix typo and convert spaces to tabs

Fix typo

Allow "make test" to complete as root. Obviously only safe on tightly
controlled developer machines.
Jeremy.

s3: Fix a NULL pointer dereference

Found by Laurent Gaffie <laurent.gaffie@gmail.com>.

Thanks!

Volker

s4:ldb fix escape parsing

sscanf can return also on short reads, in this case an invalid escape
sequence like '\1k' would be accepted, returning 1 as value and swallowing the
'k'. Use an auxiliar function to validate and convert hex escapes.

s3:tldap add own filter parsing

Also add torture test to check filter parsing.

s4:winbind - use "unsigned" variables where possible

s4:winbind/wb_cmd_getgroups.c - fix up warnings

Also fix some indentations.

s4:unittest Fix unittest to reflect that wbinfo -r no longer fail

Signed-off-by: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>

s4:winbind: stub implementation of WINBINDD_PAM_LOGOFF

Signed-off-by: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>

s4:winbind: Fix a misplaced returned info

libwbclient expect to have in auth.exra_data the INFO3_TXT and in auth.unix_username the username

Signed-off-by: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>

s4:winbind Implement logic for getgroups to work

This function is called by the system everytime we do a id user or when we do wbinfo -r

Signed-off-by: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>

s4:winbind: implement calls for allowing getent groups

This is to say getgrent and setgrent, and the associated technical objects (states, build directives,...) needed.

Signed-off-by: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>

s4:torture/rpc/netlogon.c - "LogonGetDomainInfo" test - make it compatible against Windows Server 2008

This is a reworked version of the mentioned test which passes against Windows
Server 2008. The previous version, also mainly written by me passed only against
Windows Server <= 2003.

s4:netlogon RPC - "LogonGetDomainInfo" - make the call compatible with >= Windows 2008

Add more security checks and other corrections to imitate Windows Server >= 2008.

libcli/auth/schannel_state_tdb.c - fix an obviously wrong error handling

s4-smbtorture: fix uninitialized variable in winreg QueryValue call.

Guenther

s3: Fix a typo. Thanks to Christian Ambach for pointing me at it :-)

A helper function to get the Infrastructure DN.

Fixed a bug in acl tests - python error when we create user/group/ou with a descriptor.

Added a check for permissions to modify the RDN attribute on rename.

Necessary because rdn module will be moved lower than acl in the stack.

s4:dsdb/dns: change callers of samba_runcmd()

metze

lib/util: change samba_runcmd() to use tevent_req _send/_recv

metze

Revert "Fix bug #7067 - Linux asynchronous IO (aio) can cause smbd to fail to respond to a read or write."

This reverts commit a6ae7a552f851a399991262377cc0e062e40ac20.

This fixes bug #7222 (All users have full rigths on all shares) (CVE-2010-0728).
(cherry picked from commit 1c9494c76cc9686c61e0966f38528d3318f3176f)

s3:build: Fix automatic building of vfs_tsmsm if gpfs and dmapi are present.

Michael

s3:release-scripts: fix create-tarball to treat vendor patch level correctly

samba: remove Linux cifs-utils files from samba master branch

This patch removes all of the files from the samba tree that should now
be provided by the cifs-utils package. It also drops a
"README.cifs-utils" into the topdir with a URL to the main cifs-utils
webpage. This is for people who don't want the lists and might be taken
by surprise by the change. That's optional, but I think it's a good idea
for a least a release or two.

Signed-off-by: Jeff Layton <jlayton@samba.org>

s4:"dns_update_list" file: install it properly into the private directory

This is what the "samba_dnsupdate" script requests (line 220).

Revert "s4:script/installmisc.sh - install "dns_update_list" to target setup folder"

This reverts commit b49276e291274652d46eed39249c07531e32b591.

LDB:asq module - change counters to "unsigned" where appropriate

LDB:sort module - change counters to "unsigned" where appropriate

LDB:rdn name module - change counters to "unsigned" where appropriate

LDB:paged searches module - change counters to "unsigned" where appropriate

LDB:paged results module - change counters to "unsigned" where appropriate

s3: add missing Makefile.in changes for vfs_crossrename

s3: add man page for vfs_crossrename

s3: add vfs_crossrename

this module adds optional server-side support for limited rename operations
beyond filesystem boundaries, which was the previously the default.

s3: remove cross-device rename support from vfs_default

cross-device rename support has some major limitations:

- on huge files clients will timeout or hang
- ACLs and EA information is not retained

Usually a client will have to handle this. A Windows Server with a reparse
point will also just return NT_STATUS_NOT_SAME_DEVICE. We will now by default
do the same.

I will add a vfs module which will restore the old cross-device renames.

samba: remove cifs-utils tools from build systems

Now that cifs-utils are their own project, we need to go ahead and pull
them out of the samba tree. This patch represents the first step toward
that end.

Remove the cifs-utilities from the source3 and source4 builds. Please
pay particular attention to the source4 piece. I'm not at all familiar
with the build system there, and would appreciate someone sanity
checking my changes.

It also adds a small README.cifs-utils file in the topdir. This is
optional, but I think it's a good idea to carry this for a release or
two.

Once this patch looks ok, I'll plan to do another patch to actually
remove the client dir and the relevant docs-xml files from the tree
altogether.

Signed-off-by: Jeff Layton <jlayton@redhat.com>

LDB:common - Change counters to "unsigned" where appropriate

To count LDB objects use variables of type "unsigned (int)" or "long long int"
on binary or downto searches.

To count characters in strings use "size_t".

To calculate differences between pointers use "ptrdiff_t".

s4:LDB TDB index code - reintroduce accidentally removed code part

This was removed by 95d726f3018ef5d249f89d56bde24b7ee0c24ecf. Sorry.

s4:ldb_ldap.c - fix indentation

LDB:TDB backend - change counter variables to "unsigned" where appropriate

LDB:SQLITE3 backend - change counter variables to "unsigned" where appropriate

LDB:LDAP backend - change a counter variable to "unsigned"

LDB:map - make LDB "signed-safe" on counter variables where appropriate

s4:ldif_handlers - Change "unsigned int" to "uint32_t" which fits better here

s4-gensec: Fixed wrong usage of error_string.

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

s4:extended_dn_out LDB module - change counter variables to "unsigned" where appropriate

s4:repl_meta_data LDB module - change counter variables to "unsigned" where appropriate

I used "unsigned int" counters where we count LDB objects (LDB specification
prescribes to use "unsigned" index variables).
But on DSDB replication object counters I used "uint32_t" typed variables as it
is suggested.
If a counter variable counts both types of objects I used "unsigned int" since
size(unsigned int) >= size(uint32_t), but on most platforms equal.

s4:local_password LDB module - change counter variables to "unsigned" where appropriate

s4:ranged_results LDB module - change counter variables to "unsigned" where appropriate

s4:objectguid LDB module - change counter variables to "unsigned" where appropriate