libcli/smb: move SMB2_IOCTL_FLAG_IS_FSCTL to smb2_constants.h

metze

s4:ntvfs/ipc: don't echo back the smb2 ioctl input_buffer

metze

s3: oplock_timeout is a talloc child of fsp

Jeremy, I know you like it explicit, but I stumbled across this
explicit TALLOC_FREE and asked myself about a potentially wrong
talloc hierarchy.

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Thu Apr 26 23:00:03 CEST 2012 on sn-devel-104

s3: Remove code TALLOC_FREE already does

torture: change dfs tests now that we return a level 4 when level 4 is requested

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Thu Apr 26 21:25:37 CEST 2012 on sn-devel-104

dfs_server: fix the response to please XP and Windows 2008R2 doing so avoid continious DFS requests from W2K8R2

replace: Fix use of mktemp

mktemp always returns the template, so checking for NULL doesn't cactch any
error. Errors are reported by turning the template into an empty string.

Autobuild-User: Simo Sorce <idra@samba.org>
Autobuild-Date: Thu Apr 26 16:14:24 CEST 2012 on sn-devel-104

addns: clean up headers

All this stuff is already included properly in the replace headers on top.

s3: Remove an unused variable

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Thu Apr 26 12:17:14 CEST 2012 on sn-devel-104

s4-torture: never return false in the raw.composite torture test

We must use torture_assert() macros to ensure that the failure is
recorded in the subunit stream correctly, rather than being returned
as an unknown 'error'.  (We cannot handle error results as knownfail).

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Apr 26 04:00:16 CEST 2012 on sn-devel-104

selftest: push --target selection into plansmbtorturetestsuite

s4:torture: add a rpc.samba3.smb-reauth2 test

* Open session.
* Reauth to anon.
* Open lsa pipe.
* Do lsa get username.
* Rauth to original user.
* Do lsa get username again.

expected result: the getusername works and
shows 'NT AUTHORITY\ANONYMOUS LOGON' in both calls.

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Thu Apr 26 02:27:12 CEST 2012 on sn-devel-104

s4:torture: add a rpc.samba3.smb-reauth1 test

It does reauth while using an rpc pipe:

* open session
* open lsa pipe
* do lsa getusername
* reauth to anonymous
* do lsa getusername again
* reauth back to original user
* do reauth again

result: lsa getusername is always possible
and always gives the first (authenticated user)

s4:torture: add a raw.session suite with a raw.session.reauth test

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

Fix bug reported by Dina_Fine@Dell.com.

Don't ask the DC for an IP list when locating kdc's. Ask for the
name and use getaddrinfo to get all possible addresses instead.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Apr 25 22:01:18 CEST 2012 on sn-devel-104

libcli: Use data_blob_null

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Wed Apr 25 18:54:51 CEST 2012 on sn-devel-104

selftest: remove printserver.enum_printers_servername from flapping

Signed-off-by: Andreas Schneider <asn@samba.org>
Autobuild-User: Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date: Wed Apr 25 17:18:28 CEST 2012 on sn-devel-104

selftest: remove samba3.rpc.spoolss.*printserver.openprinter_badnamelist from flapping

Signed-off-by: Andreas Schneider <asn@samba.org>

selftest: mark the driver_info_winreg tests knownfail.

They only get executed when driver files are around.

Signed-off-by: Andreas Schneider <asn@samba.org>

selftest: mark failing print_tests more specifically

Signed-off-by: Andreas Schneider <asn@samba.org>

selftest: remove .*printer.*print_test_extended from flapping

Signed-off-by: Andreas Schneider <asn@samba.org>

selftest: remove the "rpc.spoolss.printer" test from flapping

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: replace call to reg_openkey() in reg_createkey() by accesscheck.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: remove a superfluous fill_subkey_cache() in reg_createkey()

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

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: use fill_subkey_cache to check exsistence in regkey_open_onelevel().

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

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: let fill_subkey_cache return WERR_BADFILE when the subkey list could not be loaded

WERR_NO_MORE_ITEMS seems inappropriate.

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

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: convert reg_openkey() to use talloc instead of SMB_STRDUP etc

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry untangle an assignment from the check in regkey_open_onelevel()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: untangle assignment from check in regkey_open_onelevel()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: fix seqnum race in regdb_fetch_keys_internal

This prevents race between fetching seqnum and key content.

Because there is currently no way to atomically fetch the
record along with the seqnum, I use a loop.
This is far from optimal and should should ideally be done
differently. But for now it fixes the race.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: fix seqnum race in fetch_values_internal

This prevents race between fetching seqnum and key content.

Because there is currently no way to atomically fetch the
record along with the seqnum, I use a loop.
This is far from optimal and should should ideally be done
differently. But for now it fixes the race.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: update the seqnum in the subkey cache at the end of regval_store_keys

The purpose is to prevent next reads from going to disk.

Note that this will currently only be effective with local tdbs, not
with ctdb: For tdb, store and delete bump the seqnum while transaction
commit does not. For ctdb, transaction commit bumps the seqnum, while
store and delete don't... This needs fixing (in ctdb).

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry:db: update the value container seqnum after storing/deleting to prevent next read from going to disk if possible

