libcli: created smb_constants.h

this starts the (long!) process of moving some of the SMB constants
into common files. This just moves the FLAGS2_ defines, which are
needed for common string routines (for FLAGS2_UNICODE_STRINGS)

tdb: rename convert_string() to tdb_convert_string()

this prevents a conflict with the convert_string() in samba

s3-libsmb: work our correct length for EA strings

s3-string: sec_len==-1 support is no longer needed

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

s3-dfs: removed the last user of sec_len==-1 in pull_*_base_talloc()

hopefully there aren't any others lurking in the code

s3-lib: make pull_ucs2_base_talloc static

it is local to charcnv.c

lib: remove unused pieces of string_wrappers.h

lib: move the string wrappers from source3/include to common lib/util

this will allow the common charcnv library to use the string wrappers

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

s3-string: moved fstring functions into their own file

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

s3:dbwrap_ctdb: fix non-locked fetch on persistent db's causing corruption

When doing a non-locking fetch on a record of a persistent db when no
transaction is running, the old behaviour was to fetch locally and
do a ctdb call when the record was not found in the local db.
The call is useless for persistent dbs anyway since they are only
written to using transactions and hence kept in sync, but it is
also harmful, because a ctdb call will bump the record RSN when it
does actually migrate the record from one node to another.

Recently, ctdb has been changed to make all calls do a migration.
This uncovered the client misbehaviour for persistent dbs, because
now _each_ non-locking fetch will render the persistent db inconsistent:
A subsequent transaction which touches the record in question will
fail because the RSNs are out of sync.

This patch fixes this old bug.

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Fri Mar 25 01:26:32 CET 2011 on sn-devel-104

s3-charcnv: remove unused labels.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Thu Mar 24 23:54:25 CET 2011 on sn-devel-104

s3-rpc_server: remove two uneeded include registry/reg_parse_prs.h

Guenther

s3-ldapsam: fix ldapsam_create_user() with existing posix accounts.

We were not taking into account the existing posix attributes and thus failed
while trying to add a 2nd uid attribute.

Found by Sumit.

Guenther

s3-netapi: use cli_get_session_key() in netapi.

Guenther

Fix bug 8021 - Incorrect string termination in volume/volume_name for TRANS2-QUERY_FS_INFO/Info Volume.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Thu Mar 24 23:07:09 CET 2011 on sn-devel-104

Fix is_myname_or_ipaddr() to be robust against strange DNS setups.

If IPv6 DNS names are turned on, but Samba isn't configured to
listen on an IPv6 interface, then is_myname_or_ipaddr() can return
false on a valid DNS name that it should detect is our own. If the
IPv6 addr is returned by preference, then looking at the first addr
only causes is_myname_or_ipaddr() to fail. We need to look at all the
addresses returned by the DNS lookup and check all of them against
our interface list. This is an order N^2 lookup, but there shouldn't
be enough addresses to make this a practical problem.

Jeremy.

s3-epmapper: fix vars init and return errors

Properly initialize variables at each cycle.
Convert to the right error when returning EPMAPPER ones.

Autobuild-User: Simo Sorce <idra@samba.org>
Autobuild-Date: Thu Mar 24 20:43:49 CET 2011 on sn-devel-104

librpc: Return an error if we a broken floor.

Pair-Programmed-With: Simo Sorce <idra@samba.org>

cleanup: fix some trailing spaces

s3-epmapper: Fix allocation of data on the wrong context

p->mem_ctx is the short-lived per request context, while this data is long
lived, allocate on p instead.

s3-epmapper: Make sure we work on a description duplicate.

librpc: Added a dcerpc_binding_dup() function.

s3-epmapper: Refactor the cleanup of endpoints.

s3-epmapper: Use strcmp instead of strequal and check IPv6.

s3-epmapper: Increase debug levels.

s3: Fix some errno assignments in SMBC_opendir_ctx

Do the assignment as the last action to make sure it's not overwritten

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Mar 24 17:31:57 CET 2011 on sn-devel-104

s3: Fix Coverity ID 2325: RESOURCE_LEAK

charcnv: removed the allow_badcharcnv and allow_bad_conv options to convert_string*()

