diff options
| author | Javier Blazquez <jblazquez@riotgames.com> | 2018-06-22 20:11:26 -0700 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2018-06-23 22:31:52 +0200 |
| commit | 4c901638b4d7bab5e5a847f6970425d9c1f2ea2d (patch) | |
| tree | 3f8374daaa298195a6e15ea0ce373ece0feac3d1 /tests/libtest | |
| parent | 146178a9df83de1cfc345771b4c6eceb21ea700e (diff) | |
multi: fix crash due to dangling entry in connect-pending list
Fixes #2677
Closes #2679
Diffstat (limited to 'tests/libtest')
| -rw-r--r-- | tests/libtest/Makefile.inc | 6 | ||||
| -rw-r--r-- | tests/libtest/lib1557.c | 62 |
2 files changed, 67 insertions, 1 deletions
diff --git a/tests/libtest/Makefile.inc b/tests/libtest/Makefile.inc index ed536a564..ecb7d1dd4 100644 --- a/tests/libtest/Makefile.inc +++ b/tests/libtest/Makefile.inc @@ -28,7 +28,7 @@ noinst_PROGRAMS = chkhostname libauthretry libntlmconnect \ lib1525 lib1526 lib1527 lib1528 lib1529 lib1530 lib1531 lib1532 lib1533 \ lib1534 lib1535 lib1536 lib1537 lib1538 \ lib1540 \ - lib1550 lib1551 lib1552 lib1553 lib1554 lib1555 lib1556 \ + lib1550 lib1551 lib1552 lib1553 lib1554 lib1555 lib1556 lib1557 \ lib1900 \ lib2033 @@ -493,6 +493,10 @@ lib1556_SOURCES = lib1556.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS) lib1556_LDADD = $(TESTUTIL_LIBS) lib1556_CPPFLAGS = $(AM_CPPFLAGS) -DLIB1556 +lib1557_SOURCES = lib1557.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS) +lib1557_LDADD = $(TESTUTIL_LIBS) +lib1557_CPPFLAGS = $(AM_CPPFLAGS) -DLIB1557 + lib1900_SOURCES = lib1900.c $(SUPPORTFILES) $(TESTUTIL) $(WARNLESS) lib1900_LDADD = $(TESTUTIL_LIBS) lib1900_CPPFLAGS = $(AM_CPPFLAGS) diff --git a/tests/libtest/lib1557.c b/tests/libtest/lib1557.c new file mode 100644 index 000000000..bd0f20a85 --- /dev/null +++ b/tests/libtest/lib1557.c @@ -0,0 +1,62 @@ +/*************************************************************************** + * _ _ ____ _ + * Project ___| | | | _ \| | + * / __| | | | |_) | | + * | (__| |_| | _ <| |___ + * \___|\___/|_| \_\_____| + * + * Copyright (C) 1998 - 2018, 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 https://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" + +int test(char *URL) +{ + CURLM *curlm = NULL; + CURL *curl1 = NULL; + CURL *curl2 = NULL; + int running_handles = 0; + int res = 0; + + global_init(CURL_GLOBAL_ALL); + + multi_init(curlm); + multi_setopt(curlm, CURLMOPT_MAX_HOST_CONNECTIONS, 1); + + easy_init(curl1); + easy_setopt(curl1, CURLOPT_URL, URL); + multi_add_handle(curlm, curl1); + + easy_init(curl2); + easy_setopt(curl2, CURLOPT_URL, URL); + multi_add_handle(curlm, curl2); + + multi_perform(curlm, &running_handles); + + multi_remove_handle(curlm, curl2); + curl_easy_cleanup(curl2); + + /* If curl2 is still in the connect-pending list, this will crash */ + multi_remove_handle(curlm, curl1); + curl_easy_cleanup(curl1); + +test_cleanup: + curl_multi_cleanup(curlm); + curl_global_cleanup(); + return res; +} |