Disable inactive uvreqs caching when compiled with sanitizers

When isc__nm_uvreq_t gets deactivated, it could be just put onto array
stack to be reused later to save some initialization time.
Unfortunately, this might hide some use-after-free errors.

Disable the inactive uvreqs caching when compiled with Address or
Thread Sanitizer.

diff --git a/lib/isc/netmgr/netmgr.c b/lib/isc/netmgr/netmgr.c
index 0c906e69b98dc146147c3e93a98c1251e804c10c..3c2e47126fea603901959eca1cd467a8c03e43cb 100644 (file)
--- a/lib/isc/netmgr/netmgr.c
+++ b/lib/isc/netmgr/netmgr.c
@@ -2512,10 +2512,14 @@ isc___nm_uvreq_put(isc__nm_uvreq_t **req0, isc_nmsocket_t *sock FLARG) {
        handle = req->handle;
        req->handle = NULL;
 
+#if !__SANITIZE_ADDRESS__ && !__SANITIZE_THREAD__
        if (!isc__nmsocket_active(sock) ||
            !isc_astack_trypush(sock->inactivereqs, req)) {
                isc_mem_put(sock->mgr->mctx, req, sizeof(*req));
        }
+#else  /* !__SANITIZE_ADDRESS__ && !__SANITIZE_THREAD__ */
+       isc_mem_put(sock->mgr->mctx, req, sizeof(*req));
+#endif /* !__SANITIZE_ADDRESS__ && !__SANITIZE_THREAD__ */
 
        if (handle != NULL) {
                isc__nmhandle_detach(&handle FLARG_PASS);