Fix bug #7079 - cliconnect gets realm wrong with trusted domains.

Passing NULL as dest_realm for cli_session_setup_spnego() was
always using our own realm (as for a NetBIOS name). Change this
to look for the mapped realm using krb5_get_host_realm() if
the destination machine name is a DNS name (contains a '.').
Could get fancier with DNS name detection (length, etc.) but
this will do for now.

Jeremy.

s3/smbd: Fix string buffer overflow causing heap corruption

The destname malloc size was not taking into account the 1 extra byte
needed if a string without a leading '/' was passed in and that slash
was added.

This would cause the '\0' byte to be written past the end of the
malloced destname string and corrupt whatever heap memory was there.

This problem would be hit if a share name was given in smb.conf without
a leading '/' and if it was the exact size of the allocated STRDUP memory
which in some implementations of malloc is a power of 2.

s4:ldb Fix check made conditional by mistake

s4:ldb add support for permissive modify control

Fix a really interesting problem found by Volker's conversion of sessionsetup SPNEGO to asynchronous code.

Normally clistr_push_fn() can depend upon cli->outbuf being
initialized by negprot and sessionsetup packets, and cli->outbuf[smb_flgs2] being
correctly set with FLAGS2_UNICODE_STRINGS when cli_setup_packet() is called. When
all the sessionsetups are async, then cli_setup_packet() is never called, the async
code uses cli_setup_packet_buf() - which initializes the allocated async buffer,
not the cli->outbuf one. So the first time clistr_push_fn() is called is from
libsmb/clidfs.c:cli_dfs_get_referral(), just after the connection and tconX.
In this case cli->outbuf has never been initialized, and cli->outbuf[smb_flgs2] = 0
so the DFS query pushes ASCII on the wire, which is not what we want :-).

Remove the dependency on cli->outbuf[smb_flgs2] in clistr_push_fn(), and
fake up a SVAL(cli->outbuf, smb_flg2) value using cli_ucs2(cli) function
instead, which has been initialized. We only care about the FLAGS2_UNICODE_STRINGS
bit anyway.

I don't think this is an issue for 3.5.0 as the sessionsetup is still
synchronous there, but Volker PLEASE CHECK !

Jeremy.

Fix const warning.

Jeremy

Revert "s4:include/includes.h - Need to include "system/network.h""

This reverts commit 97fd03a15a694450e80310fc776a58c6fde58a52.

This obviously broke the build. Revert it for now.

s4:include/includes.h - Need to include "system/network.h"

Otherwise I don't get the definition of "struct in_addr" for "lib/util/util.h" on CentOS 4.

s4:libcli/util/tstream.c - Need to include "system/network.h"

Otherwise I don't get "struct iovec" through "<sys/uio.h>" on CentOS 4.

libcli/nbt: fix off-by-one bug in ndr_pull_wrepl_nbt_name()

The scope starts at byte 17 with index 16.

metze

libcli/nbt: fix ndr_pull/push_wrepl_nbt_name()

[MS-WINSRA] — v20091104 was wrong
regarding section "2.2.10.1 Name Record"

If the name buffer is already 4 byte aligned
Windows (at least 2003 SP1 and 2008) add 4 extra
bytes. This can happen when the name has a scope.

metze

s3: link thread objects in libsmbclient only and adjust linker flags

AIX doesn't have MSG_DONTWAIT

Fixed a bug caused by a typo. Infrastructure role didn't work.

s4/ldap: Test to expoit ldb_ildap bug in case of nested search requests

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

s4/ldap: Fix nested searches SEGFAULT bug

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

s4: Ignore few more auto-generated files

librpc: rerun 'make idl'

metze

security.idl: add wellknown TrustedInstaller SID

metze

s3: by default don't use pthread pool support

lib/util: remove data_blob_talloc_reference()

We want to avoid the usage of talloc_reference() in Samba.

metze