we shouldn't accept bad multi-byte strings, it just hides problems

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Thu Mar 24 01:47:26 CET 2011 on sn-devel-104

charset: remove another allow_badcharcnv check

better to fail only if there is a bad character

s3-prototypes: remove protos of some dead functions.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Thu Mar 24 00:52:01 CET 2011 on sn-devel-104

librpc: bring more librpc prototypes in common.

Guenther

s3-rpc_client: actually call lsa_open_policy2() in rpccli_lsa_open_policy2().

Found by Sumit Bose <sbose@redhat.com>, thanks!

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar 23 20:12:38 CET 2011 on sn-devel-104

s3-smbta-util: fix some buildwarnings.

Guenther

s3-globals: silence build warning in smbd_messaging_context().

Guenther

s3-proto: remove some duplicate prototypes.

Guenther

s3-includes: remove unused cmdline_lp_ctx.

Guenther

s3-epmapper: Log error if we can't register the endpoint.

Autobuild-User: Simo Sorce <idra@samba.org>
Autobuild-Date: Wed Mar 23 18:06:54 CET 2011 on sn-devel-104

s3-epmapper: Setup epm in smbd to forward np requests.

s3-epmapper: Remove unregister on shutdown.

This is done automatically now.

s3-epmd: Cleanup endpoints on service pipe disconnect.

s3-epmapper: Added function to delete endpoint entries.

s3-rpc_server: Added disconnect callback function.

s3-rpc_server: Rename req to subreq.

s3-rpc_server: Implement an endpoint monitor loop.

s3-rpc_server: Added a memory context to the ep regsiter state.

s3-librpc: Leave the epm registration connection open.

s3-epmd: Cleanup endpoint mapper correctly.

s3-epmapper: Shutdown the embedded epmapper cleanly.

s3-epmapper: Added a cleanup function.

s3-epmapper: Use DCERPC_AUTH_LEVEL_CONNECT for ep ncalrpc.

s3-rpc_server: Rename system_user to ncalrpc_as_system.

s3: Fix a shadowed declaration warning

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar 23 17:19:01 CET 2011 on sn-devel-104

s3: Include prctl where it is actually used

s3: Attempt to fix the build on FreeBSD

s3: Fix Coverity ID 2201, NULL_RETURNS

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar 23 13:06:20 CET 2011 on sn-devel-104

s3: Fix Coverity ID 2202, NULL_RETURNS

s3: Fix Coverity ID 2203, NULL_RETURNS

s3: Fix a (invalid) uninitialized variable warning

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar 23 11:13:14 CET 2011 on sn-devel-104

async_rec/async_sock.c - add an additional "const"

In order to suppress a build warning.

Acked-by: Volker and Metze
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Wed Mar 23 10:26:23 CET 2011 on sn-devel-104

ndrdump - make "in_pipes" and "out_pipes" const

In order to suppress a build warning.

lib/util: Merge basic string length and comparison functions

These functions now use the codepoints for more accurate string
handling and now form common code.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed Mar 23 08:21:54 CET 2011 on sn-devel-104

s3-lib prepare Samba3 to use common codepoint based string functions

This patch changes the source3 util_str.c functions so that the next
patch just contains the move into common code, without code changes.

Andrew Bartlett

s3-safe_string: Add checked_strlcpy()

This is strlcpy, just with an extra check of the parameters with
sizeof(), use only where that works.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed Mar 23 03:39:35 CET 2011 on sn-devel-104

s3-safe_str: Futher simplify the macros by removing indirection

Now that we don't need to pass in the function name and string,
another level of indirection can be safely removed, and the operation
of these macros made much clearer.

Andrew Bartlett

s3-lib Remove the clobber_region() code.

This code wrote to the full buffer in fstrcpy(), pstrcpy() and other
fixed-length string manipulation functions.

The hope of this code was to find out at run time if we were mixing up
pstring and fstring etc, and to record where this came from.  It has a
runtime performance impact (particularly if compiled with
--enable-developer).

It is being removed because of the complexity it adds, and the
distinct lack of bugs that this complexity has been credited in
finding.

