Avoid aborting when uv_timer_start() is used on a closing socket

In such a case it will return UV_EINVAL (-EINVAL), leading to
aborting, as the code expects the function to succeed.

diff --git a/lib/isc/netmgr/netmgr.c b/lib/isc/netmgr/netmgr.c
index b6ae1ca3192f2016b17e16424567058d0875d9af..ad9dc3d57fdb193ab22987183a479ca228c7795b 100644 (file)
--- a/lib/isc/netmgr/netmgr.c
+++ b/lib/isc/netmgr/netmgr.c
@@ -2033,6 +2033,10 @@ void
 isc__nmsocket_timer_restart(isc_nmsocket_t *sock) {
        REQUIRE(VALID_NMSOCK(sock));
 
+       if (uv_is_closing((uv_handle_t *)&sock->read_timer)) {
+               return;
+       }
+
        if (atomic_load(&sock->connecting)) {
                int r;