s4:auth_sam: avoid usage of data_blob_talloc_reference() and copy the session keys

metze

s4:libcli: remove unneeded talloc_reference() usage

metze

s4:kdc remove dead code and comments

s4:kdc Fill in more data fields

s4:kdc move db functions in their own file

Keep all heimdal related plugin code within hdb_samba4.c
Move interfaces needed by multiple plugins in db-glue.c

Move sequence context in main db context so that we do
not depend on db->hdb_dbc in the common code.

Remove unnecessary paremeters from function prototypes

s4:kdc Use a clearer name for the samba kdc entry

Renames hdb_samba4_private to samba_kdc_entry
Streamlines members of the entry and the kdc db contextto avoid
unnecessary duplication.

s4:kdc Use better db context structure

This allows to use a common structure not tied to hdb_samba4
Also allows to avoid many casts within hdb_samba4 functions

This is the first step to abstract samba kdc databse functions
so they can be used by the MIT forthcoming plugin.

Fix bug #7069 - smbget does not return an error status after some errors

A combination patch from Johannes Poehlmann <johannes@lst.de> and
Jeremy. Fix the return codes from smb_download_file() and smb_download_dir().

Jeremy.

s3/winbind_ccache: Fix typo in debug message.

Karolin

tsocket: Fix the description of tstream_writev_queue_send/recv

tsocket: Fix description for tstream_readv_pdu_queue_send/recv

s4-ldbtest: fixed python import

s4-drs: implementation of some delete test cases

Signed-off-by: Andrew Tridgell <tridge@samba.org>

cleanup: remove trailing spaces and tabs

s4-drs: Reads uSNUrgent and sets Urgent Replication Bit for DS_ReplicaSync when necessary

If an urgent replication is necessary, so the uSNUrgent stored is equal to the
uSNHighest stored, then when sending the DS_ReplicaSync message it sets the
DRSUAPI_DRS_SYNC_URGENT bit on DRS_OPTIONS.

Signed-off-by: Fernando J V da Silva <fernandojvsilva@yahoo.com.br>
Signed-off-by: Andrew Tridgell <tridge@samba.org>

Second part of fix for bug #7072 - Accounts can't be unlocked from ldap.

Missed read of entry_timestamp (was entry->entry_timestamp).

Jeremy.

Fix bug #7072 - Accounts can't be unlocked from ldap.

Fix suggested by Andy Hanton <andyhanton@gmail.com>. The LOGIN_CACHE
struct contains two time_t entries, but was being written to and
read from via tdb_pack/tdb_unpack functions using explicit 32-bit int specifiers.
This would break on machines with a 64-bit time_t. Use correct int
sizes for tdb_pack/tdb_unpack.

We have to fix this properly before 2037 :-).

Jeremy.

testsuite/libsmbclient use source3 in the path of the C and LFLAGS

Link testsuite/libsmbclient against required libtdb and libwbclient

s3: syntax fix

s3: Install and uninstall the static version of libwbclient

This requires to call configure with --enable-static which isn't the case
by default.

s4:windc move windc plugin in its own file

Keep all heimdal related plugin code within wdc-samba4.c
Leave only interfaces common to multiple plugins in pac-glue.c

s4:PAC make common functions public

s4:PAC Streamline pac-glue step 2

Split functions so that no assumption is made about which plugin
is using them

s4:PAC Streamline pac-glue

First step, preparing to share the code between multiple plugins.

s3: get pthread detection also on HP-UX right

Implemented net fsmo command for transferring fsmo roles

The command allows the user to transfer a fsmo role to the server to which
the connection is established. Roles can be transferred or seized. By default a
transfer is attempted even if seize option is chosen, as it is dangerous to
seize a role if the current owner is still running.
example use:
net fsmo show --host=hostnameoraddress --username=username --password=password
net fsmo transfer --role=role --host=hostnameoraddress --username=username --password=password
net fsmo seize --role=role --host=hostnameoraddress --username=username --password=password [--force]

