From 466302131e57266dd2820e78277fc2dd4dc10b78 Mon Sep 17 00:00:00 2001 From: Linus Nordberg Date: Wed, 3 Feb 2016 14:57:09 +0100 Subject: [PATCH] Don't treat "domain" or "search" as a nameserver. Continue the while fgets() loop as soon as we're done with "domain" or "search". Simplify the logic of the function by removing the if else constructs. --- src/context.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/context.c b/src/context.c index c47d4992..89491179 100644 --- a/src/context.c +++ b/src/context.c @@ -864,8 +864,9 @@ set_os_defaults(struct getdns_context *context) *token = 0; (void) strlcpy(domain, parse, sizeof(domain)); - - } else if (strncmp(parse, "search", 6) == 0) { + continue; + } + if (strncmp(parse, "search", 6) == 0) { parse += 6; do { parse += strspn(parse, " \t"); @@ -879,8 +880,9 @@ set_os_defaults(struct getdns_context *context) *token = prev_ch; parse = token; } while (*parse); - - } else if (strncmp(parse, "nameserver", 10) != 0) + continue; + } + if (strncmp(parse, "nameserver", 10) != 0) continue; parse += 10;