summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2011-01-22 16:22:42 +0100
committerKarolin Seeger <kseeger@samba.org>2011-03-05 14:34:52 +0100
commit350ad25db6ed3af91339e336e6a6dafe580fa54f (patch)
tree55ca2db4ad3b4fda8fa9ce9f86da4531a920761d
parent33a17fe21ba9bed23c48b6f9e6c173997cb5917b (diff)
downloadsamba-350ad25db6ed3af91339e336e6a6dafe580fa54f.tar.gz
samba-350ad25db6ed3af91339e336e6a6dafe580fa54f.tar.xz
samba-350ad25db6ed3af91339e336e6a6dafe580fa54f.zip
s3: Fix connecting to port-139 only servers
When the TCP RST came before the 5 msecs timeout kicked in, we viewed this as final, as state->req_139 was not set yet. Fix bug introduced by a fix for bug #7881 (winbind flaky against w2k8). (cherry picked from commit f2a19b87725f9318e983dff6358a3eee721bff08)
-rw-r--r--source3/libsmb/smbsock_connect.c8
1 files changed, 5 insertions, 3 deletions
diff --git a/source3/libsmb/smbsock_connect.c b/source3/libsmb/smbsock_connect.c
index 8ab12c5e2a2..b6c1c3c058a 100644
--- a/source3/libsmb/smbsock_connect.c
+++ b/source3/libsmb/smbsock_connect.c
@@ -221,12 +221,14 @@ struct tevent_req *smbsock_connect_send(TALLOC_CTX *mem_ctx,
/*
* After 5 msecs, fire the 139 request
*/
- subreq = tevent_wakeup_send(state, ev, timeval_current_ofs(0, 5000));
- if (tevent_req_nomem(subreq, req)) {
+ state->req_139 = tevent_wakeup_send(
+ state, ev, timeval_current_ofs(0, 5000));
+ if (tevent_req_nomem(state->req_139, req)) {
TALLOC_FREE(state->req_445);
return tevent_req_post(req, ev);
}
- tevent_req_set_callback(subreq, smbsock_connect_do_139, req);
+ tevent_req_set_callback(state->req_139, smbsock_connect_do_139,
+ req);
return req;
}