]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
fix: dev: Standardize CHECK and RETERR macros
authorEvan Hunt <each@isc.org>
Thu, 4 Dec 2025 03:15:12 +0000 (03:15 +0000)
committerEvan Hunt <each@isc.org>
Thu, 4 Dec 2025 03:15:12 +0000 (03:15 +0000)
Previously, there were over 40 separate definitions of `CHECK` macros, of
which most used `goto cleanup`, and the rest `goto failure` or `goto out`.
There were another 10 definitions of `RETERR`, of which most were identical
to `CHECK`, but some simply returned a result code instead
of jumping to a cleanup label.

This has now been standardized throughout the code base: `RETERR` is for
returning an error code in the case of an error, and `CHECK` is for jumping
to a cleanup tag, which is now always called `cleanup`. Both macros are
defined in `isc/util.h`.

Merge branch 'each-check-and-cleanup' into 'main'

See merge request isc-projects/bind9!10472


Trivial merge