From 2e39530c16a949a76d0a273a43d44682d9dbe109 Mon Sep 17 00:00:00 2001 From: Andreas Schwab Date: Wed, 18 Nov 2015 15:45:59 +0100 Subject: [PATCH] Fix resource leak in resolver (bug 19257) The number of currently defined nameservers is stored in ->nscount, whereas ->_u._ext.nscount is set by __libc_res_nsend only after local initializations. (cherry picked from commit 5e7fdabd7df1fc6c56d104e61390bf5a6b526c38) (cherry picked from commit 317da342ba4417c30d985f5593d78bb1364a62c3) --- resolv/res_init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resolv/res_init.c b/resolv/res_init.c index e0b6a80dc76..6c951f526ef 100644 --- a/resolv/res_init.c +++ b/resolv/res_init.c @@ -594,7 +594,7 @@ __res_iclose(res_state statp, bool free_addr) { statp->_vcsock = -1; statp->_flags &= ~(RES_F_VC | RES_F_CONN); } - for (ns = 0; ns < statp->_u._ext.nscount; ns++) + for (ns = 0; ns < statp->nscount; ns++) if (statp->_u._ext.nsaddrs[ns]) { if (statp->_u._ext.nssocks[ns] != -1) { close_not_cancel_no_status(statp->_u._ext.nssocks[ns]); -- 2.47.2