From: Peter Eisentraut <peter@eisentraut.org>
Date: Fri, 27 Sep 2019 07:30:49 +0000 (+0200)
Subject: dnslookup: Consistently use log_error during impl_init()
X-Git-Tag: pgbouncer_1_12_0~9
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=beff924b5197377363dfccdd445be9e21038de28;p=pgbouncer

dnslookup: Consistently use log_error during impl_init()

This was done inconsistently between the different backends.  Some
used log_warning() for conditions that would ultimately be fatal.
---

diff --git a/src/dnslookup.c b/src/dnslookup.c
index 1dbca64..6fefc71 100644
--- a/src/dnslookup.c
+++ b/src/dnslookup.c
@@ -371,7 +371,7 @@ static bool impl_init(struct DNSContext *ctx)
 {
 	ctx->edns = evdns_base_new(NULL, 1);
 	if (!ctx->edns) {
-		log_warning("evdns_base_new failed");
+		log_error("evdns_base_new failed");
 		return false;
 	}
 	return true;
@@ -511,7 +511,7 @@ static bool impl_init(struct DNSContext *ctx)
 	/* i/o callback setup */
 	fd = dns_open(dctx);
 	if (fd <= 0) {
-		log_warning("dns_open failed: fd=%d", fd);
+		log_error("dns_open failed: fd=%d", fd);
 		return false;
 	}
 	event_set(&udns->ev_io, fd, EV_READ | EV_PERSIST, udns_io_cb, ctx);