s3: Fix Coverity ID 83: RESOURCE_LEAK

s3: Convert cli_raw_tcon to cli_smb

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 27 19:50:21 CEST 2011 on sn-devel-104

s3: Fix a memleak

s3: Replace a use of cli_errstr

s3: Replace cli_negprot_sendsync() by cli_negprot_send()

s3: Remove a pointless static

s3: Remove a use of cli_send_smb

s3: Remove two calls to cli_errstr

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 27 13:17:47 CEST 2011 on sn-devel-104

s3: Fix Coverity ID 2296: UNUSED_VALUE

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sun Mar 27 12:12:35 CEST 2011 on sn-devel-104

s3: Fix Coverity ID 2327: FORWARD_NULL

Make check_ft_info robust against an invalid type.

Günther, please check!

s3: Fix Coverity ID 2328: FORWARD_NULL

Make startsmbfilepwent robust against an invalid open type

s3: Fix Coverity ID 2329: RESOURCE_LEAK

s3: Fix Coverity ID 2330: RESOURCE_LEAK

s3: Fix Coverity ID 2331: RESOURCE_LEAK

s3: Fix an uninitialized variable read

The "is_valid_policy_handle" in the exit path reads uninitialized data

Lets see if this fixes Coverity ID 2172 which complains about h being
dereferenced in the dcerpc_winreg_CloseKey.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat Mar 26 14:24:28 CET 2011 on sn-devel-104

s3: Fix Coverity ID 2326, CHECKED_RETURN

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

This reverts commit 042aafb87df6c05877b8fc7ef0d44877689d860a.

I need to test against OS/2 to see if this is something Win28K
breaks for older clients before allowing this.

Jeremy.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Mar 26 01:19:55 CET 2011 on sn-devel-104

Fix bug 8040 - smbclient segfaults when a Cyrillic netbios name or workgroup is configured.

As discovered by David Disseldorp <ddiss@suse.de>, convert_string_talloc()
doesn't always return consistent results for a zero length string. The
API states an incoming string must *always* contain the terminating null,
but unfotunately too much code expects passing in a zero source length
to return a null terminated string, so at least ensure we return a
correct null string in the required character set and return the
correct length.

Also ensure we cannot return a zero length for a converted string
(we ensure that the returned buffer is always allocated and zero
terminated anyway) as calling code depends on the fact that returning
true from this function will *always* return a non-zero length (as
it must include the terminating null).

Jeremy.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Fri Mar 25 23:25:40 CET 2011 on sn-devel-104

s3-netapi: fix memoryleak while not using talloc_tos() in cli_get_session_key() usage.

Thanks Volker!

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Fri Mar 25 18:21:18 CET 2011 on sn-devel-104

s3: Compile IDL files in autogen, some configure tests need this

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Mar 25 16:35:30 CET 2011 on sn-devel-104

Revert "s3-includes: fixed CONST_DISCARD() to actually discard const"

This reverts commit 37261928bec145f4cbd85f3e409063563c00ac9e.

In source3 this creates a warning of the form

passdb/pdb_ldap.c:4565: warning: passing argument 2 of ‘pull_utf8_talloc’ from incompatible pointer type
include/proto.h:444: note: expected ‘char **’ but argument is of type ‘char ***’

for every use of CONST_DISCARD.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Mar 25 13:16:35 CET 2011 on sn-devel-104

charcnv: ensure convert_string_error doesn't conflict with s3 function

use _handle for handle based functions

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

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Fri Mar 25 05:22:24 CET 2011 on sn-devel-104

s3-charcnv: fixed converted_size return in fast paths

s3-charcnv: removed unused function

lp_failed_convert_char() is not needed any more

lib/util/charset rename iconv_convenience to iconv_handle

This better reflects what this structure is

Andrew Bartlett

s3-charcnv: convert_string_internal() should not display errors

debug error display happens in the convert_string() outer function

s3-string: removed the conv_silent global

use convert_string_error() instead

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

s3-string: removed last use of conv_silent

This replaces the push_ascii_nstring() implementation with a call to
convert_string_error()

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

s3-charcnv Add convert_string_error()

This function returns errors rather than printing them.

Andrew Bartlett

charcnv: added convert_string_error()

this will allow us to remove conv_silent

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

s3-includes: fixed CONST_DISCARD() to actually discard const

a cast is not enough

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

unistr: moved some UCS2 macros into util_unistr

we need to move towards eliminating smb_ucs2_t. This moves a couple of
the related macros into the only file they are used in

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

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