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

s4:testprogs Prove kerberos still works after a password change

Changing the machine account password should not prevent connections
with a current, valid CCACHE.  This is because when the password is
changed, the server-side keytab keeps one old password around.

Andrew Bartlett

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

s4 upgradeprovision: Adapt the list of attribute modified

* isMemberOfPartialAttributeSet is now allowed to be deleted (on schema
 objects)
* attributeDisplayNames is now allowed to be added and modified (used on
  display specifiers)
* spnMapping is now allowed to be altered on Directory Service objects
* minPwdAge is now modified if the previous value was 0

We issue a clear information about the userControl attribute for
administrator to invite the user to modify himself the value.

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

s4 ldb modules: relax some tests about attributes that should not be here

For attributes that we know that are harmless and that used to be stored
in the ldb we relax the tests on the existance in a given objectclass.

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

s4 upgradeprovision: Synchronize the calculated keyversionnumber with the one previously stored

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

s4 upgradeprovision: do not copy RID Set it's automaticaly created by the RID manager

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

s4 upgradeprovision: add function to backup the provision before updating

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

s4 upgradeprovision: fix whitespaces

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

s4: Add unit test for increment_calculated_keyversion_number

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

s4 upgradeprovision: introduce a new function to update the field use for calculating msds-keyversionnumber

This function change the version field of the unicodePwd in the
replPropertyMetaData so that the version is equal or
superior to the reference value passed.

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

s4 python: Add functions to samdb to manipulate version of replPropertyMetaData attribute

This change contains also helpers for attribute id to attribute oid
conversion and from attribute id to attribute name.
It brings also unit tests

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

s4 dsdb: Use the changereplmetadata control

This control allow to specify the replPropertyMetaData attribute to
be specified on modify request. It can be used for very specific needs
to tweak the content of the replication data.

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

s4 dsdb: create a new control: changereplmetadata

This control is designed to allow replmetadata to be specified

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

ldb: allow ldb_sequence_number to be called in python

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

s4: Unit test update_machine_account_password through kinit

This patch is for testing the chgdcpass script which is mostly a call to
update_machine_account_password.

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

s4: Add a simple script to change dc password

This script will mostly be used by unit test (blackbox type) to test the
change of the dc password

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

s4 provision: move update_machine_account_password to helpers

This is to allow reuse of this function and also unit tests

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

s3: Fix crashes in the printing code

create_conn_struct did not create the conn->sconn!=NULL assumption we now
depend on. Thanks to Andreas Schneider for testing!

s3-dcerpc: Break memory hierarchy for shared structure

Handles are shared among multiple pipes_struct. We cannot allocate
them on any specific pipes_struct or it will vanish for all others
as soon as that pipes_struct is freed, leaving back dangling
pointers.

Signed-off-by: Andreas Schneider <asn@samba.org>

s4: Added acl search tests for anonymous connection.

The tests make sure that we comply with dsHeuristics setting and
restrict anonymous access to rootDSE. They will be enabled when the
implementation is pushed. tests are verified against win2k8.

s3-dcerpc: fix crash bug in error path of process_complete_pdu().

Guenther

Fixed system_session_anon to actually make an anonymous session

It seems that because the flag is false, this always used the supplied credentials
rhather than establish anonymous connection.

s3:auth Change auth_ntlmssp_server_info API to return NTSTATUS

This fixes a bug where register_existing_vuid() could be called with a
NULL server_info if the alloction failed.

Andrew Bartlett

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

s3:smbd Give the kerberos session key a parent

Nothing will free this, so this prevents a memory leak.

Andrew Bartlett

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

s3:smbd Fix segfault if register_existing_vuid() fails

The register_existing_vuid() call will handle both the ntlmssp_end and
vuid invalidation internally, so we don't want to do it again.

Andrew Bartlett

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

s3-selftest: enable RPC-BIND against s3.

Guenther

s4-smbtorture: add very simple RPC-BIND testsuite.

