summaryrefslogtreecommitdiffstats
path: root/source3/rpc_client
Commit message (Collapse)AuthorAgeFilesLines
* s3:rpc_client: remove more unused codeStefan Metzmacher2010-03-293-59/+0
| | | | metze
* s3:rpc_client: remove unused code, we handle transport failures in the ↵Stefan Metzmacher2010-03-291-14/+0
| | | | | | transport layer now metze
* s3:rpc_client: return at least 10 sec as old timeout in rpccli_set_timeout() ↵Stefan Metzmacher2010-03-291-3/+12
| | | | | | instead of 0 metze
* s3:rpc_client: add set_timeout hook to rpc_cli_transportStefan Metzmacher2010-03-294-20/+63
| | | | metze
* s3:rpc_client: add rpccli_is_connected()Stefan Metzmacher2010-03-294-18/+171
| | | | metze
* s3:rpc_client: don't mix layers and keep a reference to cli_state in the callerStefan Metzmacher2010-03-291-8/+17
| | | | | | | | We should not rely on the backend to have a reference to the cli_state. This will make it possible for the backend to set its cli_state reference to NULL, when the transport is dead. metze
* s3:rpc_transport_np: add comment about bad usage in a destructorStefan Metzmacher2010-03-291-1/+2
| | | | metze
* s3:rpc_transport_np: use cli_state_is_connected() helperStefan Metzmacher2010-03-291-1/+1
| | | | metze
* s3:ntlmssp: pass names and use_ntlmv2 to ntlmssp_client_start() and store themStefan Metzmacher2010-03-241-1/+5
| | | | | | | | Inspired by the NTLMSSP merge work by Andrew Bartlett. metze Signed-off-by: Günther Deschner <gd@samba.org>
* s3-rpcclient: fix rpcclient after spoolss_GetPrinterData{Ex} IDL change.Günther Deschner2010-03-051-3/+13
| | | | Guenther
* s3:rpc_transport_np: handle trans rdata like the output of a normal readStefan Metzmacher2010-02-241-0/+17
| | | | | | Inspired by bug #7159. metze
* s3:cli_netlogon: keep the the correct negotiate_flags on the cli->dc structureStefan Metzmacher2010-02-231-2/+6
| | | | | | | | | This should fix the rpccli_netlogon_set_trust_password() against DC's without netr_ServerPasswordSet2 support. This fixes bug #7160. metze
* Second part of fix for bug #7159 - client rpc_transport doesn't cope with ↵Jeremy Allison2010-02-191-0/+6
| | | | | | | | | | bad server data returns. If server returns zero on a NP read. Report pipe broken. Prevents client from looping if it thinks there should be more data. Jeremy.
* First part of fix for bug #7159 - client rpc_transport doesn't cope with bad ↵Jeremy Allison2010-02-192-0/+16
| | | | | | | | | | | server data returns. Ensure that subreq is *always* talloc_free'd in the _done function, as it has an event timeout attached. If the read requests look longer than the cli->timeout, then the timeout fn is called with already freed data. Jeremy.
* Fix bug #7146 - Samba miss-parses authenticated RPC packets.Jeremy Allison2010-02-171-15/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Parts of the Samba RPC client and server code misinterpret authenticated packets. DCE authenticated packets actually look like this : +--------------------------+ |header | | ... frag_len (packet len)| | ... auth_len | +--------------------------+ | | | Data payload | ... .... | | +--------------------------+ | | | auth_pad_len bytes | +--------------------------+ | | | Auth footer | | auth_pad_len value | +--------------------------+ | | | Auth payload | | (auth_len bytes long) | +--------------------------+ That's right. The pad bytes come *before* the footer specifying how many pad bytes there are. In order to read this you must seek to the end of the packet and subtract the auth_len (in the packet header) and the auth footer length (a known value). The client and server code gets this right (mostly) in 3.0.x -> 3.4.x so long as the pad alignment is on an 8 byte boundary (there are some special cases in the code for this). Tridge discovered there are some (DRS replication) cases where on 64-bit machines where the pad alignment is on a 16-byte boundary. This breaks the existing S3 hand-optimized rpc code. This patch removes all the special cases in client and server code, and allows the pad alignment for generated packets to be specified by changing a constant in include/local.h (this doesn't affect received packets, the new code always handles them correctly whatever pad alignment is used). This patch also works correctly with rpcclient using sign+seal from the 3.4.x and 3.3.x builds (testing with 3.0.x and 3.2.x to follow) so even as a server it should still work with older libsmbclient and winbindd code. Jeremy
* s3: Remove some unused variablesVolker Lendecke2010-01-101-7/+0
|
* s3: Fix infinite loop in NCACN_IP_TCP asa there is no timeout. Assume ↵Bo Yang2010-01-065-4/+112
| | | | | | lsa_pipe_tcp is ok but network is down, then send request is ok, but select() on writeable fds loops forever since there is no response. Signed-off-by: Bo Yang <boyang@samba.org>
* s3:ntlmssp: only include ntlmssp.h where actually neededAndrew Bartlett2009-12-221-0/+1
| | | | Andrew Bartlett
* s3:ntlmssp: remove the typedef NTLMSSP_STATEAndrew Bartlett2009-12-221-1/+1
| | | | Andrew Bartlett
* s3-spoolss: fix enumprinter key client and server.Günther Deschner2009-12-101-2/+5
| | | | Guenther
* samba-spoolss: use spoolss_StringArray2 in spoolss_EnumPrinterKey.Günther Deschner2009-12-021-20/+4
| | | | | | | This should finally resolve the endian issues we were seeing on sparc and is much cleaner for spoolss clients and servers. Guenther
* s3-kerberos: only use krb5 headers where required.Günther Deschner2009-11-271-0/+1
| | | | | | | This seems to be the only way to deal with mixed heimdal/MIT setups during merged build. Guenther
* s3-rpc: running minimal_includes.pl on rpc_client and rpc_server.Günther Deschner2009-11-261-2/+0
| | | | Guenther
* s3-rpc: Avoid including every pipe's client and server stubs everywhere in ↵Günther Deschner2009-11-264-1/+4
| | | | | | samba. Guenther
* s3-spoolss: fixes for _spoolss_EnumPrinterKey client and server.Günther Deschner2009-11-241-2/+2
| | | | | | Thanks Metze for review! Guenther
* s3-spoolss: fix spoolss_EnumPrinterKey client and server code.Günther Deschner2009-11-231-3/+21
| | | | Guenther
* s3: Add min_setup, min_param and min_data to cli_trans_recvVolker Lendecke2009-11-141-2/+2
| | | | | | | | | | Every caller that expects to receive something needs to check if enough was sent. Make this check mandatory for everyone. Yes, this makes the parameter list for cli_trans a bit silly, but that's just the way it is: A silly protocol request :-) While there, convert some _done functions to tevent_req_simple_finish_ntstatus.
* fix bogus "out of memory" winbind msgVolker Lendecke2009-11-131-1/+2
| | | | Signed-off-by: Michael Adam <obnox@samba.org>
* Remove erroneous 'presult = NULL' changes. Now presult only gets setJeremy Allison2009-11-121-9/+2
| | | | | if NTSTATUS == OK. Jeremy.
* Revert "Ensure every return path initializes presult as NULL."Jeremy Allison2009-11-121-40/+0
| | | | | | Vl is correct, this is the wrong way to fix this. This reverts commit 83c2c177a5e86d04da37384f1f04230c8274e1e6.
* Ensure all callers to the rpc_client/cli_pipe functions correctlyJeremy Allison2009-11-121-2/+2
| | | | | initialize return variables. Jeremy.
* Ensure every return path initializes presult as NULL.Jeremy Allison2009-11-121-0/+40
| | | | | | Ensures no crashes in calling code that forgets to init return as null. Jeremy.
* s3-rpc_client: make sure cli_rpc_pipe_open_schannel() does not always return ↵Günther Deschner2009-11-101-1/+3
| | | | | | NT_STATUS_OK. Guenther
* Revert "s3: Do not directly reference the ndr_table_* in rpcclient"Volker Lendecke2009-11-081-14/+0
| | | | This reverts commit 70c698fd547c4bc19cf77693608bbb34acac40b5.
* Revert "s3: Do not reference ndr_table_<pipe> in the cli_ routines directly"Volker Lendecke2009-11-081-18/+3
| | | | This reverts commit daa964013bc5d036f4da571ce22c0052ef40943a.
* s3: Do not reference ndr_table_<pipe> in the cli_ routines directlyVolker Lendecke2009-11-081-3/+18
|
* s3: Do not directly reference the ndr_table_* in rpcclientVolker Lendecke2009-11-081-0/+14
|
* s3: Do the printing for DEBUGLEVEL>=10 centrallyVolker Lendecke2009-11-071-1/+12
| | | | | | 12 insertions(+), 10651 deletions(-) I think that says it all :-)
* s3: Register the ndr_interfaces dynamicallyVolker Lendecke2009-11-071-37/+110
|
* s3: Get rid of a NULL terminatorVolker Lendecke2009-11-071-2/+1
|
* s3: Get rid of explicit pipe namesVolker Lendecke2009-11-071-49/+54
|
* s3: get_pipe_name_from_iface -> get_pipe_name_from_syntaxVolker Lendecke2009-11-072-9/+11
|
* s3-kerberos: modify cli_krb5_get_ticket to take a new impersonate_princ_s arg.Günther Deschner2009-11-061-1/+1
| | | | Guenther
* s3: Remove debug_ctx()Volker Lendecke2009-11-031-17/+17
| | | | | | | | smbd just crashed on me: In a debug message I called a routine preparing a string that itself used debug_ctx. The outer routine also used it after the inner routine had returned. It was still referencing the talloc context that the outer debug_ctx() had given us, which the inner DEBUG had already freed.
* s3-netlogon: fix updating trust accout passwords with downlevel domains.Günther Deschner2009-10-161-2/+2
| | | | | | | | When choosing the netlogon password set function, make sure to look at the *negotiated* flags in the cli->dc state, not the ones we start the negotiation with. Guenther
* s3: use enum netr_SchannelType all over the place.Günther Deschner2009-10-132-2/+2
| | | | Guenther
* s3-netlogon: pass down account name to remote password set functions.Günther Deschner2009-10-131-1/+2
| | | | Guenther
* s3/s4 - Adapt the IDL changes on various locationsMatthias Dieter Wallnöfer2009-10-081-1/+1
|
* s3-netlogon: setup NETLOGON credential chain in ↵Günther Deschner2009-10-061-13/+14
| | | | | | rpccli_netlogon_set_trust_password() only when needed. Guenther
* Revert "s3: Attempt to fix machine password change"Volker Lendecke2009-10-051-34/+17
| | | | | | This reverts commit 20a8ea91e10af167067cc794a251265aaf489e75. Ooops, this should not have been committed.