s4-auth Move libcli/security/session.c to the top level

This code is now useful in common, as the elements of the
auth_session_info structure have now been defined in common IDL.

Andrew Bartlett

s3:auth: change num_groups to from size_t to uint32_t

This will help with the change from UNIX_USER_TOKEN to security_unix_token

metze

s3-auth Rename auth_serversupplied_info varaiables: server_info -> session_info

These variables, of type struct auth_serversupplied_info were poorly
named when added into 2001, and in good consistant practice, this has
extended all over the codebase in the years since.

The structure is also not ideal for it's current purpose.  Originally
intended to convey the results of the authentication modules, it
really describes all the essential attributes of a session.  This
rename will reduce the volume of a future patch to replaced these with
a struct auth_session_info, with auth_serversupplied_info confined to
the lower levels of the auth subsystem, and then eliminated.

(The new structure will be the output of create_local_token(), and the
change in struct definition will ensure that this is always run, populating
local groups and privileges).

Andrew Bartlett

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

s4-smbtorture: use torture_comment() instead of printf in raw.write test.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon Feb 21 15:26:38 CET 2011 on sn-devel-104

s4-smbtorture: fix some indent in raw.write test.

Guenther

s4-smbtorture: fix trailing whitespace in raw.write test.

Guenther

s4:scripting/python/modules.[ch] - explicitly say that "py_update_path" takes no argument

This quiets some build warnings.

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Mon Feb 21 14:42:00 CET 2011 on sn-devel-104

s3:tests: move the log directory of the smbclient_s3 test to st/

And use a proper mktemp call to create a unique name.
The old location under /tmp/ was bad.

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Mon Feb 21 13:39:06 CET 2011 on sn-devel-104

s3:tests: clean up old log directories of the registry.roundtrip test

s4:smbtorture do not leave output directory around

calling smbtorture with no or invalid arguments left the temporary
output directory around
this patches removes the dead-end exit from usage and makes the
logic go on until the final cleanup state is reached

output directory will still be left around when test times out
or testcases itself force an exit

also make sure that the directory itself is deleted, not just
the objects in it

Signed-off-by: Matthias Dieter Wallnöfer <mdw@samba.org>
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Mon Feb 21 11:35:30 CET 2011 on sn-devel-104

libcli/security/security_descriptor.c - fix three wrong counter variables

These strictly need to be "uint32_t" since "acl*->num_aces" has been
defined by this type.

This counter patchset has been reviewed by Andrew Bartlett.

libcli/security/privileges.c - fix wrong counter type

This strictly needs to be from type "uint32_t" since "privset->count"
is defined with this type.

libcli/security/privileges.c - fix the counting of privileges

Since the privileges are always counted with a signed integer, there is no
reason to specify the upper limit with a "uint32_t".

libcli/security/sddl.c - fix wrong counter type

This strictly needs to be from type "uint32_t" since "acl->num_aces" is
defined of this type.

libcli/security/display_sec.c - fix wrong counter type

This strictly needs to be of type "uint32_t" due to
"sec_acl->num_aces" which is of type "uint32_t".

s4: avoid overwritting the dceprc.h file from toplevel librpc

Change back to what was the configuration of header_path before
177ddcb084e8f8608bf2012a7c58f7b1e56d2acd so that dcerpc.h is installed
in ${prefix}/include and not in ${prefix}/include/gen_ndr as the
toplevel librpc wants also to put a dcerpc.h file in this directory.

Add some documentation in order to avoid any new regression.

This change should fix problems for openchange team.

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Mon Feb 21 09:47:05 CET 2011 on sn-devel-104

pyldb: add tests for getting control results

pyldb: Add tests for the ldb_result object

pyldb: create LdbResult, return value from ldb.search is now a LdbResult

s4-unittests: replace assertEquals(res, []) by assertEquals(len(res), 0)

ldb: controls marshalling/unmarshalling improvement

Add a function to marshall a control to a string
Refactor the code of ldb_control_parse_strings to allow to extract the
core code into ldb_control_parse_from_string so that this function can
be called for just 1 string

Fix missing dependency on libreplace.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Mon Feb 21 01:17:33 CET 2011 on sn-devel-104

s4/drs-tests:Blackbox test to verify DsReplicaSync handling

DsReplicaSync is indirectly called using 'samba-tool drs options' command
to enable/disable replication and 'samba-tool drs replicate' command
to trigger inbound replication cycle

Autobuild-User: Kamen Mazdrashki <kamenim@samba.org>
Autobuild-Date: Sun Feb 20 23:03:03 CET 2011 on sn-devel-104

s4/samba.tests: Raise BlackboxProcessError exception in case check_output method fails

This way the caller gets the chance to receive STDOUT and STDERR for
a failed process.
We may use this info to check if a process has failed properly,
e.g. with expected output

