s3: Remove smbd_server_conn from reply_tcon_and_X

s3: Remove smbd_server_conn from reply_tcon

s3: Remove smbd_server_conn from make_connection_snum

s3: Remove smbd_server_conn from change_to_user

s3: Remove smbd_server_conn from reply_ntcancel

s3: Remove smbd_server_conn from send_nt_replies

s3: Remove smbd_server_conn from reply_smb2002

s3: Explicitly pass sconn to negprot_spnego()

s3: Remove some global references to smbd_server_conn

s3: Remove sconn arg from smbd_notify_cancel_by_smbreq

s3: Initialize smb_request->sconn in smbd_smb2_fake_smb_request

s3: Use "sconn" argument in smbd_[un]lock_socket_internal

Metze, please check!

s3: Pass sconn to init_smb_request()

s3: Add "smbd_server_connection" to smb_request

s3: Fix some nonempty lines

s4:dsdb Allow calling dsdb_convert_object_ex() directly

This will allow the libnet_vampire code to manually convert individual
schema objects.

Andrew Bartlett

s4:dsdb Add debug

s4:dsdb Simplfy match of objectclass in dsdb_schema_set_el_from_ldb_msg

There is no need to do a full ldb_match_msg() for a simple case
insensitive string.

Andrew Bartlett

s4:provision Allow both additional and override prefixmaps in Schema

The idea here is to allow some callers to specify a new prefixMap that
will override the values loaded from the prefixMap.txt.

Andrew Bartlett

s4:dsdb Allow a binary prefix map to be specified in the LDIF

This allows it to be specified in either binary or as a string.

Andrew Bartlett

s4:dsdb Provide a function to convert from DRS prefix maps to the LDB prefixmap

This allows us to push a prefixmap directly into the schema we
generate in the provision code.

Andrew Bartlett

s4:dsdb Add more debugs to help track down failures to parse the prefixmap

s4:dsdb Put back the reference and set_attributes in dsdb_reference_schema

I'm not sure why I removed these in fe3e1af901c970f738bee92baac5d7d4f5736e17

Andrew Bartlett

check if LD_AS_NEEDED breaks linking with libreadline fixes #7209

Signed-off-by: Simo Sorce <idra@samba.org>

s4/test-libnet: Re-align code a bit - more readable this way IMO

s4/test-libnet: Refactor test_group_create() a little to:

- Close unused policy handle for created group
- re-align code - it is more readable this way IMHO

s4/test-libnet: Refactor test_user_create() little bit to:

- fix a flaw that user RID is not returned in case
  user is recreated
- close unused policy handle to created user
- do not take into account the result from
  test_samr_close_handle - we don't case

s4/test-libnet: add few comments for utility functions

s4/test-libnet: Move LSA close handle function to a common location

s4/test-libnet: remove unused code

Pre-connecting SAMR and LSA pipes is implemented in
test_libnet_context_init() function

s4/test-libnet: Fix compile-time warning

Adjusted ACL tests to use encripted connection.

This way we get rid of the conditional if.Samba checks, because users are
successfully created and enabled.

s3-auth: Fix valgrind warning (unitialized var) in samu_to_SamInfo3().

Guenther

s3:auth do not fail if there are 0 group sids

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

configure.in prevent generation of file "shlib.so"

s3:auth fix samu->info3 conversion

Some pdb_get_ functions where missing because of previous mis-patching

s3: Fix starving the echo responder

When both the echo responder and the 445 socket want to send stuff to the
worker smbd, the select loop is not fair. It always chooses the smaller file
descriptor to work on. This can mean that on a busy system the echo responder
never gets around to feed its stuff to the parent.

This fix chooses the async echo responder socket when both the 445 and the echo
responder socket are readable.

Yes, it is a very hackish fix which is required *now* I think. The proper fix
would be to either assign priorities to fd's in tevent, or the from my point of
view better fix would be to make tevent kindof round-robin.

Round-robin would mean that whenever a fd has been dealt with, it is taken off
the list of interested sockets, and only if no other socket is active, all of
the ones waiting are put back. This is a bit like EPOLL_ONESHOT, which I would
like to use for this in the epoll case. Although, I need to do some research if
maybe epoll already guarantees round-robin, I did not find anything in the docs
yet.

Volker

s4:rootdse LDB module - use LDB result constants

s4:passwords.py - set the "dSHeuristics"

As per Nadia's request and abartlet's suggestion the test now also sets the
"dSHeuristics" attribute properly to be able to perform the tests also against
Windows without further configuration.

