s3-lib Remove unused str_ascii_charnum()

s3-lib Move strisnormal to it's only user in mangle_hash.c

libcli/ldap Re-add ldap_ndr.h as a public header

as noticed by Brad Hards

Andrew Bartlett

s4/libcli: do not use netbios name in NTLMv2 blobs w/o spnego

I have seen domain controllers rejecting NTLMv2 blobs presented to
NetrLogonSamLogonEx with LOGON_FAILURE when the MsvAvNbComputerName
was a FQDN or an IP address

I have not seen this field in NTLMv2 blobs send by Windows clients
when extended security was not available, so omitting the field
makes Samba similar to Windows.

This prevents errors with some smbtorture testcases that disable
spnego and when a target name is specified that is not a valid
netbios name.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Thu Apr 14 02:19:08 CEST 2011 on sn-devel-104

libcli: allow exclusion of netbios name in NTLMV2 blob

when no hostname is given, leave away the MsvAvNbComputerName part
of the ntlmv2 blob

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s4/libcli: do not announce NT error code support when it is disabled

when the support of NT error codes has been disabled in the options of a
CLI connection, support for them should not be announced during
protocol negotiation

Signed-off-by: Andrew Bartlett <abartlet@samba.org>

s3: only include smb profiling where needed.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Thu Apr 14 01:31:39 CEST 2011 on sn-devel-104

s3-waf: add --with-profiling-data configure option.

Guenther

s3: fix the build of --with-profiling-data.

Guenther

s3-includes: only include acl prototypes when needed.

Guenther

s3-includes: absolutely no need to globally include debugparse.h

Guenther

s3-includes: include lib/util/debug.h headers in includes.h not smb.h

Guenther

s3-includes: remove unused schema_types enum.

Guenther

s3-includes: move struct pipe_open_rec to srvsvc rpc server.

Guenther

Fix WAF build for sids2xids.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Thu Apr 14 00:33:53 CEST 2011 on sn-devel-104

s3: Add a test for sid2xids

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Add some debug to winbindd_sids_to_xids

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Use wbcSidsToUnixIds in create_local_token

Signed-off-by: Jeremy Allison <jra@samba.org>

nsswitch: Add wbinfo --sids-to-unix-ids

Signed-off-by: Jeremy Allison <jra@samba.org>

libwbclient: Add wbcSidsToUnixIds

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Export WINBINDD_SIDS_TO_XIDS via the winbind pipe

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Add wbint_Sids2UnixIDs idl & implementation

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Make idmap_find_domain public

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Add is_domain_online

Signed-off-by: Jeremy Allison <jra@samba.org>

nsswitch: wbinfo --lookup-sids

Signed-off-by: Jeremy Allison <jra@samba.org>

lib: Move next_token next to next_token_talloc

Signed-off-by: Jeremy Allison <jra@samba.org>

libwbclient: Add wbcLookupSids

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Add winbindd_lookupsids

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Add wb_lookupsids

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Add wbint_LookupSids

This will be called from wb_lookupsids to query remote DCs via lsa

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Make parse_sidlist public

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Simplify parse_sidlist

Signed-off-by: Jeremy Allison <jra@samba.org>

Add dom_sid_parse_endp

This returns a pointer to the first non-parsed character, along the lines of
strtoul for example.

Signed-off-by: Jeremy Allison <jra@samba.org>

Don't print "success" message after error message in change_file_owner_to_parent(). Remove "goto" in change_dir_owner_to_parent().

s3-build: finally remove references to ndr and dcerpc headers that were included globally.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Apr 13 23:07:05 CEST 2011 on sn-devel-104

s3-rpc_client: add and use rpc_client/rpc_client.h.

Guenther

s3-netapi: remove two shadowed variables.

Guenther

s3:WHATSNEW.txt: remove the "strict allocate" default value

commit dc36d75d5fddaa351025e8eb8140f401b66aeb9d
reverted the change to true, because it makes Samba
unusable on systems without Linux and a modern Filesystem.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Apr 13 21:21:10 CEST 2011 on sn-devel-104

s3-cli_pipe: fix timeout in rpc_pipe_open_tcp_port().

Make sure we use a timeout of 60 seconds, not 60 milliseconds...

This prevented us from successfully using the ncacn_ip_tcp client in a lot of
places, I guess.

Guenther

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Apr 13 18:59:19 CEST 2011 on sn-devel-104

