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

s3: Implement db_id for dbwrap_cache

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Sat Apr 21 13:46:00 CEST 2012 on sn-devel-104

s3: Initialize "stored_callback" in dbwrap_cache

This should fix one of the recent flaky tests

s3: Remove two unused variables

s3:smbcontrol: remove an unused variable

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Sat Apr 21 00:55:13 CEST 2012 on sn-devel-104

s3:memcache: remove the idmap-part from memcache

This was useful before the idmap cache was moved to gencache.
Nowadays it is available to smbd through gencache, so we
can remove the extra caching layer.

s3:id_cache: do not use the in-memory idmap cache (it is going to be removed)

This also removes the ID_CACHE_FLUSH message.

s3:passdb: do not use the in-memory idmap cache (it will be removed)

s3-dbwrap: dbwrap_watch_record_stored => NT_STATUS_NOT_FOUND is ok...

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Fri Apr 20 17:05:52 CEST 2012 on sn-devel-104

s3-g_lock: Use dbwrap_record_watch_send/recv

This simplifies the g_lock implementation. The new implementation tries to
acquire a lock. If that fails due to a lock conflict, wait for the g_lock
record to change. Upon change, just try again. The old logic had to cope with
pending records and an ugly hack into ctdb itself. As a bonus, we now get a
really clean async g_lock_lock_send/recv that can asynchronously wait for a
global lock. This would have been almost impossible to do without the
dbwrap_record_watch infrastructure.

s3-dbwrap: Add "listwatchers" to dbwrap_tool

s3-dbwrap: Add dbwrap_record_watch_send/recv

With this API you can asynchronously wait for a record to be modified

s3-dbwrap: Add dbwrap_set_stored_callback

This is a per-db function that is called whenever some record is modified

s3-dbwrap: Add "db_context" to "db_record"

s3-dbwrap: Add dbwrap_db_id

This returns a blob uniquely identifying the database

s3: Fix msg_channel in the cluster case

s3: Remove an unused variable

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Fri Apr 20 12:44:20 CEST 2012 on sn-devel-104

selftest: Add extra info on what is failing in wbclient tests

This may help me chase down the failure in my idmap rework.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri Apr 20 09:51:35 CEST 2012 on sn-devel-104

wafsamba: allow certain public libraries to be forced to be private

This will help installations where the Samba4 libraries must be used but
the main system is not using the system libs that would normally
be installed.  This in particular impacts on libwbclient, which is a
core dep, but is different to that used by the rest of a Samba 3.x based
system.

Use eg:  ./configure --private-libraries=wbclient

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri Apr 20 03:27:22 CEST 2012 on sn-devel-104

libwbclient: bump version to 0.9, and add this version to .pc file

Move kdc_get_policy helper in the lsa server where it belongs.

This was used in only 2 places, db-glue.c and the lsa server.
In db-glue.c it is awkward though, as it forces to use an unconvenient lsa
structure and conversions from time_t to nt_time only to have nt_times
converted back to time_t for actual use. This is silly.

Also the kdc-policy file was a single funciton library, that's just ridiculous.

The loadparm helper is all we need to keep the values consistent, and if we
ever end up doing something with group policies we will care about it when it's
the time. the code would have to change quite a lot anyway.

Autobuild-User: Simo Sorce <idra@samba.org>
Autobuild-Date: Fri Apr 20 01:53:37 CEST 2012 on sn-devel-104

Use loadparm helper to find lifetime defaults

Implictly fixes buggy use of int for time_t

loadparm: Add helper function to fetch default lifetime policies

This use long to fetch time_t quantities, because there are architectures were
time_t is a signed long but long != int, So long is the proper way to deal with
it.

loadparm: Add convenience function to return long integers

loadparm: Fix broken lp_ulong utility function

Move README file in the right place.

Someone forgot to move the README when they moved the code ...

Fix bug #8877 - Syslog broken owing to mistyping of debug_settings.syslog.

Setting "syslog only = yes" did not divert log messages to syslog.  The test in
lib/util/debug.c:Debug1():747

   if( syslog_level < state.settings.syslog )

produces wrong results since .syslog is typed "bool" rather than "int".
The attached patch fixes this by typing this field correctly as "int".

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Apr 20 00:06:12 CEST 2012 on sn-devel-104

