ECLIPSE + PyDev Project

s3-winreg: Fix _winreg_QueryValue crash bugs and implement windows behavior.

Found by RPC-WINREG smbtorture test.

Guenther

Explain why we don't use certain characters in the generated pw

lib/util - Removed curly braces from generate_random_password().

s4:provision - Updated FDS schema mapping.

s4:provision Improve the handling of provision errors

The backtraces were too confusing for our users, and didn't tell them
what to do to fix the problem.  By printing the string (rather than a
backtrace), and including in the error what to do, and what file to
remove, we give them a chance.

Andrew Bartlett

Try and fix bug #7233 - print fails with jobs >4GB from Win7 clients.

Remove an arbitrarty 4G B limit that *doesn't need to be there* !
Jeremy.

s4:libcli Use integrated name resolution when connecting SMB

This avoids pulling the address into a string and back again if given
a name, by letting the next async layer down do the name resolution.
If it was an IP address to start with, then the resolver library just
converts that to the struct socket_address.

Andrew Bartlett

s4:libcli/resovle File based lookup module for DNS name types

This uses the new common code to read a file containing DNS host
names, so we don't have to use real DNS lookups in our test
environment.

Andrew Bartlett

libcli/nbt Add parser for a 'hosts' file that takes DNS record types

s4:samba_dnsupdate Add a 'file based' mode to samba_dnsupdate

For the testsuite to use DNS like names, we need to write these names
to a file.

Also, to have this run in 'make test' the usual rules about 'no 127.*'
IP addresses in DNS must be skipped, so glue.interface_ips takes two
arguments now

s4:lib/socket Don't go via a string when resolving addresses in connect_multi

This also removes the special case for IP addresses, and leaves that
to the code in the resolver library.

Andrew Bartlett

s4:libcli/resolve Use a more robust way to return the string address

By going via these tevent functions, we avoid needing to dereference
the struct socket_address, which may contain a 'struct sockaddr' or
strings.  The new dns_host_file resolver returns in the form of a
struct sockaddr.

Andrew Bartlett

s4:lib/socket Add function to set a port on the socket address

Move prototype to header of common code for set_sockaddr_port

s4/rpc_server Don't segfault over replPropertyMetaData contents

The replPropertyMetaData may contain attrid values that we don't yet
have in the local schema.  We need to deal with this - it is a serious
error, but we should not segfault.

Andrew Bartlett

vfs_netatalk: Segfault if hide files or veto files has no ".AppleDouble"

s4-smbtorture: only look at DATA_BLOB pointer when einfo[0].data_length > 0.

Guenther

Added regression test for bug #7234 -  Symlink delete fails but incorrectly reports success.

Ensure we can delete a symlink by path.

Jeremy.

s4-smbtorture: while testing PrinterDataEx also compare Get and Enum results.

Guenther

Fix bug #7234 - Symlink delete fails but incorrectly reports success to client.

Typo called LSTAT instead of STAT in the unlink by pathname path.

Jeremy.

Revert "s4-smbtorture: disable winreg QueryValue test for today."

This reverts commit 420e3b8553c9e721fba27dd64eb78b2c7105ce64.

ѕ4: EDQUOT needs to be mapped to NT_STATUS_DISK_FULL

s4:libnet - change variables to "unsigned" where needed

s4:libnet_become_dc - change counter variables to "unsigned" where they count LDB objects

s4:winreg RPC - fix up the "QueryValue" call to work against the enhanced torture test

Found out by gd's updated torture test.

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>