]> git.ipfire.org Git - thirdparty/bind9.git/commit
projects / thirdparty / bind9.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 5eda8a2) | patch
remove deadcode in `query_addbestns()`
authorColin Vidal <colin@isc.org>
Thu, 2 Apr 2026 08:48:41 +0000 (10:48 +0200)
committerColin Vidal <colin@isc.org>
Thu, 2 Apr 2026 09:51:12 +0000 (11:51 +0200)
commit50a2fce68f891293743e052dba0f3b39be53e51e
tree1dbbd8b5e3b7e12cef4533acb61e0966b190ac0btree | snapshot
parent5eda8a2543070a143cca39a53a1ea2fd9344a4ddcommit | diff
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.
lib/ns/query.c diff | blob | blame | history
Mirror of https://gitlab.isc.org/isc-projects/bind9.git