s3-docs: Add more verbose description of "username map cache time".

s3-docs: Remove older documentation of smb2 parameters.

Jeremy, these parameters were already documented. My fault.
Sorry for the noise!

Karolin

s4: Remove trailing whitespaces

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

ldb: Mark _DEPRECATED_ ldb_msg_diff() and ldb_msg_canonicalize() functions

They are not quite safe to use (requires caller to steal
resulting message in own context) and may lead to holding
memory for too long.

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

s4-dsdb: use ldb_msg_normalize() in ldb_msg_difference()

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

s4-test: Use ldb_msg_normalize() in sqlite3 backend

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

s4-test: Use ldb_msg_normalize() in torture/rpc/dssync.c test

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

s4-dsdb: use ldb_msg_normalize() in ldbadd-process_file()

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

s4-dsdb: use ldb_msg_normalize() in source4/lib/ldb/common/ldb.c

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

s4-dsdb: use ldb_msg_normalize() in source4/dsdb/schema/schema_set.c

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

s4-ldb: Add ldb_msg_normalize() to accept a memory context from client

Previos implementation from ldb_msg_canonicalize()
was moved into this function and now ldb_msg_canonicalize()
is based on ldb_msg_normalize()

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

s4-ldb: Use _ldb_msg_add_el() in ldb_msg_add()

Previous implementation was 'leaking' attribute name
string, that is allocated by ldb_msg_add_empty()

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

s4-ldb: Use _ldb_msg_add_el() in ldb_msg_add_empty()

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

s4-ldb: Add separate function to add empty element into ldb_msg

It just adds another element, nothing more.
Caller is responsible to fill-in the added element and
determine how to handle data allocation contexts.

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

s4-ldb: Write more explanatory comment for ldb_msg_add()

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

s4-pyldb: Use ldb_msg_difference() in py_ldb_msg_diff()

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

s4-test: Use ldb_msg_difference() in torture/rpc/dssync.c test

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

s4-tools: use ldb_msg_difference() in ldbedit - modify_record()

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

s4-dsdb/schema/schema_set.c: fix trailing spaces and comments spelling

Few comments split on several lines also...

(Sorry Metze, I know you hate reviewing "and this, and that"
type of patches, but those are just cosmetics)

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

s4-dsdb: use ldb_msg_difference() in source4/dsdb/schema/schema_set.c

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

s4-ldb: Implement ldb_msg_difference() function to accept a memory context from client

Old implementation from ldb_msg_diff() was moved into
this this function but with changed interface
so that a memory context may be passed.

ldb_msg_diff() function is now based on ldb_msg_difference(),
which fixes a hidden leak - internal ldb_msg object
(returned from ldb_msg_canonicalize) wasn't freed
and stays attached to ldb_context for the connection lifetime.

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

s3-auth: Use talloc hierarchies to properly free auth_ntlmssp_state contexts

Turn auth_ntlmssp_end into a destructor and attach it to auth_ntlmssp_state.
Remote auth_ntlmssp_end and use TALLOC_FREE in the callers.

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

s3-auth: auth_make ntlmssp_state the parent context

There is no need for a separate mem_ctx member.
Also make the ntlmssp_state a children of auth_ntlmssp_state
Also cleanup auth_ntlmssp_end to free only what is not automatically freed

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

s3-auth: Simplify how we free the auth_context

Turn the freeing function into a destructor and attach it to the
auth_context.
Make all callers TALLOC_FREE() the auth_context instead of calling
the free function.

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

s3-auth: Cleanup and readability fixes

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

s3-ntlmssp: Remove ntlmssp_end and let the talloc hierarchy handle it.

All the members are children of ntlmssp_state anyway.

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

s3: Actually use the usecs in aio_fork_suspend

Jeremy, please check!

s3: Fix an uninitialized variable

s3: Work better without the aio sighandler

Refuse async I/O if we can't set up the signal handler

s3: Fix some nonempty blank lines

s3: Remove a direct use of procid_self()

