aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGokhan Sengun <gokhansengun@gmail.com>2011-12-06 23:41:24 +0100
committerDaniel Stenberg <daniel@haxx.se>2011-12-07 15:37:05 +0100
commit2b24dd870e81512542f238cb1b124cecb942a34f (patch)
tree151c695ddd70f74cfc0e62503aeef575a6adbdac
parent46724b87b769ff50e1e6b90d2aabc50b8ba7c1a9 (diff)
multi interface: fix block when CONNECT_ONLY option is used
-rw-r--r--lib/multi.c2
-rw-r--r--tests/data/Makefile.am2
-rw-r--r--tests/data/test59737
-rw-r--r--tests/libtest/Makefile.inc4
-rw-r--r--tests/libtest/lib597.c145
-rw-r--r--tests/libtest/test.h25
6 files changed, 212 insertions, 3 deletions
diff --git a/lib/multi.c b/lib/multi.c
index f3b892c91..e5073432e 100644
--- a/lib/multi.c
+++ b/lib/multi.c
@@ -1239,7 +1239,7 @@ static CURLMcode multi_runsingle(struct Curl_multi *multi,
easy->easy_conn->bits.close = FALSE;
multistate(easy, CURLM_STATE_DONE);
easy->result = CURLE_OK;
- result = CURLM_OK;
+ result = CURLM_CALL_MULTI_PERFORM;
}
else {
/* Perform the protocol's DO action */
diff --git a/tests/data/Makefile.am b/tests/data/Makefile.am
index 23e2a7ddd..93faca212 100644
--- a/tests/data/Makefile.am
+++ b/tests/data/Makefile.am
@@ -48,7 +48,7 @@ test551 test552 test553 test554 test555 test556 test557 test560 test561 \
test562 test563 test564 test565 test566 test567 test568 test569 test570 \
test571 test572 test573 test574 test575 test576 test578 test579 test580 \
test581 test582 test583 test584 test585 test586 test587 test588 test590 \
-test591 test592 test593 test594 test595 test596 \
+test591 test592 test593 test594 test595 test596 test597 \
test600 test601 test602 test603 test604 \
test605 test606 test607 test608 test609 test610 test611 test612 test613 \
test614 test615 test616 test617 test618 test619 test620 test621 test622 \
diff --git a/tests/data/test597 b/tests/data/test597
new file mode 100644
index 000000000..458bb6453
--- /dev/null
+++ b/tests/data/test597
@@ -0,0 +1,37 @@
+<testcase>
+<info>
+<keywords>
+FTP
+CONNECT_ONLY
+</keywords>
+</info>
+
+# Client-side
+<client>
+<server>
+ftp
+</server>
+<tool>
+lib597
+</tool>
+ <name>
+FTP connect only option
+ </name>
+
+<command>
+ftp://%HOSTIP:%FTPPORT
+</command>
+
+</client>
+
+#
+# Verify data after the test has been "shot"
+<verify>
+<protocol>
+USER anonymous
+PASS ftp@example.com
+PWD
+QUIT
+</protocol>
+</verify>
+</testcase>
diff --git a/tests/libtest/Makefile.inc b/tests/libtest/Makefile.inc
index 1693ee7c0..e37e63b9e 100644
--- a/tests/libtest/Makefile.inc
+++ b/tests/libtest/Makefile.inc
@@ -16,7 +16,7 @@ noinst_PROGRAMS = chkhostname \
lib543 lib544 lib545 lib547 lib548 lib549 lib552 lib553 lib554 lib555 \
lib556 lib539 lib557 lib560 lib562 lib564 lib565 lib566 lib567 lib568 \
lib569 lib570 lib571 lib572 lib573 lib582 lib583 lib585 lib586 lib587 \
- lib590 lib591
+ lib590 lib591 lib597
chkhostname_SOURCES = chkhostname.c $(top_srcdir)/lib/curl_gethostname.c
chkhostname_LDADD = @CURL_NETWORK_LIBS@
@@ -176,3 +176,5 @@ lib587_CPPFLAGS = $(AM_CPPFLAGS) -DLIB587
lib590_SOURCES = lib590.c $(SUPPORTFILES)
lib591_SOURCES = lib591.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS)
+
+lib597_SOURCES = lib597.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS)
diff --git a/tests/libtest/lib597.c b/tests/libtest/lib597.c
new file mode 100644
index 000000000..6a5ee4f8f
--- /dev/null
+++ b/tests/libtest/lib597.c
@@ -0,0 +1,145 @@
+/***************************************************************************
+ * _ _ ____ _
+ * Project ___| | | | _ \| |
+ * / __| | | | |_) | |
+ * | (__| |_| | _ <| |___
+ * \___|\___/|_| \_\_____|
+ *
+ * Copyright (C) 1998 - 2011, Daniel Stenberg, <daniel@haxx.se>, et al.
+ *
+ * This software is licensed as described in the file COPYING, which
+ * you should have received as part of this distribution. The terms
+ * are also available at http://curl.haxx.se/docs/copyright.html.
+ *
+ * You may opt to use, copy, modify, merge, publish, distribute and/or sell
+ * copies of the Software, and permit persons to whom the Software is
+ * furnished to do so, under the terms of the COPYING file.
+ *
+ * This software is distributed on an "AS IS" basis, WITHOUT WARRANTY OF ANY
+ * KIND, either express or implied.
+ *
+ ***************************************************************************/
+#include "test.h"
+
+#include "testutil.h"
+#include "warnless.h"
+#include "memdebug.h"
+
+#define TEST_HANG_TIMEOUT 5 * 1000
+
+/*
+ * Test case for below scenario:
+ * - Connect to an FTP server using CONNECT_ONLY option
+ * - transfer some files with re-using the connection (omitted in test case)
+ * - Disconnect from FTP server with sending QUIT command
+ *
+ * The test case originated for verifying CONNECT_ONLY option shall not
+ * block after protocol connect is done, but it returns the message
+ * with function curl_multi_info_read().
+ */
+
+enum {
+ CONNECT_ONLY_PHASE = 0,
+ QUIT_PHASE,
+ LAST_PHASE
+};
+
+int test(char *URL)
+{
+ CURL *easy = NULL;
+ CURLM *multi = NULL;
+ int res = 0;
+ int running;
+ int msgs_left;
+ int phase;
+ CURLMsg *msg;
+
+ start_test_timing();
+
+ res_global_init(CURL_GLOBAL_ALL);
+ if(res) {
+ return res;
+ }
+
+ easy_init(easy);
+
+ multi_init(multi);
+
+ for (phase = CONNECT_ONLY_PHASE; phase < LAST_PHASE; ++phase) {
+ /* go verbose */
+ easy_setopt(easy, CURLOPT_VERBOSE, 1L);
+
+ /* specify target */
+ easy_setopt(easy, CURLOPT_URL, URL);
+
+ /* enable 'CONNECT_ONLY' option when in connect phase */
+ if (phase == CONNECT_ONLY_PHASE)
+ easy_setopt(easy, CURLOPT_CONNECT_ONLY, 1L);
+
+ /* enable 'NOBODY' option to send 'QUIT' command in quit phase */
+ if (phase == QUIT_PHASE) {
+ easy_setopt(easy, CURLOPT_CONNECT_ONLY, 0L);
+ easy_setopt(easy, CURLOPT_NOBODY, 1L);
+ easy_setopt(easy, CURLOPT_FORBID_REUSE, 1L);
+ }
+
+ multi_add_handle(multi, easy);
+
+ for(;;) {
+ struct timeval interval;
+ fd_set fdread;
+ fd_set fdwrite;
+ fd_set fdexcep;
+ long timeout = -99;
+ int maxfd = -99;
+
+ multi_perform(multi, &running);
+
+ abort_on_test_timeout();
+
+ if(!running)
+ break; /* done */
+
+ FD_ZERO(&fdread);
+ FD_ZERO(&fdwrite);
+ FD_ZERO(&fdexcep);
+
+ multi_fdset(multi, &fdread, &fdwrite, &fdexcep, &maxfd);
+
+ /* At this point, maxfd is guaranteed to be greater or equal than -1. */
+
+ multi_timeout(multi, &timeout);
+
+ /* At this point, timeout is guaranteed to be greater or equal than -1. */
+
+ if(timeout != -1L) {
+ interval.tv_sec = timeout/1000;
+ interval.tv_usec = (timeout%1000)*1000;
+ }
+ else {
+ interval.tv_sec = TEST_HANG_TIMEOUT/1000+1;
+ interval.tv_usec = 0;
+ }
+
+ select_test(maxfd+1, &fdread, &fdwrite, &fdexcep, &interval);
+
+ abort_on_test_timeout();
+ }
+
+ msg = curl_multi_info_read(multi, &msgs_left);
+ if(msg)
+ res = msg->data.result;
+
+ multi_remove_handle(multi, easy);
+ }
+
+test_cleanup:
+
+ /* undocumented cleanup sequence - type UA */
+
+ curl_multi_cleanup(multi);
+ curl_easy_cleanup(easy);
+ curl_global_cleanup();
+
+ return res;
+}
diff --git a/tests/libtest/test.h b/tests/libtest/test.h
index 34b8c9f3e..861a571a5 100644
--- a/tests/libtest/test.h
+++ b/tests/libtest/test.h
@@ -242,6 +242,31 @@ extern int unitfail;
/* ---------------------------------------------------------------- */
+#define exe_multi_remove_handle(A,B,Y,Z) do { \
+ CURLMcode ec; \
+ if((ec = curl_multi_remove_handle((A),(B))) != CURLM_OK) { \
+ fprintf(stderr, "%s:%d curl_multi_remove_handle() failed, " \
+ "with code %d (%s)\n", \
+ (Y), (Z), (int)ec, curl_multi_strerror(ec)); \
+ res = (int)ec; \
+ } \
+} WHILE_FALSE
+
+#define res_multi_remove_handle(A,B) \
+ exe_multi_remove_handle((A),(B),(__FILE__),(__LINE__))
+
+#define chk_multi_remove_handle(A,B,Y,Z) do { \
+ exe_multi_remove_handle((A),(B),(Y),(Z)); \
+ if(res) \
+ goto test_cleanup; \
+} WHILE_FALSE
+
+
+#define multi_remove_handle(A,B) \
+ chk_multi_remove_handle((A),(B),(__FILE__),(__LINE__))
+
+/* ---------------------------------------------------------------- */
+
#define exe_multi_perform(A,B,Y,Z) do { \
CURLMcode ec; \
if((ec = curl_multi_perform((A),(B))) != CURLM_OK) { \