The code has the neat feature that it undoes the change and resets the
behaviour as it was before.

s4:passwords.py - use "self" before "enable_account" to make really sure which LDB connection should be used

s4:ldap.py - check how the directory server behaves when the 'distinguishedName' attribute wasn't set correctly

(On add operations the server does completely ignore it and uses the "header"
DN)

s4:ldap.py - check if the "name" of an entry was really ignored

ldb:ldb_sqlite3 backend - remove a warning which has already been resolved

ldb:ldb_sqlite3 backend - remove checks which are now done elsewhere

(In SAMBA LDB modules)

Add missing stub functions.

s4-python: Fix formatting.

s3:rpc improve handles memory hierarchy and use better name.

While there also cleanup the code a bit.

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

s4-smbtorture: rework internals of RPC-SPOOLSS-ACCESS testsuite.

Guenther

s4/test: use test_libnet_context_init() function to create libnet_context

This way we ensure that LIBNET call will be issued against
the DC we are testing against

s4/test: fix SEGFAULT in test_group_create when rid is NULL

Caller of this function may not need group RID and
in such case it passes NULL for rid parameter

s4/test-libnet: Helper func to initialize libnet_context

and create rpc connections to SAMR and LSA pipes.
This function should be used in cases where we need
'deterministic' connection to the DC we are testing against.

The problem is that most of the test follow the pattern:
1. Connect to server supplied on command line
2. Create user/group through connection in 1.
3. Use 'fresh' libnet_context to query/alter the object in
   step 2.

In singe-DC environment this work well. But in multiple-DC
environment libnet may connect to another DC (step 3), not
the one we use in step 1 and 2. This leads to false error -
object created in step 2 is not found on DC we are querying
in step 3.

s4/test: Call test_user_cleanup with original RDN for the user

'name' var is changed during the test execution and should
contain samAccountName for the user, while test_user_cleanup()
expects account RDN to be passed

s4/test: Fix incorrect NTSTATUS comparison

In case group already exists, we should get NT_STATUS_GROUP_EXISTS.

s4/test: remove create/delete group implementation in API-CREATEGROUP test

We already have common implementation for create/delete
group in libnet/utils.c

s4/test: Remove duplicated test_opendomain() functionality

We already have a nice common test_domain_open()
function in libnet/util.c

s4/test: libnet_user tests should use common test_user_cleanup

implementation instead of not-so-informative test_cleanup()

s4/test: Refactor test_user_cleanup to be more robust

We always call this function with user's account RDN
while samr_LookupNames() searches for samAccountName.

s4/test: remove duplicated comment

s3:misc make use of server_[event/messaging]_context directly

Untangle these functions from smbd specific dependencies so they can be freely
used in multiple servers.

s3:winbindd use common server context functions

s3:lib make server contexts generic

Pair-programmed-with: Andreas Schneider <asn@samba.org>

Fix the build in the non WITH_AIO case (sorry).

s3: readd h_errno struct member but rename it

as pointed out by metze this is a structure of fixed size, which should not be
changed.

Since idtree assigns sequentially, it rarely reaches high numbers.
But such numbers can be forced with idr_get_new_above(), and that
reveals two bugs:

1) Crash in sub_remove() caused by pa array being too short.
2) Shift by more than 32 in _idr_find(), which is undefined, causing
   the "outside the current tree" optimization to misfire and return NULL.

Implement AIO in SMB2. Doesn't allow cancel calls yet (to be added).

Jeremy.

Ensure we correctly setup req->subreq before testing for async. Stops crashes when
XX_send functions set async to stop interim replies being sent.

Jeremy.

Don't use the autofree context for the globals. This causes child smbd's forked
by modules to crash due to destructors being called (found when using the vfs_aio_fork
module with smb2).

Jeremy.

s3:cleaunp reformatting for readability

s3:cleanup remove trailing spaces

s3: fix build on HP-UX

this struct member h_errno is not used in the HP-UX code paths, it was just
there because Solaris has it, too. As h_errno is a function call macro on HP-UX
when thread support is enabled we run into trouble here. Just commenting it out
should be okay as we don't use it anyway.

s4:ldap.py - add testcase which demonstrates the reset of the "primaryGroupID"

s4:samldb LDB module - fix up the case when the old and new "primaryGroupID" are the same

s4:samldb LDB module - don't create multiple "ac" module contexts on modify operations

Since we do now run sequentially through all checks we don't need multiple "ac"
contexts anymore.