s3-util_sock: very brief documentation for open_socket_out().

The most important bit is to document that timeout is in MILLIseconds, not
seconds.

Guenther

s3: Make send_nt_replies() static

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Apr 13 16:25:29 CEST 2011 on sn-devel-104

talloc: fix compiler warnings with -Wc++-compat

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Apr 13 14:33:23 CEST 2011 on sn-devel-104

s3: Fix Coverity ID 2470: STRAY_SEMICOLON

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Apr 13 12:57:43 CEST 2011 on sn-devel-104

s3: Use talloc_tos() in the S3 build

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Apr 13 09:30:55 CEST 2011 on sn-devel-104

build: Bring DYNCONFIG into samba-util library to avoid symbol duplication

When this was depended on directly as a subsystem, it ended up in
multiple libraries.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Wed Apr 13 08:39:35 CEST 2011 on sn-devel-104

s3-build Remove distinct LOCALEDIR subsystem

This is no different to the rest of dynconfig, and so should be dealt
with there.

Andrew Bartlett

s4-test: added a test for E_deshash()

this particularly checks the boundary conditions near passwords of
length 14 characters

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

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Wed Apr 13 07:31:55 CEST 2011 on sn-devel-104

s3-charcnv: Move convert_string() et al to lib/util/charset

This is the first step to this being the common convert_string
implementation.

Andrew Bartlett

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

libcli/auth Use convert_string_error to check LM hash calculation.

This allows us to know if the LM hash was built correctly or not.

NOTE: talloc_tos() is not available in the common code at this time.

Andrew Bartlett

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

lib/util/charset Add many more charset tests

This confirms that the behaviour of the convert_string() API (with the
process-wide iconv handle).

Andrew Bartlett

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

lib/util/charset Add convert_string_error()

This adds an interface that matches the source3/ convert string code.

Andrew Bartlett

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

lib/util/charset Make ASCII conversion validate it's input

We should not just strip the high bits off unicode strings being
converted to ASCII, we need to actually fail the conversion.

Andrew Bartlett

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

s3-selftest Add workaround for RAP test failure

The rap.sam test reads 0xFFFFFFFF as a string in the level 2
r->HomeDir attribute, which once we start validating ASCII strings
fails.  This restores a unchecked dos charset for this test only,
until it is determined if the client or server RAP code is at fault.

Andrew Bartlett

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

lib/util/charset Rename convert_string test to allow a 'non_handle' test

A future commit will test (with a subset of tests) the varient of this
function without _handle.

Andrew Bartlett

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

lib/util/charset Add more tests for convert_string_error_handle()

This helps define the semantics of this function very clearly,
particularly for partial and invalid inputs.

Andrew Bartlett

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

lib/util/charset Preserve 'pull' errors even when converting via UTF16

When we do not have a direct iconv handle between any two charsets, we
must go iva UTF16.  However, we should still return the same buffer
and error code as if we were able to go direct - including the partial
conversion and the error code.

This is important for locating the invalid multibyte character in the
stream, for example.

Andrew Bartlett

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

lib/util/charset Add tests for convert_string_error_handle

These confirm that the errno is set correctly and that we stop on a
partial multibyte character

Andrew Bartlett

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

lib/util/charset Add expected values for upper/lower case tests

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

lib/util/charset Fix and add public interface for convert_string_error_handle

It makes much more sense for this to match the source3/ interface and
return a bool.

Andrew Bartlett

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

s4/torture Fix calls to charcnv functions to always supply converted_size

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

lib/util: Make string_replace from s3 common

The s4 implementation didn't do multibyte strings, so was only good
for '/' which is known to be safe in all multibyte charsets.

Andrew Bartlett

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

lib/util ucs2_align is identical, put it in common

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

lib/util Move simple string routines into common code.

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

lib/util/charset Move source3/lib/util_unistr.c to the common code.

This file (largely) contains functions to deal with UTF16 strings.

Andrew Bartlett

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

s3-lib Move strdup_w to it's only user in mangle_hash.c

Andrew Bartlett

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

s3-lib Move isvalid83_w to mangle_hash.c

This means that there is no need for the 'valid.dat' table to be
loaded by anything other than smbd, so the unloader is also removed.

The concept of a 'valid dos character' has been replaced by the hash2
mangle method.

Andrew Bartlett

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

s3-lib make static and remove more _w functions

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