s4/samba.tests: Extend CalledProcessError class to store STDOUT and STDERR for a failed rocess call

s4/drs-tests: Allow _net_drs_replicate to force replication and return output from command execution

s4/drs-tests: Split samba-tool command line generation into separate method

so we can use to exec samba-tool passing different commands

s4/drs-tests: Inherit DrsReplSchemaTestCase from DrsBaseTestCase

and remove duplicated functionality

s4/drs-tests: Inherit DrsFsmoTestCase from DrsBaseTestCase

and remove duplicated functionality

s4/drs-tests: Inherit DrsDeleteObjectTestCase from DrsBaseTestCase

and remove duplicated functionality

s4/drs-tests: Add a base class to hold common functionality for all DRS python tests

s3: Fix pdb_ads_enum_aliasmem for empty aliases

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Feb 20 21:40:40 CET 2011 on sn-devel-104

s3: Support SetGroupInfo in pdb_ads

s3: Fix a debug message

s3: Fix pdb_ads_enum_group_members for empty groups

s3: Fix error returns in pdb_ads_mod_groupmem

s4-version: Fix version finding for bzr branches.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sun Feb 20 17:35:45 CET 2011 on sn-devel-104

ldb-samba: Add missing dependency on pyldb-util.

s3: Improve a debug message, we have more than pdb_tdb.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Feb 20 16:50:05 CET 2011 on sn-devel-104

s3: Protect tldap_errstr against a NULL ld

s3: Fix return code of pdb_ads_getsamupriv if the user does not exist

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Feb 20 11:12:01 CET 2011 on sn-devel-104

s3: Add ACB_NORMAL to workstations, match RPC-SAMR

s3: Make QueryDispInfo and QueryInfo match for guest

s3: NO_SUCH_USER is a lot more likely than NO_MEMORY

s3: Convert init_system_info to NTSTATUS

s3: Slightly simplify Get_Pwnam_alloc

s3: Fix some nonempty blank lines

s3:tests: fix the smbclient_s3 test for multiple users running it.

The test created /tmp/log.smbclient, which lets the test fail if
two users are running the test simultaneously.

Now /tmp/log.test_smbclient_s3.$$ is used.

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Sat Feb 19 13:06:57 CET 2011 on sn-devel-104

s3:tests: remove unused script/tests/test_functions.sh

This is not used any more now with selftest.

s3:tests: include subunit.sh instead of test_functions.sh in test_posix_s3.sh

It would actually not be necessary at all to include a script providing
testit() and so on, since this script snippet is currently only sourced
in tests.sh to plan the included test.

s3:tests: include subunit.sh instead of test_functions.sh in test_ntlm_auth_s3.sh

It would actually not be necessary at all to include a script providing
testit() and so on, since this script snippet is currently only sourced
in tests.sh to plan the included test.

s3:tests: include subunit.sh instead of test_functions.sh in test_smbtorture_s3.sh

It would actually not be necessary at all to include a script providing
testit() and so on, since this script snippet is currently only sourced
in tests.sh to plan the included test.

s3:tests: include subunit.sh instead of test_functions.sh in test_wbinfo_s3.sh

It would actually not be necessary at all to include a script providing
testit() and so on, since this script snippet is currently only sourced
in tests.sh to plan the included test.

s3:tests: use the blackbox subunit output formatter for the testparm_s3 test.

With the old output, failures would go unnoticed by selftest.

s3:tests: include subunit.sh instead of test_functions.sh in test_local_s3.sh

It would actually not be necessary at all to include a script providing
testit() and so on, since this script snippet is currently only sourced
in tests.sh to plan the included test.

s3:tests: use the blackbox subunit output formatter for the smbclient_s3 test.

With the old output, failures would go unnoticed by selftest.

s3: Pass logonHours through pdb_ads

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat Feb 19 12:22:08 CET 2011 on sn-devel-104

s3: Add "len" to pdb_set_hours

s3: Let pdb_ads write accountExpires

s3: Fix pdb_ads_pull_time

vfs_smb_traffic_analyzer shall also transfer the clients IP address.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Feb 19 01:53:18 CET 2011 on sn-devel-104

Fix alignment issues in SPOOLSS "BUFFER" marshalling.
Guenther and Metze please check !

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Feb 18 18:49:55 CET 2011 on sn-devel-104

s3:vfs:syncops add option to disable metasync per share

introduce an option to disable the metadata sync
in case the filesystem handles this correctly the sync can be
skipped, but synchronization of the data that was written can still
be configured (in opposition to the disable flag disabling all
sync operations)

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Feb 18 17:31:59 CET 2011 on sn-devel-104

s3-smbsharemodes: libsmbsharemodes does neither depend on krb5 or ldap.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Fri Feb 18 16:46:35 CET 2011 on sn-devel-104

nsswitch: make libwbclient a public library again.

Guenther

