Commit
This patch updates the documentation for the SMB/CIFS protocol.
- Loading branch information
There are no files selected for viewing
5 comments
on commit 96c3b1a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks Gisle,
I can't seem to reproduce this with a native Visual Studio build (in either Debug or Release) as I always seem to have a valid password pointer. I tried not including the -u argument as well as -u :
However, I have pushed commit f9b7132 to prevent this from happening.
Would you be so kind to confirm whether it fixes the issue for you please?
Cheers again
Steve
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(I rewrote as new comment since Github's markup fails on my email reply).
I can't seem to reproduce this with a native Visual Studio build (in either Debug or Release)
as I always seem to have a valid password pointer.
I've always build libcurl + curl with '-DCURLDEBUG' and '-DDEBUGBUILD'
(but in MSVC release mode). Hence the assert in memdebug.c. So now when
I try w/o any user/password, I always get the content! This command:
curl.exe -v smb://localhost/G_DISK/Delme.bat
give me the contents of 'g:\Delme.bat' (only 194 bytes). But I have
no entry for localhost nor 127.0.0.1 in my %HOME/.netrc. So where does
the credential come from now? Or is it %HOME/_netrc?
However, I have pushed commit f9b7132 f9b7132
to prevent this from happening.Would you be so kind to confirm whether it fixes the issue for you please?
This issue may have been fixed. But there are other issues with
transferring files > ~32 kB. E.g.
curl.exe -v smb://localhost/G_DISK/large-file.txt
* Marked for [closure]: SMB: failed to communicate
* Closing connection 0
* The cache now contains 0 members
curl: (26) Failed to open/read local data from file/application
Studying smb.c + smb_recv_message(), it seems to me it handles
SMB-messages like datagrams (UDP). Here:
nbt_size = ntohs(*(unsigned short *)(buf + sizeof(unsigned short))) +
sizeof(unsigned int);
if(smbc->got < nbt_size)
return CURLE_OK;
it seems 'buf' is treated like the start of an SMB-header.
I fails to see how that can be guaranteed since TCP is used
in my case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@captain-caveman2k : is there any reason why you dont bring this discussion to the libcurl
Not really. I only commented on the commit since I deleted any SMB-related messages to libcurl.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hiya Gisle,
Was there a reason for mentioning my handle in your last comment - whilst I think this discussion should now take place on the mailing list (as we need to involve Bill) - the question was asked by Gunter and not myself ;-)
FYI. Connecting using SMB and no credentials throws an abort() inside memdebug.c / curl_domalloc().
Call-stack: