ldns_resolver_search_status: ensure returning pkt with LDNS_STATUS_OK

Ensure we return a pkt by returning the final one in case the input is
not an absolute dname and no error occurred getting a response packet.

The return of ldns_resolver_query_status can be LDNS_STATUS_OK, but the
rcode from pkt an error.  In such case, when all search domains are
exhausted and the final query is done on the input dname as is, *pkt is
set freed and set to NULL, while s still is LDNS_STATUS_OK.  As a
result, the function returns with LDNS_STATUS_OK and pkt NULL.

This particular return makes the caller unable to handle the return
sensically, other than guessing there was an error of some sort.  By
returning the last retrieved pkt, the caller is able to get the RCODE
and return a meaningful error (such as SERVFAIL).

Signed-off-by: Fabian Groffen <grobian@bitzolder.nl>

Merge pull request #281 from NLnetLabs/develop-vision-2

Add PRs and bug fixes to maintenance

Add PRs and bug fixes to maintenance

Merge pull request #280 from NLnetLabs/development-vision

Add development vision

Add development vision

Merge pull request #270 from NLnetLabs/features/additional-rr-types

Features/additional rr types

Thanks for the review @wcawijngaard

Merge pull request #251 from tgreenx/is_rrset

Updates to ldns_is_rrset()

Thanks @tgreenx ! Looks good to me!

Merge branch 'develop' into features/additional-rr-types

* Fix RR types NSAP-PTR, GPOS and RESINFO to print unquoted strings.

Move TTL check of ldns_is_rrset() to a new, separate function

* Add ldns-verify-zone -s option. It checks all signature results,
  instead of passing by when one RRSIG validates. That prints output
  for spurious RRSIGs, the failures for them.

* Fix #271: Intermittent build failure with multi-job
  builds (make -j).

The DSYNC, CLA and IPN RR types

Changelog entry for memory leaks

Merge branch 'develop' into bugfix/verify-zone-memory-leaks

Mention that dnssec_*_add_rr can return EQUAL_RR

This is an api change, so we bump version to ldns-1.9.0

Fix #266 in Changelog

Merge pull request #267 from NLnetLabs/bugfix/266

Fixes issue #266.

Reduce github actions platforms

Because they don't run anymore

Eliminate use after free warning

Alignment error

Address the memory leaks in ldns-verify-zone

As reported by CI

Fixes issue #266.

Merge branch 'master' into develop