s3-waf: we need to globally -Ilib/replace

Guenther

s3-waf: fix popt support.

Guenther

lib/util/charset Add back setlocale(), but only when called from binaries

When called from a library, we don't want to call this, as we may
overwrite some of our calling program's context.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Fri Feb 18 09:29:35 CET 2011 on sn-devel-104

lib/util/charcnv Move iconv handle setup in common

We now use the struct smb_iconv_convenience at the core of all our
iconv code, and use global_iconv_convenience for the callers that
don't specify one.

Andrew Bartlett

s3-charcnv Don't genreate valid_table on the fly, rely on valid.dat

This file is always installed, and is only even required for the old,
depricated mangle hash method.

Andrew Bartlett

lib/util Remove #if _SAMBA_BUILD_ == 4 that isn't required any more

lib/util/charset Use top level iconv.c in source3

The two files were very similar already, the only change required was
to adopt the s3 module registration fucntion name.

(NTSTATUS wasn't used as the charset code does not otherwise use that
type).

Andrew Bartlett

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

s4-idl: rename s4 server_id.idl to server_id4.idl

this avoids a conflict with the new s3 server_id.idl

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

librpc make ndr-standard a common library

librpc push NDR_SECURITY and NDR_DCERPC to the top level wscript files

There isn't any reson to keep these private to source3 and source4 and
more, and doing so creates problems for the top level build.

Andrew Bartlett

nsswitch: depend on dl

The previous merge in this area didn't preserve the 'dl' depedency,
which may be required on some platforms.

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

charset Remove use of {isupper,islower,toupper,tolower}_w functions

These now call the common _m functions that consider UTF16 code points.

This removes the code which will make up a 'lame' table in memory, as
this can just as correctly be handled by running the algorithm at runtime (which is to call toupper() and tolower() on characters < 128).

When used, a top level waf build will always locate the correct table
- in the build tree or outside - due to relinking the installed
binary.

Andrew Bartlett

s3-libads Remove MIT-specific krb5_princ_realm macro calls.

When compiled against heimdal, we need to use a more elegant API.

Andrew Bartlett

s3-waf use lib/util/wscript_build, and avoid duplicate subsystems

lib/util Make UTIL_LDB conditional on an s4 build

This allows this file to be used in common.

s3-waf: use SAMBA3_*() build rules in source3/build

this brings the s3 waf build much closer to the proposed s3build top
level build, using the same bld.SAMBA3_*() rules

There are a few renames of subsystems in here, with a 3 suffix where
it would create a conflict.

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

build: put links for libs in the right directory

when a library is declared as libdir/libname, still put the symlink in
bin/shared

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Fri Feb 18 06:03:57 CET 2011 on sn-devel-104

s4-util: removed the valgrind_strlen() routine

this was for a bug in valgrind from 7 years ago.

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

libutil: make waf rules usable by s3 waf build

we need samba-util to be s4 only for now, because of the debug system
differences

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

waf: improved error message for bad paths in SAMBA_LIBRARY()

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

waf: support building libraries with a directory prefix

SAMBA_LIBRARY('libsmb/smbclient') can now be built, which
distinguishes it from the binary 'smbclient'

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s3-waf: fixed the include paths for samba public libraries

this fixes the extra global includes for the s3 waf build to be
conditional on whether talloc, tevent and tdb are system libraries or
not.

This fixes a problem where in-tree includes could be used with system
libraries

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

charset: allow s3 waf build to use lib/util/charset

the only conflict is with 'CHARSET' itself, which now builds
conditionally on _SAMBA_BUILD_==4

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

waf: added SAMBA3_*() build rules to samba3.py

these provide samba build rule wrappers for Samba3 specific subsystems

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

waf: give a more sensible error on symlink_bin with missing directory

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

lib/util/charset use get_dyn_CODEPAGEDIR(), which is in common

This is provided by the s3 and s4 dynconfig implementations.

Andrew Bartlett

lib/util/charset split codepoints.c into it's own subsystem

lib/util/charset use a path to dynconfig.h that works in s3 and s4

lib/util/charset add functions isupper_m and islower_m

s4-auth: rename 'auth' subsystem to 'auth4'

this prevents conflicts with the s3 auth modules. The auth modules in
samba3 may appear in production smb.conf files, so it is preferable to
rename the s4 modules for minimal disruption.

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

build: remember the flags used for undefined variable errors

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s3build: added NMBDSOCKETDIR to dynconfig

this is needed for the s3 top level build

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s4-dsdb: operational module does not need auth any more

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

s3-includes: force _SAMBA_BUILD_=3 in source3 includes.h

this allows a common set of cflags for a top level combined build

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>

uid-wrapper: handle uwrap_enabled() as a macro

some s3 code defines uwrap_enabled() as a macro. Detect this, and
don't redeclare the functions

Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org>