summaryrefslogtreecommitdiff
path: root/source4/torture/libnet/userinfo.c
AgeCommit message (Collapse)AuthorFilesLines
2007-10-10r7491: libnet functions used in tests have different names now.Rafal Szczesniak1-10/+10
rafal (This used to be commit d54bf09701b1085451479a32569849998c53c811)
2007-10-10r6847: Fix some unused variable warnings in the libnet torture test.Tim Potter1-1/+0
(This used to be commit 448795607f271bb00d5f9e1c5a6cad8df75c8b6a)
2007-10-10r6717: - torture test of async useradd function and monitor messages.Rafal Szczesniak1-1/+1
- make message handling functions static. rafal (This used to be commit 96446e5e1e53eb519cbfeb64d0dd2f4052d75b0f)
2007-10-10r6637: Test for asynchronous function and monitor messages.Rafal Szczesniak1-2/+81
rafal (This used to be commit 4ac3c145fccc203323e3bf8772bccd290a82cf63)
2007-10-10r6432: Restorin previous construction of conditions after a little discussionRafal Szczesniak1-1/+3
with Richard and Andrew. rafal (This used to be commit 400f6d5f87c7a0b49bfd289521a3cf4af2586235)
2007-10-10r6426: DCE/RPC bind string parsing is not needed anymore, as we useRafal Szczesniak1-10/+1
lp_workgroup parameter as domain name to operate on. rafal (This used to be commit ea251d4f79c96145b4c14074d258aeb07c742f41)
2007-10-10r6165: fixed up the userinfo composite code. Fixes include:Andrew Tridgell1-5/+4
- talloc should always be done in the right context. For example, when creating the userinfo_state structure, place it inside the composite structure, not directly on the pipe. If this isn't done then correct cleanup can't happen on errors (as cleanup destroys the top level composite context only) - define private structures like userinfo_state in the userinfo.c code, not in the public header - only keep the parameters we need in the state structure. For example, the domain_handle is only needed in the first call, so we don't need to keep it around in the state structure, but the level is needed in later calls, so we need to keep it - always initialise [out,ref] parameters in RPC calls. The [ref] part means that the call assumes the pointer it has been given is valid. If you don't initialise it then you will get a segv on recv. This is why the code was dying. - don't use internal strucrure elements like the pipe pipe->conn->pending outside of the internal rpc implementation. That is an internal list, trying to use it from external code will cause crashes. - rpc calls assume that rpc call strucrures remain valid for the duration of the call. This means you need to keep the structures (such as "struct samr_Close") in the userinfo_state strucrure, otherwise it will go out of scope during the async processing - need to remember to change c->state to SMBCLI_REQUEST_DONE when the request has finished in the close handler, otherwise it will loop forever trying to close Mimir, please look at the diff carefully for more detailed info on the fixes (This used to be commit 01ea1e7762e214e87e74d6f28d6efeb6cdea9736)
2007-10-10r5941: Commit this patch much earlier than I would normally prefer, but ↵Andrew Bartlett1-2/+2
metze needs a working tree... The main volume of this patch was what I started working on today: - Cleans up memory handling around DCE/RPC pipes, to have a parent talloc context. - Uses sepereate inner loops for some of the DCE/RPC tests The other and more important part of this patch fixes issues surrounding the new credentials framwork: This makes the struct cli_credentials always a talloc() structure, rather than on the stack. Parts of the cli_credentials code already assumed this. There were other issues, particularly in the DCERPC over SMB handling, as well as little things that had to be tidied up before test_w2k3.sh would start to pass. Andrew Bartlett (This used to be commit 0453f9d05d2e336fba1f85dbf2718d01fa2bf778)
2007-10-10r5902: A rather large change...Andrew Bartlett1-2/+2
I wanted to add a simple 'workstation' argument to the DCERPC authenticated binding calls, but this patch kind of grew from there. With SCHANNEL, the 'workstation' name (the netbios name of the client) matters, as this is what ties the session between the NETLOGON ops and the SCHANNEL bind. This changes a lot of files, and these will again be changed when jelmer does the credentials work. I also correct some schannel IDL to distinguish between workstation names and account names. The distinction matters for domain trust accounts. Issues in handling this (issues with lifetime of talloc pointers) caused me to change the 'creds_CredentialsState' and 'struct dcerpc_binding' pointers to always be talloc()ed pointers. In the schannel DB, we now store both the domain and computername, and query on both. This should ensure we fault correctly when the domain is specified incorrectly in the SCHANNEL bind. In the RPC-SCHANNEL test, I finally fixed a bug that vl pointed out, where the comment claimed we re-used a connection, but in fact we made a new connection. This was achived by breaking apart some of the dcerpc_secondary_connection() logic. The addition of workstation handling was also propogated to NTLMSSP and GENSEC, for completeness. The RPC-SAMSYNC test has been cleaned up a little, using a loop over usernames/passwords rather than manually expanded tests. This will be expanded further (the code in #if 0 in this patch) to use a newly created user account for testing. In making this test pass test_rpc.sh, I found a bug in the RPC-ECHO server, caused by the removal of [ref] and the assoicated pointer from the IDL. This has been re-added, until the underlying pidl issues are solved. (This used to be commit 824289dcc20908ddec957a4a892a103eec2da9b9)
2007-10-10r5681: fix the build and compiler waningsStefan Metzmacher1-4/+3
metze (This used to be commit bb2622a0c94127a31855adc41d90289219a1ddd7)
2007-10-10r5679: Complete basic test of rpc_composite_userinfo function (whichRafal Szczesniak1-3/+151
btw proves the function is completely broken but that's what tests are for) rafal (This used to be commit 7278c7d8370d52fed1e6a7dfc848a3efa06ccd61)
2007-10-10r5661: Be a little stricter on syntax regarding arrays. A pointer to anJelmer Vernooij1-0/+2
array can now only be : type *name[]; rather then : type *name; which was supported in the past. Warnings will be given when the first syntax is used. Reasons for this change in behaviour include improved readability and the fact that the second format makes dealing with multiple levels of pointers harder. (This used to be commit a416de5825c540fd3741731c4be05e9a659a6fdb)
2007-10-10r5651: A bit more code in userinfo test and hook up test function toRafal Szczesniak1-3/+19
main torture binary. rafal (This used to be commit 94955e5325ceddd35673da74afb19d1676b5b23c)
2007-10-10r5610: Starting libnet test of userinfo call. Unfinished yet, thoughRafal Szczesniak1-0/+100
doesn't break anything at the moment. rafal (This used to be commit eb13c1bcfc330b046b76563c4eeb9d1ae438d37b)