Tested against Win2008. Does not work for samba 4 yet as we are missing the GetNCChanges extensions.

s4-smbtorture: pick correct last packet while checking backchannel replies in RPC-SPOOLSS-NOTIFY.

Guenther

s3-selftest: try to get RPC-SPOOLSS-NOTIFY running during make test on the buildfarm.

Guenther

s4:torture: refactor querying of domain info out into test_QueryDomainInfo2_level()

Used in several places.

(Note: The _level suffix in the function name is just because
test_QueryDomainInfo2() already exists as an overall test for all levels.)

Michael

Signed-off-by: Günther Deschner <gd@samba.org>

s4:torture: create new test_SetDomainInfo_ntstatus()

for checking for SetDomainInfo giving a desired return code.

Michael

Signed-off-by: Günther Deschner <gd@samba.org>

s4:torture: refactor setting of domain inf out into test_SetDomainInfo()

and use it in several places.

Michael

Signed-off-by: Günther Deschner <gd@samba.org>

s4:torture: refactor setting account flags out into test_SetUserInfo_acct_flags()

and use this in four places (enabling/disabling accounts).

Michael

Signed-off-by: Günther Deschner <gd@samba.org>

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

Only works on Linux kernels 2.6.26 and above. Grants CAP_KILL capability
to allow Linux threads under different euids to send signals to each other.

Jeremy.

Add dependency of bin/smbfilter to libwbclient.

s4-libcli: Fixed a talloc_reference error.

Signed-off-by: Andreas Schneider <asn@redhat.com>
Signed-off-by: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>

s4:winsrepl.idl: add random interface uuid

This is needed to include the wrepl interface into
ndrdump.

metze

s4:smbtorture: add BASE-BENCH-HOLDOPEN

This is useful for manual performance testing with a large
number of share mode entries.

metze

mount.cifs: don't allow it to be run as setuid root program

mount.cifs has been the subject of several "security" fire drills due to
distributions installing it as a setuid root program. This program has
not been properly audited for security and the Samba team highly
recommends that it not be installed as a setuid root program at this
time.

To make that abundantly clear, this patch forcibly disables the ability
for mount.cifs to run as a setuid root program. People are welcome to
trivially patch this out, but they do so at their own peril.

A security audit and redesign of this program is in progress and we hope
that we'll be able to remove this in the near future.

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

mount.cifs: check for invalid characters in device name and mountpoint

It's apparently possible to corrupt the mtab if you pass embedded
newlines to addmntent. Apparently tabs are also a problem with certain
earlier glibc versions. Backslashes are also a minor issue apparently,
but we can't reasonably filter those.

Make sure that neither the devname or mountpoint contain any problematic
characters before allowing the mount to proceed.

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

mount.cifs: take extra care that mountpoint isn't changed during mount

It's possible to trick mount.cifs into mounting onto the wrong directory
by replacing the mountpoint with a symlink to a directory. mount.cifs
attempts to check the validity of the mountpoint, but there's still a
possible race between those checks and the mount(2) syscall.

To guard against this, chdir to the mountpoint very early, and only deal
with it as "." from then on out.

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

s4-smbtorture: also test smbc_getOptionUseCCache

s3: Enable use of ccache by default for libsmbclient

Disable this by setting the environment variable LIBSMBCLIENT_NO_CCACHE, which
has the advantage over an smb.conf option to be easily settable per
application.

Correct fix for unused variable return from ndr_decode. Use it :-).
Jeremy.

Revert "s3: remove unused Variable"

This reverts commit 9536d94d5478b63fc05047964b40d8786a7246c4.

Bjorn, your change removed the ndr_decoding of the dos attribute.
Not a good idea :-).

Jeremy.

Revert "libreplace: fix unused varibale warning on IRIX"

this one was broken

s3: fix detection and flags for using pthread

I hope this fixes the build on Tru64, IRIX and maybe bug #6983