s3-rpc_client: Fixed a segfault in rpccli_samr_chng_pswd_auth_crap().

This fixes the WINBIND-WBCLIENT test. The test set
old_lm_hash_enc_blob.length to 0 and we don't check the length here. So
the memcpy segfaulted.

s3-waf: Fixed the build.

s3: Don't use as-needed

This is just broken. RHEL5 can't deal with it, FreeBSD8 is broken with it, and
who knows what other platforms are broken. I can rather live with a system that
links in a bit too much than a system that does not work at all.

pam_winbind: Fix shadowing a global on FreeBSD

FreeBSD defines a function pam_error()

s3-dcerpc: Fix build and remove unneeded headers

Sorry, botched a rebase.

s3-dcerpc: Move mere processing functions from srv_pipe_hnd.c to srv_pipe.c

Fix a typo

s3-dcerpc: Move common cli/srv functions into a common file

s3: Really fix the 64-bit warnings

Hmm. Forgot to git commit this piece, sorry :-(

s3: Fix some 64-bit warnings

s3: Right now we do not have includes in Makefile.in

Skip the expansion step on non-GNU-make systems

s4 pam_winbind: don't build if pam header are not installed

s4 nss_switch: fix build problem on computer without the iniparser

Although the build was ok on my workstation it appears that on build
server it was not because the include path was not correct.

s4-waf: add the missing bits for builing nss_winbind and pam_winbind

This is something that was not picked up during the migration to waf

s4 unittests: add blackblox test for ktpass

s4: create a simple version of ktpass

This script is intended to be a replacement for the ktutil of Windows.
It's use is for exporting keytab that will be used for kerberized
services.

s3-build: add LIBNDR_XATTR_OBJ.

Guenther

Fix bug #7570 - New smb2 max parameters are undocumented.

Jeremy.

s3-dcerpc: rename SCHANNEL_SIG_SIZE to NL_AUTH_SIGNATURE_SIZE and move to IDL.

Guenther

s3-perfcount: make it possible to compile the example perfcount daemon.

Guenther

s4-smbtorture: run all testcases with bigendian push as well in RPC-BIND.

Guenther

s4-smbtorture: rearrange testlist in RPC-BIND.

Guenther

s4-smbtorture: make sure to wipe out dcerpc auth flags before RPC-BIND starts.

Guenther

s3-dcerpc: Fix ability to receive Big Endian PDUs

smbtorture shell: Allow any config option to be set.

Not sure what I was thinking by restricting the shell to "torture:"
parametric options. It's really extemely useful to be able to set
arbitrary options.

Make the "map to guest" parameter work correctly with NTLMSSP (spnego
and raw) under SMB2. Still need to investigate fixing this with krb5
auth (does this make sense ?).

Jeremy.

s4-loadparm: 2nd half of lp_ to lpcfg_ conversion

this converts all callers that use the Samba4 loadparm lp_ calling
convention to use the lpcfg_ prefix.

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

s4-loadparm: change lp_*() to lpcfg_*() and provide s3 compatibility lp_*()

This patch changes all Samba4 loadparm functions that take a
loadparm_context or a loadparm_service structure to use a lpcfg_*()
prefix. It then creates lp_*() functions which are wrappers around the
lpcfg_*() functions. This should allow us to mix Samba3 and Samba4
code which uses loadparm calls.

This first patch just makes the manual changes to the core loadparm
code. The tree will not build with just this patch. The next patch
replaces all of the callers, and was generated automatically with a
perl script.

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

WHATSNEW: Start 3.6.0pre1 release notes.

Karolin

libcli/netlogon: re-enable debugging.

Now that we do not share binary objects anymore, we can safely enable
debugging here again.

Guenther

This reverts commit 3eb122069b9f81196ac658375fcb828924af3e94.

s3-registry: use some prs macros to the only place where they are used.

Guenther

s3-waf: fix the build.

Guenther

s3-dcerpc: fix c++ warning.

Guenther

Move the remnants of rpc_parse code under registry/

The registry code is the only last user of this code.
Move everything under registry/ and hope someone will get rid od
it completely from there in the future.

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

s3-parse_prs: Remove unuesd functions

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

s3-dcerpc: Use a local function to dump pdu regions

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

s3-misc: Move smb_io_time() to regfio.c

This is the last file using this function and we do not want anyone
else to keep using hand marshalled stuff anyway.
So make it also private to that file.

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

s3-dcerpc: Use DATA_BLOB for pipes_struct input data

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

s3-dcerpc: Use DATA_BLOB instead of prs_struct for req_data

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

s3-dcerpc: Use DATA_BLOB instead of prs_struct for rpc_out

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

s3-dcerpc: Avoid unnecessary memory allocation for dcerpc_ctx_list

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

s3-dcerpc: Use DATA_BLOB instead of prs_struct for reply_pdu

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

s3-dcerpc: Use DATA_BLOB instead of prs_struct for incoming fragments

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

s3-dcerpc: Read a pdu in 2 steps, header first the payload

We can't really cope with receiving more data than expected anyway
as there is code that checks if the size of the buffer is consistent with
pkt->frag_length and throws an error if not.

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

s3-dceprc: Use IDL defined constants for length calculations

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

Reformat and cleanup

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

s3-dcerpc: Move common helpers into a common file

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

s3-dcerpc: Padding is never done on a DCERPC_PKT_BIND_ACK packet.

Empirically verified against Windows.
It never pads the auth credentials on this packet type.

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

dcerpc: remove ncacn_packet_header and related functions

ncacn_packet_header was an artifact and it is not necessary anymore

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

s3-dcerpc: Refactor code and stop using ncacn_packet_header

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

s3-dcerpc: consolidate respones packet creation code

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

s3-dcerpc: Convert frag from prs_struct to a simple DATA_BLOB

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

s3-dcerpc: Convert rdata from prs_struct to a simple DATA_BLOB

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

s3-dcerpc: Remove unused functions and headers

parse_rpc.c is dead, long live parse_rpc.c !

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

s3-dcerpc: Stop using hand marshalling in create_next_pdu_noauth()

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

s3-dcerpc: Stop using hand marshalling in create_next_pdu_schannel()

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

s3-dcerpc: Stop using hand marshalling in create_next_pdu_ntlmssp()

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

s3-dcerpc: Stop using hand marshalling in rpc_finish_auth3_bind_send()

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

s3-dcerpc: Do not parse the packet twice.

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

s3-dcerpc: Remove unused function

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

s3-dcerpc: Do not reparse the header

Instead pass back and use the packet we have already parsed.

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

s3-dcerpc: Pull the whole packet at once instead of fetching just the header

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

s3-dcerpc: Delay parsing rpc header in client code

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

Fix bug - 7551 smbclient does not return proper exit code.

Catch error in cli_push.

Jeremy.

s4:testprogs Operate the blackbox kinit and net tests using the :local config

This :local tells selftest.pl to use the local smb.conf for the test
environment, not the generic client smb.conf

This then makes the rest work properly - otherwise, it may attempt to
connect to the wrong KDC for example.

The only problem is that we can't test the 'net join' with this set,
so this is removed from the test.  The member server test environment
checks this anyway.

Andrew Bartlett

s4:testprogs Show that we no longer delete the old keytab entries

By using a CCACHE obtained while the old password was still valid, we
can tell if the server still accepts incoming Kerberos connections
with the old password.

Andrew Bartlett

s4:provision Handle machine account password changes while keeping keytab

The challenge here is to update the existing record if it already
exists, rather than deleting the old record.  This ensures that the
secrets.keytab handling code keeps the previous password in the
keytab.

Andrew Bartlett

s4:pyldb whitespace fix

s4:pyldb Fix memory handling for ldb_message_element

The problem here is that we need to use the array, not the individual
message element as the memory context.

Andrew Bartlett