From: Colin Vidal <colin@isc.org>
Date: Thu, 2 Apr 2026 09:51:25 +0000 (+0200)
Subject: fix: dev: remove deadcode in `query_addbestns()`
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=43c58dafcc1fac168698f818936d75692a8550bd;p=thirdparty%2Fbind9.git

fix: dev: remove deadcode in `query_addbestns()`

The local variable `zfname` was released in the cleanup part of the
function if not NULL, but it turns out it is now always NULL at that
point.

The flow can get to that part only in two cases: either `zfname` is not
NULL, and then it's ownership is moved to a different variable (thus, it
is now NULL), or `zfname` is already NULL.

Removing the bit of deadcode releasing it.

Merge branch 'colin/fix-getbestns-deadcode' into 'main'

See merge request isc-projects/bind9!11790
---

43c58dafcc1fac168698f818936d75692a8550bd