Note that this will currently only be effective in the local TDB implementation.
For CTDB, this wont work since seqnum currently works differently there (needs
fixing): For tdb, store and delete operations bump the db seqnum, while
transaction commits don't. For ctdb, the seqnum is bumped by the transaction
commit but not by store and delete operations.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: wrap reg_deletekey() into a transaction

This is wrong layering but fixes a race condition.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: wrap reg_createkey() in a transaction

This is wrong layering (calling into regdb_transaction* in the reg_api code)
but fixes a potential race. It makes the multi-step create procedure atomic.

This should completely be done in the backend.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: untangle assignments from checks in reg_createkey()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: wrap reg_deletevalue() in a transaction

This is at the wrong layer, but if fixes a race potentially causing
data corruption by concurrent access.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: untangle assignment from check in reg_deletevalue()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: fix race in reg_setvalue that could lead to data corruption

(there was no lock around fetching the values and storing them)

The layering is wrong in that it uses regdb transactions in reg_api

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: untangle assignment from check and add a debugmessage in reg_setvalue()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: don't leak the old contents when updating the value cache

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: fix debug message in regdb_store_values_internal()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: improve log message in regdb_unpack_values()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: fix a debug message typo

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: add a new function regval_ctr_value_byname()

This is like regval_ctr_key_exists() but does not return bool,
but the regval_blob instead, if found, and NULL if not found.

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: rename regval_ctr_key_exists() to regval_ctr_value_exists()

Signed-off-by: Andreas Schneider <asn@samba.org>

s4:torture:rpc:spoolss: also initialize driverName before checking it in test_PrinterData_DsSpooler()

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry:reg_api: fix reg_queryvalue to not fail when values are modified while it runs

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: make regdb_values_need_update() static

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: make regdb_subkeys_need_update() static

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: make regdb_store_values() static

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: make regdb_fetch_values() static

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: make regdb_fetch_keys() static

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: make regdb_store_keys() static

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: printing backend: use regdb options via ops struct, not directly

just like the other backends.

This is in preparation of making the backend functions private

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: remove usage of reg_objects from net_rpc_printer.c

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: remove usage of reg_objects from cmd_spoolss.c

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:eventlogadm make a transaction for addsource

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:eventlogadm reimplement addsource using reg_api

Signed-off-by: Andreas Schneider <asn@samba.org>

s3-waf: Cleanup smbregistry.

s3:registry: remove usage of reg_objects from srv_spoolss_nt.c

Signed-off-by: Andreas Schneider <asn@samba.org>

s3:registry: remove usage of reg_objects from libads/ldap_printer.c

Signed-off-by: Andreas Schneider <asn@samba.org>

s4-messaging: Use generate_random() to get a unique ID for messaging clients

The call to random() resulted in duplicate values for s3fs configurations
which, due to the forked child, all started with the same random seed.

A future improvement would be to move to a proven unique value.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed Apr 25 11:43:40 CEST 2012 on sn-devel-104

s4-torture: enable raw.composite test again, and convert to new style test

This test has been skipped for some time, but is an important test for parallel operation.

Andrew Bartlett

lib/replace: split out GSSAPI from lib/replace/system/kerberos.h into lib/replace/system/gssapi.h

With waf build include directories are defined by dependencies specified to subsystems.
Without proper dependency <gssapi/gssapi.h> cannot be found for embedded Heimdal builds
when there are no system-wide gssapi/gssapi.h available.

Split out GSSAPI header includes in a separate replacement header and use that explicitly
where needed.

Autobuild-User: Alexander Bokovoy <ab@samba.org>
Autobuild-Date: Wed Apr 25 00:18:33 CEST 2012 on sn-devel-104

s3: Simplify check_reduced_name a bit

It's pointless to do a talloc_asprintf with a SMB_STRDUP on the
result. Use asprintf directly.

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Tue Apr 24 18:18:05 CEST 2012 on sn-devel-104

s3: Fix a typo

s3: Introduce variable "indyniov" for easier reading

