Fix convert_string() to take a *converted_size arg. and return a bool.

Makes these interfaces much harder to misuse and easier to ensure error
checking.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Mar 30 23:59:37 CEST 2011 on sn-devel-104

s3-build: make it a little easier to build smbtorture4.

Only configure with --enable-socket-wrapper --enable-nss-wrapper
--enable-uid-wrapper, not --enable-developer which turns on at least one -Werror
switch.

Guenther

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

s3-vfs: try to fix build of aio_fork module.

Guenther

Change convert_string_internal() and convert_string_error() to bool return.

Move closer to makeing all convert_string_XXX functions return bool.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Mar 30 20:58:10 CEST 2011 on sn-devel-104

Fix the nstring calls to use the correct sizes.

This doesn't look like it has anything to do with character set conversion, but it does :-).

Turns out one of the *really* significant differences between
convert_string() in source4 and source3, is that the one in
source3 will return 0 for byte length converted when called
with dest_len = 0 whereas the one in source4 returns (size_t)-1
and sets errno to E2BIG.

Allow the ndr_string code to cope with the (arguably correct)
way that the source4 implementation works. This code only gets
excercised in the print spooler tests, which aren't run in source4,
which is why this bug has lasted for so long.

You don't want to know how long it took me to find this :-).

Jeremy.

s3: Fix g_lock_lock after the select/poll conversion

Without clustering we don't have an fd to listen on, and sys_poll
needs one element of space

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar 30 18:36:50 CEST 2011 on sn-devel-104

s3-smbd: try to fix the irix build.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar 30 17:17:58 CEST 2011 on sn-devel-104

s3-vfs acl modules: more non-linux build fixes.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar 30 16:00:02 CEST 2011 on sn-devel-104

s3-passdb: move LOOKUP_NAME_ flags to passdb where they belong to.

Guenther

s3-passdb: move some passdb defines to passdb.h

Guenther

s3-proto: pure cosmetics, remove some generated comments.

Guenther

s3-winbind: try to fix the build on hosts w/o winbind support.

Guenther

s3-vfs_solarisacl: try to fix the solaris build.

Guenther

s3: Use cli_ntcreate to when listing snapshots

This works for directories as well

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar 30 14:37:02 CEST 2011 on sn-devel-104

s3: Fix Coverity ID 2041, "UNUSED_VALUE"

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar 30 13:49:35 CEST 2011 on sn-devel-104

lib/util/charset Add tests for convert_string_talloc_handle()

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed Mar 30 13:02:47 CEST 2011 on sn-devel-104

lib/util Move base64 functions into lib/util/base64.c

Andrew Bartlett

s3:lib make lazy_initialize_conv() static

lib/util/charset Remove pointless static bool initialised

s3: try to fix the build on some non-linux buildfarm machines.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar 30 11:39:31 CEST 2011 on sn-devel-104

s3: Fix Coverity ID 2217: RESOURCE_LEAK

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Mar 30 10:45:44 CEST 2011 on sn-devel-104

s3: Fix Coverity ID 2220: RESOURCE_LEAK

s3: Fix Coverity ID 2221: RESOURCE_LEAK

s3: Fix Coverity ID 2222: RESOURCE_LEAK

s3: Fix Coverity ID 2223: RESOURCE_LEAK

s3: Fix Coverity ID 2236: REVERSE_INULL

We have already dereferenced "b", and the other calls in this loop can only
have worked with a valid handle

s3: Fix Coverity ID 2332: MISSING_BREAK

s3: Fix Coverity ID 2237: REVERSE_INULL

tdb: Fix Coverity ID 2238: SECURE_CODING

s3: Fix Coverity ID 2304: FORWARD_NULL

s4:lib/policy/gp_ldap.c - make more use of LDB result constants

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Wed Mar 30 09:33:38 CEST 2011 on sn-devel-104

s4:lib/registry/registry.h - align a copyright line in header comment

talloc - improve doxygen comment of "talloc_move"

Express better that this should be a pointer of a pointer.

Reviewed-by: Tridge

s4:smb_server/smb/trans2.c - "talloc_move" isn't strictly necessary here

Since the "set" will be free'd afterwards "talloc_steal" is enough.

Reviewed-by: Tridge

s3-ctdb_wrap: remove unused variable from db_ctdb_fetch_persistent().

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Mar 30 02:00:50 CEST 2011 on sn-devel-104

s3-rpc_server: move access check functions out of samr server.

Guenther

s3: only include ntioctl.h where needed.

Guenther

s3: only include lib/privileges.h where needed.

This finally removes the global lsa.h inclusion.

Guenther

s3: include ../librpc/gen_ndr/lsa.h where needed.

Guenther

s3-passdb: move lsa_dom_info and lsa_name_info out of smb.h into passdb.

Guenther

s3-includes: only include ntdomain.h where needed.

