Moved access_check_on_dn from acl module as an utility.

Made this an utility function so it can be used for access checking
outside of the acl ldb module, such as checking validated writes and
control access rights in other protocols (e. g drs)

s3-winreg: make QueryValue pass RPC-WINREG test again.

Guenther

s4-smbtorture: skip extended SetValue test against Samba (both dont survive).

Guenther

s4-smbtorture: test NULL and "" valuenames as well in query for wellknown winreg key.

Guenther

winreg: add winreg_KeyType enum.

Guenther

s3-winreg: fix some function names in debug statements.

Guenther

s4-winreg: continue processing in WERR_MORE_DATA case in dcesrv_winreg_QueryValue().

Matthias, please check.

Guenther

s4-smbtorture: add extended SetValue test to RPC-WINREG.

Really looks like a blob based database frontend.

As it seems, we can set *any* type of data with *any* content on Windows and get
the exact same data back.

Guenther

s4-smbtorture: more work on test_key_value() RPC-WINREG test.

Guenther

s3: Make init_smb_request return bool

Centralize the exit_server_cleanly()

RPC-DSSYNC: handle DCERPC_FAULT_OP_RNG_ERROR as not supported for DsGetNT4ChangeLog()

metze

s4:torture/libnet/libnet_user.c: fix double free in error path

metze

s4-registry: add all winreg types to reg_value_types table.

Guenther

s4:selftest: move samba4.net.api.become.dc from skip to knownfail

Someone should fix the remaining bug, as this test is expected to work.

metze

NET-API-BECOME-DC: convert to torture api

metze

s4:rpc_server/drsuapi: don't reset [out,ref] pointer to NULL in dcesrv_drsuapi_DsRemoveDSServer()

metze

selftest: let the client also use the dns_host_file

metze

RPC-DSSYNC: fix the build after idl changes

metze

librpc: rerun make idl

metze

drsuapi.idl: finish drsuapi_DsGetNT4ChangeLog idl

metze

s4: allow pam_winbind.so to be build on samba4

* Modify the nsswitch/config.m4 to add tests and build that will be put in configure by the autoconf/autoheader
  We test if there is pam headers and pam library to be able to build the pam module
  We add s4 build directive (that are normaly in standalone config.mk) this is due to the fact that we need
  to rely on path that are guessed during configure.
  Add tests not to build pam_winbind if pam dev files is not present

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

s4: create ../lib/iniparser_build/config.m4

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

s4: Modify auth/config.m4 to move pam tests to nsswitch.m4 so that we tests things in one place

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

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.