s4:dcesrv_samr_Add/DeleteAliasMember - provide better NTSTATUS return codes when something didn't work

s4:dcesrv_samr_GetAliasMembership - fix type of counter variables

s4:dcesrv_samr_DeleteAliasMember - add more braces to fit better the coding styles

s4:dcesrv_samr_AddAliasMembership - Merge the two error blocks into one

s4:dcesrv_samr_Add/DelGroupMember - remove the account type check

MS-SAMR 3.1.5.8 speaks from accounts which are not necessarely only users.

s4:dcesrv_samr_AddGroupMember - also the error code "LDB_ERR_ATTRIBUTE_OR_VALUE_EXISTS" is allowed

This is returned when the group is the primary group of the specified entry.

s4:samba_dsdb LDB module - move the "objectclass_attrs" module back

I think it should be lower in order to control also the "instanceType" module.

s4:ldap.py - add a test to demonstrate the 'instanceType' behaviour

s4:instancetype LDB module - prevent all types of "instanceType" manipulation

Also on Windows Server you aren't able to change it.

s3: Fix EnumDomainAliases when no aliases are in LDAP

We used to return NT_STATUS_ACCESS_DENIED, now we just return 0 entries, just
like W2k8 does.

usrmgr.exe was pretty unhappy with the NT_STATUS_ACCESS_DENIED

s3:pdb_ldap: fix bug 7505 - init_sam_from_ldap stores group in sid2uid cache

s3: Add a dummy test to prove the maxfd calculation

To be run in a debugger, there's no way to inspect the internal tevent_select.c
maxfd calculation

tevent: Fix maxfd calculation in tevent_select

When doing

        fd1 = tevent_add_fd(ev, ev, 2, 0, NULL, NULL);
        fd2 = tevent_add_fd(ev, ev, 3, 0, NULL, NULL);
        TALLOC_FREE(fd2);
        fd2 = tevent_add_fd(ev, ev, 1, 0, NULL, NULL);

we end up with select_ev->maxfd==1. This is wrong.

An alternative fix might be to make select_ev->maxfd an unsigned int and make
EVENT_INVALID_MAXFD==UINT_MAX. But in theory we might end up with an fd of
UINT_MAX.

std_event_add_fd() contains exactly the same piece of code, so I'm directly
pushing it.

Volker

s3-smbd: Fix typo in comment.

Karolin

s3-passdb: Fix typo in comment.

Karolin

Rename "allow_smb2" -> "using_smb2" and make the usage clearer.

Split out the "finished write processing" code into a function so it can be called
by both sync and async code.

Jeremy.

Ensure we don't send SMB1 keepalives on an SMB2 connection.

Jeremy.

Split out the "finished read processing" code into a function so it can be called
by both sync and async code.

Jeremy.

s3-winbind: Fixed setting default sequence number.

s3: fix calculation of st_blocks in streams_xattr

Thanks to Joachim Schmitz for finding that miscalculation.

s3: fix calculation of st_blocks in init_stat_ex_from_stat

this is just relevant for the very rare systems that don't have the st_blocks
struct member. This is a fixup of the fix for bug 7474.

Thanks to Joachim Schmitz for spotting this!

Added guids for the validated writes.

s3: Fix bug 7253

acct_ctrl is 32 bit in LOGIN_CACHE, but "w" as a format specifier for
tdb_unpack only writes 16 bits. Okay on x86, not okay on Solaris.

Thanks to Vladimir.Marek@Sun.COM!

Volker

Fix a valgrind error found by SMB2-COMPOUND test.

If a file is closed we must also NULL out all chained_fsp
pointers when the fsp is freed to prevent invalid pointer
access.

Jeremy.

Second part of fix for 7501 - SMB2: CREATE request replies getting mangled.

Based on code from Ira Cooper <samba@ira.wakeful.net>, and also
advice on refactoring the patch into a function call. outbuf vectors
can be reallocated by smb2 processing code, so when returning interim
responses we must not make assumptions about vector size.

Jeremy

Fix for bug 7501 -  SMB2: CREATE request replies getting mangled.

Code for dup_smb2_req() was duplicating the wrong vector (i, instead
of i+2) when returning a non-minimal SMB2 response.

s3-selftest: disable domain creds smbclient_s3 tests for now.

Not sure they were ever working before.

With this patch, make selftest is down to 4 failures from 259 tests.

(two times LOCK9 from s3 smbtorture, one spoolss notify test and
samba3.posix_s3.unix.info2).

Guenther