diff options
author | Volker Lendecke <vl@samba.org> | 2011-01-22 16:22:42 +0100 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2011-03-05 14:34:52 +0100 |
commit | 350ad25db6ed3af91339e336e6a6dafe580fa54f (patch) | |
tree | 55ca2db4ad3b4fda8fa9ce9f86da4531a920761d | |
parent | 33a17fe21ba9bed23c48b6f9e6c173997cb5917b (diff) | |
download | samba-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.c | 8 |
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; } |