Guenther

s4-smbtorture: rename rpc/bind.c to rpc/multi_bind.c.

Guenther

s3-libnet: add missing header file, sorry.

Guenther

s3-libnet: better separate headers.

Guenther

s3-winbind: Don't cache queries to builtin and own sam domain.

s3-winbind: Set status before we leave in some msrpc functions.

Revert "Remove the global char *LastDir."

Volker pointed out I'd missed the "last directory" cache
part of this code. Return us to caching the directory we're
in (reduces sys call load).

Mea maxima culpa.

Jeremy.

This reverts commit 2f30aea3324f32f9b8555e961256fc1280da2871.

s3-waf: fix the build.

Guenther

s3-ldb: fix build warning.

Guenther

s4: Reorganized dsHeuristics reset so the code can be reused

Moved the setting of dsHeuristics to a method as soon we will have to set other
values as well in different tests

s3-dcerpc: Remove unused functions and headers

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

s3-dcerpc: Use dcerpc_push_ncacn_packet() in api_pipe_alter_context()

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

s3-dcerpc: Use dcerpc_push_dcerpc_auth() for spnego auth continuations.

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

s3-dceprc: Use dcerpc_push_ncacn_packet() in api_pipe_bind_req()

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

s3-dcerpc: Use a DATA_BLOB in bind processing and avoid one mem copy

Also move dcerpc_push_dcerpc_auth() invocation in api_pipe_bind_req()
to simplify the workflow.

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

s3-dcerpc: Rearrange api_pipe_bind_req() to do checks before allocating memory

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

s3-dcerpc: fix dcerpc_push_ncacn_packet, pass arg by address

There is no need to copy the whole structure twice by passing it in by value.

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

s3-dcerpc: pass the packet to setup_bind_nack()

Allows to not rely on p->call_id but use the value directly from the request
packet header.

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

s3-dceprc: Add debug messages

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

s3-dcerpc: Remove unused members

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

s3-dcerpc: Remove unused functions

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

s3-dcerpc: Use dcerpc_pull_ncacn_packet() to unmarshall dceprc requests

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

dcerpc: move dcerpc_pull_auth_trailer into a the common dcerpc_util.c file

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

s3-dceprc: Store opnum in its own variable

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

s3-dceprc use a DATA_BLOB to hold the curren pdu in pipes_struct

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

s3-dcerpc: use dceprc_push_ncacn_packet_header in create_next_pdu_noauth()

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

s3-dcerpc: Use dcerpc_push_ncacn_packet_header() in create_next_pdu_schannel()

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

s3-dcerpc: use dcerpc_push_ncacn_packet_header() in create_next_pdu_ntlmssp()

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

sr-dcerpc: add dcerpc_push_ncacn_packet_header()

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

s3-dcerpc delay rpc header unmarshalling

Use dcerpc_get_frag_length() to get the whole pdu first

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

s3-dcerpc: Remove unused headers

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

s3-dcerpc: Use dcerpc_set_frag_length()

Automatically calculate the fragment length where possible.

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

s3-dcerpc: link against ../librpc/rpc/dcerpc_util.c

Guenther

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

s4-dcerpc: move some util functions up to main librpc directory.

Guenther

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

s3-dcerpc: Remove unused prototype

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

s3-dcerpc: Fix ntlmssp sign/seal.

Header calculation was misplaced.

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

s3-dceprc: Fix auth_length in auth3 response

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

s3-dcerpc: Fix miscalculation of buffer start address

This was breaking schannel

tsocket: Fix some unreachable code

s3: [ug]id_to_unix_... can not fail

Remove some silly failure checks

README.Coding: fix typos

Thanks to Michael Wood <esiotrot@gmail.com> for the review.

metze

README.Coding: add examples for good and bad comments

metze

s4:provision: remove --policy-guid and --policy-guid-dc cmdline options

metze

s4:provision: also use fixed GUID names of the default group policies for domain and domain controllers in tests

