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

s3: Fix Coverity ID 770, REVERSE_INULL

We dereference "res" in various places, no point in checking. All current
callers send "res!=NULL".

idmap.idl: Fix whitespace

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Mar 21 13:36:48 CET 2011 on sn-devel-104

pyldb: minor fixup, fix a memory leak

Autobuild-User: Matthieu Patou <mat@samba.org>
Autobuild-Date: Sun Mar 20 12:13:50 CET 2011 on sn-devel-104

pyldb: add more tests for control manipulation

dsdb: read acl, sd can be null and ret == LDB_SUCCESS

dsdb: acl_read fix a missed talloc_steal

upgradeprovision: Fix typo

ldb:fix control parsing for dirsync

ldapcli: allocate control value on the control object not on the control container

s3: Fix early tldap_search cancels

A callback of tldap_search_send might not interested in the rest of the results
and could do a TALLOC_FREE of the search request. In this case, "subreq" is
already free'ed. So we have to set it to pending before the callback is called.
The TALLOC_FREE of the search request will set it to non-pending again via
tldap_msg_destructor.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 20 11:26:57 CET 2011 on sn-devel-104

lib/replace/test: Add missing include for prototype.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sat Mar 19 16:45:56 CET 2011 on sn-devel-104

rpc_server/remote: Add missing prototype.

lib/registry/tests: Fix prototypes.

selftest/Samba3: use 'debug pid=yes'

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sat Mar 19 10:55:34 CET 2011 on sn-devel-104

s3:smbd: remove uses of "./" in unlink_internals()

Commit 78b931ce6c840ade646c1c6030fa2d6acb77208b tried to do that,
but didn't have any effect.

metze

s3: use getgrset() when it is available

When getgrouplist() is not defined, use getgrset() if it is defined
instead of using the initgroups() + getgroups() combo

Major contributions from Yannick Bergeron <yaberger@ca.ibm.com>

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat Mar 19 10:09:38 CET 2011 on sn-devel-104

pidl: Fix prototype for server init functions.

Autobuild-User: Jelmer Vernooij <jelmer@samba.org>
Autobuild-Date: Sat Mar 19 04:07:40 CET 2011 on sn-devel-104