| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Guenther
(cherry picked from commit b5bec1a6d73f5939b306e157937d027a7286163c)
(cherry picked from commit c7d5e0a19057b6ad7301a390fc766bb438967eb2)
|
|
|
|
|
| |
Jeremy.
(cherry picked from commit be46b98ee38e17e4561ab58fe9b6ecc6b7919392)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Jeremy, I think the ability to say
smbclient //foo/bar -I <ip-address> -p 139
making the called name to "foo" got lost with 3d2d0203. Was this removed
deliberately? If so, please revert this patch. If not, please merge
appropriately.
Thanks,
Volker
(cherry picked from commit f13763ca4be49c08dfc533e39dd338b508ba4709)
|
|
|
|
|
|
| |
Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 574a6a8c350a4bab3f42f3f9cfb382db721d69b5)
(cherry picked from commit 7e6d6eeff3e082d7223264c17cb27c2ab89df9aa)
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixed.
It turns out there were a number of places where cli_resolve_path() was called
and the error path upon that function failing did not set errno. There were a
couple of places the failure handling code did set errno to ENOENT, so I made
them all consistent, although I think better errno choices for this condition
exist, e.g. EHOSTUNREACH.
Derrell
(cherry picked from commit ebc4c0b50db16cdb11222c3d536a211bb86bcdfd)
|
|
|
|
|
|
|
| |
match_mailslot_name
(cherry picked from commit db5677d071fc58f38cab4ab800111455a8637edb)
(cherry picked from commit 0f01296bbb08d75ce693a65ea6cb0df80c1b4927)
|
|
|
|
|
|
|
|
|
|
|
| |
ntlmssp_seal_packet creates its own signature data blob, which we then
have to free.
Jeremy, please check and merge appropriately (Yes, I'm asking you to do the
janitor work, I want you to *look* at this :-))
Volker
(cherry picked from commit 17616dddfaa26688387f671c870873056896f6b7)
|
|
|
|
|
|
| |
Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 87b428e424e2e3cca975ecd0efed327e72950a1d)
(cherry picked from commit a2e138419138a2f675f6370426a6caeda0a314b4)
|
|
|
|
|
| |
(cherry picked from commit 298bff5019be5a40dc0023b5af1e980fba91abd9)
(cherry picked from commit c96d0ffd944f536a1228812c761ba4fb4313ee05)
|
|
|
|
|
|
| |
Guenther
(cherry picked from commit 1524abd8bf12d82e1fb0063585fc9a465fc7bf9c)
(cherry picked from commit 42a692da477d7986767ad90a15087aa8a2ebcb40)
|
|
|
|
| |
(cherry picked from commit b20cace3a7c312f44f39390954bc087b888d9b27)
|
|
|
|
|
|
|
|
|
|
|
| |
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
(cherry picked from commit 88dd6af605dc5754b7e146a068272d37651da710)
(cherry picked from commit 5bbf96dd63227a19fe1f95ff8d8f2b3c75a5a497)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
(cherry picked from commit 880fbc4e8cd67de73c4bcda94489eb1e1422a04b)
(cherry picked from commit 70466990b4b7c68ae95dbbcf741cd3f41f2dd0b3)
|
|
|
|
|
|
| |
selected (we need to path in pathname /that/look/like/this).
Jeremy.
(cherry picked from commit 913d1f2af87fa449881a6fd2774a9dfdcb198fb4)
|
|
|
|
|
|
| |
Guenther
(cherry picked from commit 531af136f9dd5c6050f78948837294aed02de440)
(cherry picked from commit 91216ffa95b4ed53e54b11665e96b911cb4e4ab5)
|
|
|
|
|
|
|
|
|
|
| |
callers pass in a struct user_auth_info * instead. This commit causes
smbc_set_credentials() to print out a message telling callers to use
smbc_set_credentials_with_fallback() instead, as smbc_set_credentials()
has a broken API (no SMBCCTX * pointer). No more global variables used
in the connection manager API for client dfs calls.
Jeremy.
(cherry picked from commit 8b44877eb82c1c9270cb8099fefc621922f82719)
|
|
|
|
|
| |
(cherry picked from commit 20cee26a3dbd231672eec9133c6e84641def298d)
(cherry picked from commit 1e08799f3c5bb7059f6d355b99f8617461c31d1a)
|
|
|
|
| |
(cherry picked from commit 68456130ff23cb8139c9710fc674ca5b52230197)
|
|
|
|
|
|
| |
password talloc'ed strings within the cli_struct.
Jeremy.
(cherry picked from commit 6be4bf17de47d1b468389de1225a72daa94e0119)
|
|
|
|
|
|
|
|
| |
the problem that stops libsmbclient being thread safe. Subsidiary
DFS connections are now hung off a list inside the cli_state struct.
Much more to do in order to get libsmbclient to thread safety, but
this is a good start.
Jeremy.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
buffers"
This reverts commit 9579a6f193f570e4ce2af80f4aac7c2f25ae5b22.
It's confusing to have a boolean to alter the behavior of cli_push
and as the new feature isn't used yet I revert it.
We can readd a extra function later.
metze
|
|
|
|
| |
metze
|
|
|
|
| |
metze
|
|
|
|
|
|
|
|
|
|
|
|
| |
There were the following problems:
1.) if window_size was a multiple of the chunk_size,
we silently dropped the last truncated chunk.
2.) if window_size was 0 pushed only the first chunk
to the server and silently dropped the rest.
3.) we had only transferred state->num_reqs writes,
even if there would be more data to send.
metze
|
| |
|
| |
|
|
|
|
| |
metze
|
|
|
|
|
|
|
|
| |
*cli_initialise_ex()
This prepares the next changes.
metze
|
|
|
|
|
|
|
|
|
| |
[MS-SMB] 3.3.5.1 Receiving Any Message says that the seqnum
is incremented by only for ntcancel requests for any other
request it's by incremented by 2, even if it doesn't expect
a response.
metze
|
|
|
|
|
|
| |
This is only cosmetic, but it makes it easier to understand.
metze
|
| |
|
|
|
|
| |
Jeremy.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Use const in function signatures whenever appropriate, to help prevent
errant scribbling on users' buffers. smbc_set_credentials() always acted as
if its formal parameters were const char *, and changing the formal
declaration to specify that should not cause any change to the ABI. It is
still allowable to pass a writable buffer to a function which specifies that
it will not write to the buffer.
I'm making this change only in master.
Derrell
|
| |
|
| |
|
|
|
|
|
| |
const approptiately.
Jeremy.
|
| |
|
|
|
|
| |
Signed-off-by: Derrell Lipman <derrell.lipman@unwireduniverse.com>
|
|
|
|
|
|
|
|
|
|
|
| |
This extends the file_id struct to add an additional generic uint64_t
field: extid. For backwards compatibility with dev/inodes stored in
xattr_tdbs and acl_tdbs, the ext id is ignored for these databases.
This patch should cause no functional change on systems that don't use
SMB_VFS_FILE_ID_CREATE to set the extid.
Existing code that uses the smb_share_mode library will need to be
updated to be compatibile with the new extid.
|
| |
|
|
|
|
| |
Jeremy.
|
|
|
|
|
|
| |
<gmachin@sandia.gov>.
Jeremy.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
- Revert Tim's changes for the moment. I need to see what the issue is and
arrange to use "struct statvfs" if at all possible.
Derrell
|