The macro-based compile-time checking of string sizes remains.

Andrew Bartlett

charcnv: remove the automatic fail on allow_badcharcnv true

we should just fail the call if the string really is bad

waf: prevent an error in the symbol checking code

s3-config: say which config file we failed to open

saves having to strace it to work that out

fault: fixed smb_panic() prototypes

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Wed Mar 23 01:51:44 CET 2011 on sn-devel-104

fault: fixed call to fault_setup() to use new syntax

fault: moved fault.c into common library

fault: switch s4 to use the common fault.c

fault: get fault.c ready for use by s4

this moves the s3 specific dumpcore code into source3/lib/dumpcore.c,
and uses a function pointer to setup which smb_panic call to use

fault: moved s3 fault.c to top level

fault: rename fault.c to fault_s4.c

this is in preparation for merging the s3 fault code into common

s3:WHATSNEW: document changes of the id mapping system

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Tue Mar 22 23:57:29 CET 2011 on sn-devel-104

s3:selftest: fix Samba3.pm deprecated idmap config

Replace deprecated idmap uid and gid option with new
idmap config * : range

s3:idmap: make sure that the id mapping system is initialized for first access

s3:loadparm: set the default "idmap config * : backend" in initialize_globals().

s3:loadparm: deprecate "idmap uid/gid/backend" and have them set "idmap config * : range/backend"

s3:WHATSNEW: remove mention of "idmap read only" as new parameter

s3:docs: remove documentation of "idmap read only" which was removed.

s3:loadparm: remove unused parameter "idmap read only".

This has not been released yet and is now useless since we
use the "idmap config * : read only = ..." syntax.

s3:idmap: remove (now) unneeded function parse_idmap_module()

s3:idmap: simply call idmap_init_named_domain for "*" in idmap_init_default_domain

The default domain "*" is now treated exactly the same as other explicitly
configured domains.

s3:idmap: remove passdb argument from idmap_init_domain()

s3:idmap: remove the params argument from the init function

s3:idmap: remove special treatment of domain "*" from idmap_ldap_init.

The default config via domain "*" is now treated just as the explicit
domain configs.

s3:idmap: remove use of params from idmap_ldap_init - it is not used any more

s3:idmap: remove the special treatment of the default domain "*" from idmap_init_domain

s4:python bindings - handle NULL returns from "loadparm_init_global"

Reviewed-by: Jelmer
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Tue Mar 22 19:52:57 CET 2011 on sn-devel-104

s4:param/loadparm.c - check for OOM

Return NULL if the loadparm context hasn't been allocated

Reviewed-by: Jelmer

s4:auth/system_session.c - check for OOM

Reviewed-by: Jelmer

s4:ntvfs/posix: name->dos.attrib isn't initialized in pvfs_access_check_create()

That's why we have the 'container' parameter to indicate
the caller wants to create a directory.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Tue Mar 22 17:39:11 CET 2011 on sn-devel-104

s3:docs: fix xml of "printnotifybackchannel" parameter (missing closing tag)

Autobuild-User: Michael Adam <obnox@samba.org>
Autobuild-Date: Tue Mar 22 13:57:25 CET 2011 on sn-devel-104

s3-fault: removed the cont_fn from fault_setup()

cont_fn() was supposed to be a way to continue after a seg fault. It
could never be called however, as smb_panic() from fault_report()
could never return, as dump_core() never returns at the end of
smb_panic()

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Tue Mar 22 05:07:58 CET 2011 on sn-devel-104

libcli/security: make sure that we don't grant SEC_STD_DELETE to the owner by default

In the file server SEC_STD_DELETE is granted on the file/directory
or by FILE_DELETE_CHILD on the parent directory.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Mar 21 23:25:05 CET 2011 on sn-devel-104

s3:smbd: access checks should not depend on share mode flags

metze

s4:ntvfs/posix: grant SEC_STD_DELETE if the parent grants SEC_DIR_DELETE_CHILD

metze

s3: Fix Coverity ID 1048, CHECKED_RETURN

This is a real bug: tevent_req_set_endtime already calls tevent_req_nomem.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Mar 21 16:29:22 CET 2011 on sn-devel-104