diff options
author | Jeremy Allison <jra@samba.org> | 2007-05-16 17:17:25 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:22:11 -0500 |
commit | 478ccc150ba3b8c87ac67769c46be20a73d14096 (patch) | |
tree | 9c778802fe6db5591cd226467c05dee3a33c6ebd /source3/libsmb/cliconnect.c | |
parent | 0570ce21ffd6c2a52bf701cd72fbb66ed7b3eb3e (diff) | |
download | samba-478ccc150ba3b8c87ac67769c46be20a73d14096.tar.gz samba-478ccc150ba3b8c87ac67769c46be20a73d14096.tar.xz samba-478ccc150ba3b8c87ac67769c46be20a73d14096.zip |
r22950: Fix the issue Volker reported here :
"Attempt to fix some build farm failures: On port 139 the first
successful packet gives len==0 from the server, so the = in
if (len <= 0) {
in line 136 of clientgen.c throws a failure."
The irritating thing is that I already had it correct in
SAMBA_3_0_26 and forgot to merge the change across.
len == 0 is a valid return - I messed that up when
converting client_receive_smb() to return a length
rather than a BOOL.
Doh !
Jeremy.
(This used to be commit a398bdf08d9efac51af28aed29f2c0f151cd5aad)
Diffstat (limited to 'source3/libsmb/cliconnect.c')
-rw-r--r-- | source3/libsmb/cliconnect.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/libsmb/cliconnect.c b/source3/libsmb/cliconnect.c index f13aa21fbdd..86834ad081b 100644 --- a/source3/libsmb/cliconnect.c +++ b/source3/libsmb/cliconnect.c @@ -1347,7 +1347,7 @@ BOOL cli_session_request(struct cli_state *cli, cli_send_smb(cli); DEBUG(5,("Sent session request\n")); - if (!cli_receive_sessionreply(cli)) + if (!cli_receive_smb(cli)) return False; if (CVAL(cli->inbuf,0) == 0x84) { |