Minor improvements to the ldns-signzone man page. (#264)

Note that zone output may also be NSEC3 hashed, not just NSEC hashed.

Refer to the SEP bit rather than the KSK bit as there is no KSK bit in a DNSKEY.

ldns-signzone man page typo correction. (#261)

* Fix #254: Unquoted "value" rdata for CAA records fail to validate.
  Follows the long string unquoted syntax from RFC8659, section 4.1.1.

Update ldns_is_rrset() to take into account TTL and do case insensitive comparison of owner names

Credit pnax instead of ghost

ghost is generic deleted user account within github.

Credit Paul for PR #245

Merge pull request #245 from paul-internetnz/develop

Make IPv6 DNSSEC tracing work, and revert to previous behaviour otherwise

Merge pull request #248 from tgreenx/cds_rr_compare_ds

Make ldns_rr_compare_{ds,ds_dnskey}() available for CDS and CDNSKEY RRs

Make ldns_rr_compare_ds() and ldns_rr_compare_ds_dnskey() available for CDS and CDNSKEY RRs

Credit tgreenx for #246 and #247

Merge pull request #247 from tgreenx/cdnskey_key_rr2ds

Make ldns_key_rr2ds() available for CDNSKEY RR

Merge pull request #246 from tgreenx/cdnskey_calc_keytag

Make ldns_calc_keytag() available for CDNSKEY RR

Make ldns_key_rr2ds() available for CDNSKEY RR

Make ldns_calc_keytag() available for CDNSKEY RR

Make IPv6 DNSSEC tracing work, and revert to previous behaviour otherwise

If drill is executed without "-6", the previous behavior continues, with only A records being used.

Thank Petr for PR #242

Merge pull request #242 from InfrastructureServices/openssl-no-engine

Thanks you @pemensik

Add NXNAME rrtype

Remove duplicit error check in engine key loading

ldns_key_set_algorithm cannot change state of the key. Check it only
once, return specialized error code if failed.

Use header presence status to avoid openssl engine

Include presence of openssl/engine.h as indication engines should not be
used. Skip engine if header is not present.

Remove unused openssl engine header from nsec3-hash

Allow building examples with turned off engine.

Reverted the shuffle commit

Because it broke make

Remove some memory leaks

Still zome left in ldns-verify-zone, but leaving it for now

Revert "Merge pull request #192 from trofi/fix-make-depends"

This reverts commit d7767a62a01e86823c6830ced02773b9335e60c7, reversing
changes made to 8d76771999f040fa8ad8879a841b413b4507e747.

Merge branch 'develop' of github.com:NLnetLabs/ldns into develop

Merge pull request #186 from k0ekk0ek/develop

Remove unused CI recipes and extend GitHub workflow

Merge branch 'develop' into develop

Attribute Felipe for unit test fixes

Attribute Sergei Trofimovich

Merge branch 'develop' of github.com:NLnetLabs/ldns into develop

Merge pull request #191 from FGasper/16-unit-tests-edns-fix-leak

16-unit-tests-edns: Fix leak & warnings

Rerun of make depend

Merge pull request #192 from trofi/fix-make-depends

Makefile.in: build config.h before any object

Merge branch 'develop' into fix-make-depends

BSDs dont work part 2

Okay, BSDs just dont run...

Attribute Felipe

And the BSD's don't like sudo

Merge pull request #199 from FGasper/macos_ci

Enable testing in macOS CI

Merge branch 'develop' into macos_ci

Attribute Roland

and a small dereferencing void * fix

Merge branch 'develop' of github.com:NLnetLabs/ldns into develop

Merge pull request #203 from FGasper/expand_ci

Expand CI: Cygwin, BSDs, musl libc, 32-bit, and big-endian

Merge branch 'develop' of github.com:NLnetLabs/ldns into develop

Merge pull request #205 from rijswijk/develop

Add functions to extract RFC 8914 extended errors

Attribute Jelte

Merge pull request #206 from tjeb/ldnswalk-improvement

improve 'next-label' algorithm in ldns-walk

Fix small mistake and attribute Pierre

Merge pull request #214 from khorben/khorben/freebsd-build-fix

ldns: fix the build in FreeBSD's base system

Attribute Ed

Merge pull request #216 from emaste/develop

Update declaration for function with no arguments

Attribute Ørjan Malde

Merge pull request #224 from Redfoxymoon/develop

progs: don't underlink to libldns

Attribute William and Fabrice

Merge pull request #227 from ffontaine/develop

keys.c: fix build with libressl >= 3.8.1

Merge pull request #231 from wsfulton/swig-4.2.0-support

const ldns_rdf * typemap fixes to support swig-4.2.0

Attribute florian for Swig work

Merge pull request #232 from fweimer-rh/swig-4.2

SWIG_Python_str_AsChar removal in SWIG 4.2.0

Merge pull request #233 from fweimer-rh/swig-32bit

32-bit compatibility for Python SWIG bindings

Attribute Enji Cooper for PR #241

Merge branch 'develop' of github.com:NLnetLabs/ldns into develop

Merge pull request #241 from ngie-eign/coverity-fixes

Address several issues with the ultimate goal being to fix Coverity cited problems

Bump version

Release 1.8.4 this month

Add WALLET rrtype

RESINFO became RFC

ldns_dane_match_any_cert_with_data: fix types

Both `i` and `n` should match the return type for `sk_X509_num` (which
is `int`, not `size_t`). This addresses a potential issue where
`sk_X509_num(..)` could return -1, resulting in an unnecessary number of
loop iterations and undesirable behavior.

Reported by: Coverity
Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>

ldns_convert_dsa_rrsig_rdf2asn1: fix memory leak

Prior to this change `dsasig` was not freed when calling `DSA_SIG_set0`
failed. Free `dsasig` on error in that code path now.

Reported by: Coverity
Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>

Fix typos (`the the` -> `the`)

Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>

Ignore build generated files

This change ignores several files which are generated by autotools, et
al.

Signed-off-by: Enji Cooper <yaneurabeya@gmail.com>

Merge pull request #240 from botovq/hash

Don't rely on libcrypto returning static buffers

Don't rely on libcrypto returning static buffers

ldns is one of very few applications relying on being able to pass
NULL as last argument to the one-step hashing functions. BoringSSL
has removed this functionality in 2017 [1] and LibreSSL 4.0 will do
the same. Applications can pass in a correctly-sized buffer on the
stack.

[1]: https://boringssl-review.googlesource.com/14528

Added RESINFO draft RR type

enable with: --enable-rrtype-resinfo
See https://datatracker.ietf.org/doc/draft-ietf-add-resolver-info

Updated configure.ac from autoupdate

Changelog entry for bug #237

Merge pull request #238 from NLnetLabs/bugfix/237

Bugfix/237 Resolver uses nameserver commented out in /etc/resolv.conf

Move resolv.conf token parser to parse.h

Including a doxygen comment stating the limitations