metze

s4 provision: use correct GUID for default policies

The value of GUID for policy is not random for default policies, it is
described here ("How Core Group Policy Works"):
http://technet.microsoft.com/en-us/library/cc784268%28WS.10%29.aspx
at paragraph System\Policies Container.

Signed-off-by: Stefan Metzmacher <metze@samba.org>

s4 unittests: remove the provision directory before (re)generating

s4 net: Add spn module to list/add/remove spn on objects

Signed-off-by: Stefan Metzmacher <metze@samba.org>

testprogs/blackbox/subunit.sh: initialize failed to 0

This is a short-term workarround for broken scripts,
which use "exit $failed", without initializing failed.

We need a discussion on the mailing list how to handle this
in a nicer way.

This should fix some random failures in the blackbox tests.

metze

s4:winbind: let WBSRV_SAMBA3_SET_STRING() initialize the whole buffer

We should not send uninitialized bytes to the winbind pipe,
this makes also makes valgrind very unhappy.

metze

s4:wb_cmd_list_groups: also handle NT_STATUS_NO_MORE_ENTRIES

metze

s4:wb_cmd_list_users: also handle NT_STATUS_NO_MORE_ENTRIES

metze

s4:libnet_GroupList: allocate children strings on the correct talloc parent

Otherwise the _recv() function won't move the children strings to the
callers memory context and let the callers crash.

metze

s4:libnet_UserList: allocate children strings on the correct talloc parent

Otherwise the _recv() function won't move the children strings to the
callers memory context and let the callers crash.

metze

s4:buildtools: add 'make show_waf_options'

metze

s4:buildtools: make it possible to use advanced test options via the make waf wrapper

You can to do thinks like this now:
make test TEST_OPTIONS="--valgrind-server" TESTS=wbinfo

metze

s3: Cleanup of the initial SMB2 counters patch.

This reorganizes smbd_smb2_request_dispatch to have a central exit point,
and use the normal profiling macros.

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

s4:drepl_notify: hide some bugs from the make test output

It's useless to get messages like this every few seconds:

dreplsrv_notify: Failed to send DsReplicaSync to
edbf4745-2966-49a7-8653-99200f1c9430._msdcs.samba2003.example.com for
CN=Configuration,DC=samba2003,DC=example,DC=com -
NT_STATUS_OBJECT_NAME_NOT_FOUND : WERR_BADFILE

We have a non bug regarding non-linked DN attributes
and changes of the target DN.

metze

s4:drsuapi: don't return all linked attributes at the same time

Windows gives them in chunks, but I don't know the correct
rule to calculate the chunk size.

For now I'll use 1500 as the max size.

Windows isn't happy when it gets ~ 100000 linked attributes in one
response. It corrupts its directory index and later moves some objects
to the LostAndFound folder.

metze

s3-build: Add a gdbtestenv environment for Samba3.

This will open 3 xterms; smbd with gdb, winbind with gdb and a window to
run client commands.

Example: make gdbtestenv SMBD_OPTIONS="-d2"

s4:selftest: run RPC-SAMR-LARGE-DC against the vampire_dc to test the rid alloc code

metze

selftest/Samba4: set dreplsrv:periodic_startup_interval = 0

metze

s4:dsdb/repl: expose drsuapi_DsExtendedError to the caller (e.g. the ridalloc client)

metze

s4:drepl_out_helpers: don't return NT_STATUS_OK, if an extended operation doesn't return success

metze

s4:drepl_ridalloc: only ask the rid master for a new rid pool if we need to.

if we are at least half-exhausted then ask for a new pool.

This fixes a bug where we're sending unintialized alloc_pool
variable as exop->fsmo_info to the rid master and get back
DRSUAPI_EXOP_ERR_PARAM_ERROR.

metze

s4:dsdb:ridalloc: use ridalloc_ridset_values infrastructure in ridalloc_allocate_rid_pool_fsmo()

metze