From 2be2220f15b2a819a80b07c8881822ef87053c69 Mon Sep 17 00:00:00 2001 From: Pavel Březina Date: Thu, 10 Jul 2014 13:37:37 +0200 Subject: resolv: fix server sort by weight When the server list consist only from servers with zero weight the output list contained only one server. Resolves: https://fedorahosted.org/sssd/ticket/2357 Reviewed-by: Pavel Reichl --- src/resolv/async_resolv.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/resolv/async_resolv.c b/src/resolv/async_resolv.c index 38fa84d9..f9361244 100644 --- a/src/resolv/async_resolv.c +++ b/src/resolv/async_resolv.c @@ -2093,13 +2093,9 @@ static int reply_weight_rearrange(int len, r = *(start); prev = NULL; while (r != NULL) { - if (r->weight == 0) { + if (r->weight == 0 && r != *start) { /* remove from the old list */ - if (prev) { - prev->next = r->next; - } else { - *start = r->next; - } + prev->next = r->next; /* add to the head of the new list */ tmp = r; -- cgit