From 9759c20e90654be3dc057899357cff940c216b52 Mon Sep 17 00:00:00 2001 From: Bert Belder Date: Fri, 1 Jun 2012 18:30:17 +0200 Subject: [PATCH] test: verify that closing an active poll handle doesn't hang --- test/test-list.h | 2 ++ test/test-poll-close.c | 77 ++++++++++++++++++++++++++++++++++++++++++ test/test-poll.c | 2 +- uv.gyp | 1 + 4 files changed, 81 insertions(+), 1 deletion(-) create mode 100644 test/test-poll-close.c diff --git a/test/test-list.h b/test/test-list.h index 9a5ba695..58f13d66 100644 --- a/test/test-list.h +++ b/test/test-list.h @@ -168,6 +168,7 @@ TEST_DECLARE (counters_init) TEST_DECLARE (dlerror) TEST_DECLARE (poll_duplex) TEST_DECLARE (poll_unidirectional) +TEST_DECLARE (poll_close) #ifdef _WIN32 TEST_DECLARE (spawn_detect_pipe_name_collisions_on_windows) TEST_DECLARE (argument_escaping) @@ -330,6 +331,7 @@ TASK_LIST_START TEST_ENTRY (poll_duplex) TEST_ENTRY (poll_unidirectional) + TEST_ENTRY (poll_close) TEST_ENTRY (spawn_exit_code) TEST_ENTRY (spawn_stdout) diff --git a/test/test-poll-close.c b/test/test-poll-close.c new file mode 100644 index 00000000..9e5f9abb --- /dev/null +++ b/test/test-poll-close.c @@ -0,0 +1,77 @@ +/* Copyright Joyent, Inc. and other Node contributors. All rights reserved. + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to + * deal in the Software without restriction, including without limitation the + * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS + * IN THE SOFTWARE. + */ + +#include + +#ifndef _WIN32 +# include +# include +# include +#endif + +#include "uv.h" +#include "task.h" + +#define NUM_SOCKETS 64 + + +static int close_cb_called = 0; + + +static void poll_cb_fail(uv_poll_t* handle, int status, int events) { + ASSERT(0 && "poll_fail_cb should never be called"); +} + + +static void close_cb(uv_handle_t* handle) { + close_cb_called++; +} + + +TEST_IMPL(poll_close) { + SOCKET sockets[NUM_SOCKETS]; + uv_poll_t poll_handles[NUM_SOCKETS]; + int i; + +#ifdef _WIN32 + { + struct WSAData wsa_data; + int r = WSAStartup(MAKEWORD(2, 2), &wsa_data); + ASSERT(r == 0); + } +#endif + + for (i = 0; i < NUM_SOCKETS; i++) { + sockets[i] = socket(AF_INET, SOCK_STREAM, 0); + uv_poll_init_socket(uv_default_loop(), &poll_handles[i], sockets[i]); + uv_poll_start(&poll_handles[i], UV_READABLE | UV_WRITABLE, NULL); + } + + for (i = 0; i < NUM_SOCKETS; i++) { + uv_close((uv_handle_t*) &poll_handles[i], close_cb); + } + + uv_run(uv_default_loop()); + + ASSERT(close_cb_called == NUM_SOCKETS); + + return 0; +} diff --git a/test/test-poll.c b/test/test-poll.c index 0033f01f..6aa2a35b 100644 --- a/test/test-poll.c +++ b/test/test-poll.c @@ -537,7 +537,7 @@ static void start_poll_test() { #ifdef _WIN32 { struct WSAData wsa_data; - r = WSAStartup(MAKEWORD(2, 2), &wsa_data); + int r = WSAStartup(MAKEWORD(2, 2), &wsa_data); ASSERT(r == 0); } #endif diff --git a/uv.gyp b/uv.gyp index 47f87670..0a9e39dd 100644 --- a/uv.gyp +++ b/uv.gyp @@ -327,6 +327,7 @@ 'test/test-pipe-connect-error.c', 'test/test-platform-output.c', 'test/test-poll.c', + 'test/test-poll-close.c', 'test/test-process-title.c', 'test/test-ref.c', 'test/test-run-once.c',