s3: Fix Coverity ID 2682 -- NULL_RETURNS

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Thu Apr 19 22:28:34 CEST 2012 on sn-devel-104

s3: Fix Coverity ID 2727 to 2740 -- UNINIT

s3: Fix Coverity ID 2686 -- RESOURCE_LEAK

s4-python: Ensure we handle the no-eadb case for system xattrs in get/setntacl

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Apr 19 20:51:16 CEST 2012 on sn-devel-104

Added number of FSMO roles owned by the server we are trying to demote.

s4-provision: Always give the warning if we are using eadb.  It really should be for testing only

s3: Fix the build on FreeBSD

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Thu Apr 19 19:13:45 CEST 2012 on sn-devel-104

s3: Fix Coverity ID 2682: NULL_RETURNS

librpc: Make finding newest pidl file more portable

FreeBSD find does not support the -printf argument.

David, please check if this still work for you!

s3: Fix Coverity ID 2743: CHECKED_RETURN

s3: Fix Coverity ID 2744: CHECKED_RETURN

s3: Fix Coverity ID 2745 and 2746: FORWARD_NULL

We can assume that the rbt dbs are around

s3: Fix Coverity ID 2747: FORWARD_NULL

For the notify cleanup process we have a notify context without a
messaging entry. We will never call notify_add/remove for this, but
the code should protect against this.

s3: Attempt to fix Coverity ID 2748: INTEGER_OVERFLOW

Not sure this will actually please Coverity, but it fixes a severe bug

s3: Fix Coverity ID 2749: REVERSE_INULL

waf: Fix mispelling

Autobuild-User: Simo Sorce <idra@samba.org>
Autobuild-Date: Thu Apr 19 17:36:41 CEST 2012 on sn-devel-104

Fix Error messages

s3/ldap: remove outdated netscape ds 5 schema file

remove outdated netscape ds 5 schema file and put a README there pointing to
the FDS schema file instead. This fixes bug #8869

Autobuild-User: Björn Jacke <bj@sernet.de>
Autobuild-Date: Thu Apr 19 15:54:56 CEST 2012 on sn-devel-104

s3-xattr_tdb: Remove dead code: talloc_stackframe() cannot fail

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Apr 19 14:15:42 CEST 2012 on sn-devel-104

s3-dbwrap: Remove dead code: talloc_stackframe() cannot fail

s3-passdb: Remove unused sampass->pass_must_change_time

There is no need to call pdb_set_pass_must_change_time() because
nothing ever consults that value.  It is always calculated from the
domain policy.

Also, this means we no longer store the value in LDAP.  The value
would only ever be set when migrating from tdbsam or smbpasswd, not on
password changes, so would become incorrect over time.

Andrew Bartlett

s4-kcc: avoid a false alarm with rodc

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Thu Apr 19 12:32:58 CEST 2012 on sn-devel-104

samba_spnupdate: don't try to register DNS related SPN if we are not mastering the NC

For RW DC the impact is pretty small but for RODC the whole SPN set is
rejected by the target DC as RODC hasn't the right to register DNS SPN
if it is not mastering this NC.

s3: Fix a "ISO C90 forbids mixed declarations and code"

Autobuild-User: Volker Lendecke <vl@samba.org>
Autobuild-Date: Thu Apr 19 10:32:27 CEST 2012 on sn-devel-104

dbcheck: Add a check that every FSMO role has a valid owner

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Apr 19 07:49:54 CEST 2012 on sn-devel-104

s4-samba-tool: Fix samba-tool fsmo seize

This is currently untested, and a restructure broke it.

Andrew Bartlett

s4-s3upgrade: Do not ever set a domain-wide maxPwdAge of 0

This means no-expiry in s3, and so we must treat it like -1.

Andrew Bartlett

s4-s3upgrade: Ignore (with warning) groups that are listed but we cannot list members for

Fix samba3.raw.samba3hide test - ensure we set up POSIX capabilities
before doing POSIX calls like chmod.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Apr 18 23:14:40 CEST 2012 on sn-devel-104

Add smb_raw_setfsinfo() - currently only available level is SMB_SET_CIFS_UNIX_INFO.

Ensure we have 12 bytes of data for a SMB_SET_CIFS_UNIX_INFO call. Add debug.