s4:libnet: pass an explicit mem_ctx to libnet_rpc_userinfo_send() (bug #8889)

This hopefully fixes the flakey autobuild.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Apr 24 16:43:03 CEST 2012 on sn-devel-104

s3: Attempt to fix the build without kerberos

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Tue Apr 24 15:04:14 CEST 2012 on sn-devel-104

s3: Fix Coverity ID 2751: REVERSE_INULL

talloc: Fix copy&paste errors

Talloc doc: talloc_strdup_append does not return duplicated string

Talloc doc: when s == NULL in _append functions

s4-s3upgrade: print the error message from passdb.error exceptions

This gives more information on why a group membership lookup failed.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Apr 24 04:34:44 CEST 2012 on sn-devel-104

Fix pam_winbind build against newer iniparser library

iniparser_getstr is deprecated and has been removed in newer libraries
available in Fedora. Use iniparse_getstring instead.

Autobuild-User: Simo Sorce <idra@samba.org>
Autobuild-Date: Tue Apr 24 02:56:10 CEST 2012 on sn-devel-104

Avoid warning about KRB5_DEPRECATE with MIT libs

Cracknames: use krb wrapper functions so it works with MIT

Also avoid a silly game with directly modifying the principal and
then calling krb5_principal_unparse_flags to get out a string.
If we already assume it is a 2 components name and know what outcome we are
going to get, just go ahead and talloc_asprintf the linearized string.

krb5_samba: Add support for krb5_princ_size when using Heimdal

Make krb5 wrapper library common so they can be used all over

For now just disable this Heindal specific stuff in the MIT build

Make krb5 context initialization not heimdal specific

Turn the logging data to an opaque pointer.
Ifdef code and use MIT logging function when built against system MIT.

Make sure krb5_principal_get_num_comp is identified as present for Heimdal build

Common wrappers for MIT / Heimdal use krb5_principal_get_num_comp() to replace krb5_princ_size
but rely on krb5_principal_get_num_comp() identified by the build. As we know it exists in Heimdal,
define it for waf build.

Signed-off-by: Simo Sorce <idra@samba.org>

waf: rename SAMBA4_INTERNAL_HEIMDAL to SAMBA4_USES_HEIMDAL

SAMBA4_INTERNAL_HEIMDAL is defined unconditionally regardless
where Heimdal comes from, system-wide or embedded version.

This define is not used anywhere. We'll use it to distinguish
between Heimdal and MIT Krb5 builds.

Signed-off-by: Simo Sorce <idra@samba.org>

waf: move krb5 checks to a separate waf file

With PROCESS_SEPARATE_RULE in wafsamba it is now possible to simplify
configuration and checks for MIT/Heimdal Kerberos implementations.

1. Move MIT krb5 checks from source3/wscript to wscript_configure_krb5
2. Make sure they are called same way (--with-mit-krb5-checks)
3. If no configure checks identified MIT krb5 in system (or were disabled),
   make sure Heimdal build is selected, embedded (default) or system-provided.

This makes logic of configuration unchanged for Heimdal builds but adds
less hacky way to use MIT krb5 builds. The latter does not work yet as we
need to untangle more subsystems from HDB/Heimdal-specific details but
lays out a foundation for that.

Signed-off-by: Simo Sorce <idra@samba.org>

Fix bug #8882 - Broken processing of %U with vfs_full_audit when force user is set.

When doing a "force user" we need to remember what the "sanitized_username"
was from the original connect.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Apr 23 19:52:19 CEST 2012 on sn-devel-104

Add complete test program for Linux kernel aio inside configure.in (I discovered yesterday there are systems with only half the glibc changes needed to implement userspace kaio.

s4-libnet Always return after composite_error()

These instances should not cause a problem, but make it easier to audit for
this kind of problem in the future with grep.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Mon Apr 23 14:29:45 CEST 2012 on sn-devel-104

s4-libnet Always return after composite_error()

This can and does cause crashes as multiple conflicting sets of callbacks run on
memory that may have been destroyed.

Andrew Bartlett

s4-libnet: Fix continue_groupinfo_openuser to check correct state info

This meant that we would attempt to query the user that we could not open.

This is a mirror of 4ba1647d5db59e5bb4911c399111e9286aac1a8e.

Andrew Bartlett

build: Remove support for a system libsmbclient

With the new --private-libraries option, there is no longer the need
to have this support, which was aimed at avoiding the duplication
between two different libsmbclient binaries in a packaged
distribution.  By using --private-libraries instead, we do not
introduce a dependency between Samba 4.0 packages and whatever other
packages are on the system.

This effectivly reverts part of
e3ffb31554927a24df35576bd753ee4eb2dc2a3c while kpeeing the improved
SAMBA_LIBRARY declaration.

Andrew Bartlett

build: Remove support for system libwbclient

This effectively reverts commit 2c49782dc1069eebc9f5a5cd2a055b5912665595

This also adds explaination about what would need to be done to
support a system libwbclient

Andrew Bartlett

s4:librpc/rpc: directly return after composite_error()

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Apr 23 12:17:08 CEST 2012 on sn-devel-104

s4:libnet_become_dc: add missing 'return' after composite_error()

metze

selftest: Add test for bug #8884: wbinfo --group-info=administrator segfaults s4-winbindd

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Mon Apr 23 07:44:50 CEST 2012 on sn-devel-104

s4-libnet: Fix continue_groupinfo_opengroup to check correct state info

This meant that we would attempt to query the group that we could not open.

Andrew Bartlett

s4-libnet: Fix segfault shown by wbinfo --group-info=administrator

The issue was that after the LookupNames call indicated that this was
not a group, the call paths diverged, with both sucess and failure
paths running.

Andrew Bartlett

s3-dbwrap: A void function can not return a value

Only non-gcc compilers seem to notice this as an error.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Mon Apr 23 05:58:52 CEST 2012 on sn-devel-104

dlz_bind9: Also produce and install binary plugin for bind 9.9

Thanks to Alexander Bokovoy <ab@samba.org> for the fix!

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Sun Apr 22 10:51:57 CEST 2012 on sn-devel-104

idl: fix the charset for the source_dsa_address in DsReplicaDel call

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Sun Apr 22 03:00:06 CEST 2012 on sn-devel-104