Guenther

pidl: add an include for "ntdomain.h" for s3 server stubs.

Guenther

s3-messages: make ndr_messaging.h part of messages.h.

Guenther

s3-messages: only include messages.h where needed.

Guenther

s3-prototyes: user_in_netgroup and user_in_list moved to auth, out of smbd.

Guenther

s3-includes: finally remove global include of auth.h.

Guenther

s3-auth: vfs modules need auth.h

Guenther

s3-auth: printing needs auth.h

Guenther

s3-auth: smbd needs auth.h

Guenther

s3-auth: rpc_server needs auth.h

Guenther

s3-auth: use auth.h where needed.

Guenther

s3-auth: move auth prototypes to auth/proto.h.

Guenther

s3-libsmb: put namequery headers to nmblib.h

We might find a better name for it and merge other namequery related things as
well here...

Guenther

s3-locking: move locking prototypes out of proto.h.

Will later become part of locking.h

Guenther

s3: finally remove global include of smbd prototypes.

Guenther

s3: rpc_server needs parts of smbd.

Guenther

s3: printing needs parts of smbd.

Guenther

s3: auth also needs parts of smbd.

Guenther

s3-vfs: include smbd/smbd.h in vfs modules.

Guenther

s3: include smbd/smbd.h where needed.

Guenther

s3-smbd: add smbd.h and move smbd prototypes to smbd/proto.h.

Guenther

s3: move filename_util.c out of source3/smbd to source3/lib.

Guenther

s3-build: only include intl protos where needed.

Guenther

s3-passdb: finally stop including passdb.h globaly.

Guenther

s3-passdb: use passdb headers where needed.

Guenther

s3-passdb: move util_builtin.c out of passdb.

Guenther

s3-passdb: move util_unixsids.c out of passdb.

Guenther

s3-passdb: move util_wellknown.c out of passdb.

Guenther

s3-winbindd: copy acct_info to wb_acct_info so we dont need passdb for it.

Guenther

s3-passdb: add machine_sid.h and lookup_sid.h

Guenther

s3-proto: add some forward declarations.

Guenther

s3-passdb: add passdb.h where needed.

Guenther

s3-passdb: move mapping.h into passdb.h, its needed here.

Guenther

s3-passdb: move passdb headers to passdb/proto.h.

Guenther

s3-groupdb: move groupdb headers to groupdb/proto.h.

Guenther

s3: create_builtin_users/administrators belongs to passdb not auth.

Guenther

s3-build: only include transfer_file.h where needed.

Guenther

registry: create and use shared libcli/registry/util_reg.h header.

Guenther

s3-includes: no point in including all security headers globally.

Guenther

s3-includes: remove EILSEQ check, this is handled by
../lib/replace/system/iconv.h already.

Guenther

s3-includes: only include system/filesys.h when needed.

Guenther

s3-includes: remove global include of "system/network.h"

Guenther

s3-includes: only include system/passwd.h when needed.

Guenther

s3-includes: only include system/shmem.h when needed.

Guenther

s3-includes: remove global include of "system/select.h".

Guenther

s3-includes: only include system/terminal.h when needed.

Guenther

s3-includes: only include system/capability.h when needed.

Guenther

s3-includes: only include system/glob.h when needed.

Guenther

s3-includes: only include system/iconv.h when needed.

Guenther

s3-includes: only include system/syslog.h when needed.

Guenther

s3-trans2: only include trans2.h where needed.

Guenther

s3-winbind: remove global inclusion of libwbclient.

Guenther

lib/util/util_pw: share more code between lib/util/util_pw.c and source3/lib/username.c

Guenther

lib/util/util_pw: share sys_get{pw,gr} group of calls.

Guenther

lib/util/util_pw: temporary rename of getpwnam_alloc and getpwuid_alloc.

Guenther

Ensure convert_string_XXX is always called with a valid converted_size pointer.

Preparation for cleaning up this API.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Mar 29 21:01:49 CEST 2011 on sn-devel-104

charconv: Fix the slow-path character conversions

This reverts a part of 0189087e257f. That one might have fixed the fast path,
but it broke the slow path. convert_string_internal returns 0/-1 despite the
size_t result type and the misleading comment. If you follow the path to
smb_iconv and for example iconv_copy(), you will see that this routine returns
0 even after it copied something.

How to check: Use German locale, and do an smbclient "allinfo" on a file
modified in march (März in German). Notice the "ä". Before 0189087 this printed
correctly, after 0189087 it cut off the fields after the M for März.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Tue Mar 29 19:17:41 CEST 2011 on sn-devel-104

s3-libsmb: remove duplicate NTSTATUS variable.

Guenther

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue Mar 29 17:00:26 CEST 2011 on sn-devel-104

s3-waf: fix spoolssd compile.

Guenther

Signed-off-by: Günther Deschner <gd@samba.org>