From 38949026b3600d035281d8eaf833dd327585bdd0 Mon Sep 17 00:00:00 2001 From: Aurelien Jarno Date: Tue, 6 Jan 2015 08:50:23 -0800 Subject: [PATCH] resolv: fix rotate option The rotate option doesn't work correctly, and only send the query to the same server (the second in the list). The rotation code in itself is not broken, but the nsaddrs structure is reinitialized each time at the beginning of __libc_res_nsend unless RES_STAYOPEN is enabled. This is due to a call to __res_iclose from the end of __libc_res_nsend when answers from the name server have been received. This function closes all the sockets, but doesn't free the addresses (it can do that, but in that case the second argument is false). This patch change the code of __res_iclose to clear statp->_u._ext.nsinit only when the addresses are actually freed. * resolv/res_init.c (__res_iclose): Only clear nsinit if the addresses have been freed. --- ChangeLog | 5 +++++ resolv/res_init.c | 3 ++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 1204575..6ba3b8e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,10 @@ 2015-01-06 Aurelien Jarno + * resolv/res_init.c (__res_iclose): Only clear nsinit if the + addresses have been freed. + +2015-01-06 Aurelien Jarno + * resolv/res_init.c (__res_vinit): Improve comments about nserv and nservall. diff --git a/resolv/res_init.c b/resolv/res_init.c index 4c170be..553ba12 100644 --- a/resolv/res_init.c +++ b/resolv/res_init.c @@ -621,7 +621,8 @@ __res_iclose(res_state statp, bool free_addr) { statp->_u._ext.nsaddrs[ns] = NULL; } } - statp->_u._ext.nsinit = 0; + if (free_addr) + statp->_u._ext.nsinit = 0; } libc_hidden_def (__res_iclose) -- 2.7.4