s3: remove unused variable

s3:async_smb: remove unused variable

libreplace: fix unused varibale warning on IRIX

s3: remove unused Variable

Jeremy, please check!

s3:netapi: remove unused variable

lib/popt: Fix typo in README.

Karolin

s3/docs: Fix typo.

Karolin

s3/registry: Fix typo in comment.

Karolin

Revert "libcli/security: Remove a call to strncasecmp"

This reverts commit 7c687665eaf16b0c6f83c130f6d9e5459e0b2a32.

Revert "libcli/security: Convert some strtol calls to strtoul"

This reverts commit 7fe66e06c4df575c410d4d70ff38f120c2f4363b.

Revert "libcli/security: Fix a valgrind error in dom_sid_parse"

This reverts commit f1c889a4e61d6d751cbabd8014b4345b8051b97c.

s4-python: Fix formatting.

upgrade_from_s3: Remove unused imports.

pyxattr: Move to the same directory as the xattr code.

s4 includes: Include system/locale.h to get access to <ctype.h>

Revert "s3: Make string_to_sid a wrapper around dom_sid_parse"

This reverts commit dff03b61fd5d923562711b38cc7dbe996dc07283.

Revert "libcli/security: Prohibit SID formats like S-1-5-32-+545"

This reverts commit 1fbeae41655b8305834f2149b1268077eba8633d.

Apparently this breaks the build of Samba4

s4-smbtorture: test wbcResolveWinsByName/wbcResolveWinsByIP in WINBIND-WBCLIENT testsuite.

Guenther

s4-smbtorture: add test for wbcPingDc to WINBIND-WBCLIENT testsuite.

Volker, please check, you know best how this interface is designed :-)

Guenther

s4-smbtorture: split spoolss_RemoteFindFirstPrinterChangeNotifyEx test into smaller parts.

Guenther

testprogs: print all driver info levels in spoolss testsuite.

Guenther

s3-libsmbclient: Add smbc_setOptionUseCCache()

Can we enable this by default? This would be a change in behaviour, but this
feature is just too cool for everyone to catch up in the apps.

The patch would be

s3: Add --use-ccache to net

s3: add libnetapi_set_use_ccache()

s3: Fix a bug in net's use of popt

In order to add --use-ccache to net, I added another "bool opt_ccache;" to
struct net_context. popt did not like this, it took a while to figure out why.
Popt has the lines

    /* XXX Check alignment, may fail on funky platforms. */
    if (arg == NULL || (((unsigned long)arg) & (sizeof(*arg)-1)))
        return POPT_ERROR_NULLARG;

The "bool opt_ccache;" was not aligned anymore...

s3: Enable -C in rpcclient

s3: Add CLI_FULL_CONNECTION_USE_CCACHE

s3: Use -C in smbclient

$ bin/wbinfo --ccache-save=w2k3ad\\vl%Password
saving creds succeeded
$ bin/smbclient //192.168.42.160/tmp -Uvl -N -C -W w2k3ad
OS=[Windows Server 2003 R2 3790 Service Pack 2] Server=[Windows Server 2003 R2 5.2]
smb: \>
$ bin/wbinfo --ccache-save=w2k3ad\\vl%WrongPassword
saving creds succeeded
$ bin/smbclient //192.168.42.160/tmp -Uvl -N -C -W w2k3ad
Anonymous login successful
Domain=[W2K3AD] OS=[Windows Server 2003 R2 3790 Service Pack 2] Server=[Windows Server 2003 R2 5.2]
tree connect failed: NT_STATUS_ACCESS_DENIED
$

s3: Add -C (--use-ccache) to popt_common_credentials

s3: Add ccache use to cli_session_setup_ntlmssp

s3: Add NTLMSSP_FEATURE_CCACHE

Uses the winbind ccache to do authentication if asked to do so

libwbclient: Actually implement wbcCredentialCache()