s3-lib Remove unused #define

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

s3-lib Remove unused skip_unibuf()

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

s3-lib: Remove unused _w functions.

In general we don't manipulate UTF16 strings internally, particularly
as they are also multibyte, so are no easier to work with than UTF8.

Andrew Bartlett

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

s3-lib Correct comment in strlen_w()

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

s3-lib Remove more unused fstring.c functions

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

s3-charcnv: make pull_ucs2 static

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

s3-lib: Remove unused pull_ucs2_fstring()

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

build: use readelf as a replacement for ldd

using readelf allows us to do a non-recursive library listing, which
is important to remove false positive symbol duplication

Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Wed Apr 13 04:37:33 CEST 2011 on sn-devel-104

libds: Make flag_mapping a library

This is a tiny library, but otherwise the functions end up in multiple
other libraries.

Andrew Bartlett

lib/util make UTIL_TDB a library

libcli/ldap pull LIBCLI_LDAP_MESSAGE and LIBCLI_LDAP_NDR into a library

This reduces symbol duplication

Andrew Bartlett

build: speed up SYMBOLCHECK code

this uses a nm and ldd cache to speed up the duplicate symbol checking
code

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

s3: For net ads join, create a krb5.conf

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue Apr 12 14:10:05 CEST 2011 on sn-devel-104

s3: Initialize output in libnet_Join

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

s3: Retry the join with the short name

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

s3-util: separate out cmdline helpers.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue Apr 12 13:06:27 CEST 2011 on sn-devel-104

s3: add some forward declarations.

Guenther

s3-netapi: use dcerpc_try_samr_connects().

Guenther

s3-netapi: use libnetapi_get_binding_handle().

Guenther

s3-netapi: add libnetapi_get_binding_handle().

Guenther

s3-librpc: let librpc/rpc/dcerpc.h include ndr and generated dcerpc headers.

Guenther

s3-rpc_client: move rpc_cli_transport structs and protos to rpc_transport.h

Guenther

s3/selftest Tear down the environment if we fail to connect

This removes the exit(1) that previously didn't tear down the
environment correctly, and instead just fails to set it up, allowing
the rest of the script handle that with more grace.

Andrew Bartlett

Autobuild-User: Andrew Bartlett <abartlet@samba.org>
Autobuild-Date: Tue Apr 12 01:53:37 CEST 2011 on sn-devel-104

s3-printing: Use become_user_by_session() function.

We create a fake connection here and don't have an vuid. So work with
the session_info directly here.

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Apr 11 22:56:12 CEST 2011 on sn-devel-104

s3-smbd: Added a become_user_by_session() function.

This uses the provided session_info instead of searching the user via
the vuid. This is useful to work with fake connnection you need to
create if someone connects directly to a rpc service.

Signed-off-by: Jeremy Allison <jra@samba.org>

s3-smbd: Added a change_to_user_by_session() function.

Signed-off-by: Jeremy Allison <jra@samba.org>

s3: Wrap creating the svcctl keys in a transaction

This makes the startup of smbd in make test much quicker and thus more reliable
(cherry picked from commit f1aa38b414e97d8687d0bebf65baa384f75301b4)

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Apr 11 22:09:58 CEST 2011 on sn-devel-104

s3: Make reg_backend_db.h includable by itself
(cherry picked from commit 6ba31d3bd29dd11b1b98859f4d8a65d9be7dac60)

s3: Fix Coverity ID 2472, UNINIT

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Mon Apr 11 17:14:35 CEST 2011 on sn-devel-104

s3: Fix Coverity ID 2473, UNINIT

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

s3: Fix timeout calculation in event_add_to_poll_args

In the initial phase nmbd does not yet have timed events. This led to nmbd
not correctly registering its names in make test and certainly everywhere
else.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Mon Apr 11 14:35:45 CEST 2011 on sn-devel-104

s3-modules: Fix debug message

Print child descriptor instead of parent.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Apr 11 11:48:42 CEST 2011 on sn-devel-104

Fix bug 8072 - PANIC: create_file_acl_common frees handle two times.

Caused by premature optimisation storing the parent ACL on the
module handle instead of (correctly) on the file fsp. Previous
code wasn't reentrant safe. This is less optimal but doesn't
crash in the specific case :-).

Jeremy.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Apr  9 02:05:15 CEST 2011 on sn-devel-104