| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Signed-off-by: Luk Claes <luk@debian.org>
Signed-off-by: Stefan Metzmacher <metze@samba.org>
|
|
|
|
| |
metze
|
|
|
|
|
|
|
| |
metze
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Fri Nov 18 15:13:52 CET 2011 on sn-devel-104
|
|
|
|
| |
metze
|
|
|
|
| |
metze
|
|
|
|
|
|
|
| |
metze
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Wed Oct 26 17:53:15 CEST 2011 on sn-devel-104
|
|
|
|
| |
metze
|
|
|
|
| |
metze
|
|
|
|
| |
metze
|
|
|
|
| |
metze
|
|
|
|
| |
metze
|
|
|
|
| |
metze
|
|
|
|
|
|
|
|
| |
primary requests
This is needed to implement SMB signing correct.
metze
|
|
|
|
|
|
|
|
| |
We now keep the primary request open for the whole logical request.
The primary request is the one that gets all incoming replies.
While secondary requests are handled as separate one-way requests.
metze
|
|
|
|
| |
metze
|
|
|
|
|
|
| |
a comment
metze
|
|
|
|
|
|
|
|
| |
We need to align params and data to 4 byte offsets.
This also correctly recalculates the useable space after each step.
metze
|
|
|
|
|
|
|
| |
This is just to make the code more readable
and easier to notice how many words we're using in vwv.
metze
|
|
|
|
|
|
| |
SMBnttrans uses 32-bit values there.
metze
|
|
|
|
| |
metze
|
|
|
|
|
| |
Using the standard macro makes it easier to move code into common, as
TALLOC_ARRAY isn't standard talloc.
|
| |
|
|
|
|
| |
Guenther
|
|
|
|
|
|
|
| |
Guenther
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Fri Apr 29 14:00:30 CEST 2011 on sn-devel-104
|
|
|
|
|
|
|
|
|
| |
convert_string*()
we shouldn't accept bad multi-byte strings, it just hides problems
Autobuild-User: Andrew Tridgell <tridge@samba.org>
Autobuild-Date: Thu Mar 24 01:47:26 CET 2011 on sn-devel-104
|
| |
|
| |
|
|
|
|
| |
Guenther
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
Signed-off-by: Bo Yang <boyang@samba.org>
|
|
|
|
|
|
| |
should quieten some warnings with picky compilers on the buildfarm.
Jeremy.
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes calls to push_*_allocate() and pull_*_allocate(), as well
as convert_string_allocate, as they are not in the common API
To allow transition to a common charcnv in future, provide Samba4-like
strupper functions in source3/lib/charcnv.c
(the actual implementation remains distinct, but the API is now shared)
Andrew Bartlett
|
| |
|
| |
|
|
|
|
| |
Signed-off-by: Stefan Metzmacher <metze@samba.org>
|
|
|
|
|
|
|
|
|
|
| |
We store the seqnum/mid mapping in the cli_request structure
for async requests and in the cli_state structure for sync calls.
We skip the signing check for oplock requests while waiting
for async requests coming in.
metze
|
|
|
|
|
|
|
|
|
| |
Otherwise we return NO_MEMORY without a reason for fragmented trans
requests, as talloc_append_blob() returns buf if we append a 0 length
blob. When we pass buf = NULL we'll get back NULL and then assume
NO_MEMORY...
metze
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before we send the secondary requests we need to remove the
old mid=>seqnum mapping and reset cli->mid and make the new
mid=>seqnum mapping "persistent".
The bug we had in cli_send_trans was this:
The first cli_send_smb() incremented cli->mid
and the secondary requests used the incremented mid,
but as cli->outbuf still had the correct mid,
we send the correct mid to the server. The real problem
was that the cli_send_smb() function stored the seqnum
under the wrong mid.
cli_send_nttrans() was totally broken and now follows the
same logic as cli_send_trans().
The good thing is that in practice the problem is unlikely to happen,
because max_xmit is large enough to avoid secondary requests.
metze
|
| |
|
|
|
|
|
| |
The pushed strlen replaces the STR_TERMINATE flag which I personally always
find very confusing.
|
|
|
|
| |
This parameter makes smb_spice_chain add padding before the bytes field
|
|
|
|
| |
Introduce async_req_is_error() and async_req_simple_recv()
|
|
|
|
| |
Rename it to trans_oob, it will be used in the server routines.
|
|
|
|
| |
(This used to be commit 84ed752d9f17b2279bd60f534ce7c02b267a40b2)
|
|
|
|
|
| |
Logic stolen from Samba4, naturally the specific implementation differs a bit.
(This used to be commit 4b8bc5b03d35d563104791c0d8317d9886e4f032)
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a different fix than Jeremy put into 3-0-test with 040db1ce85 and other
branches with different hashes. Jeremy, I think your fix led to bug 5436, so I
reverted your fix. This fixes the original problem I found with the transs
requests for large rpc queries in a different way. Please check!
Thanks,
Volker
(This used to be commit c572d537e088a3fffb057181cad9a3692e40b815)
|
|
|
|
|
| |
This reverts commit 816aea6c1a426eb2450061b847729e22bdac33a0.
(This used to be commit e402e6508ca0806deef4c4044cfa6461b682850a)
|