tests: use _unused_ in test_fdset and test_strtonum

libknot: merge attributes with libdnssec/shared/shared.h

tests: fix 'unused variable' warning if NDEBUG

catalog: fix 'unused variable' warning if NDEBUG

Merge branch 'lmdb-del' into 'master'

libknot/lmdb: tiny fix in lmdb del()

See merge request knot/knot-dns!1294

libknot/lmdb: tiny fix in lmdb del()

Passing the data means to only delete the data that matches,
which is not what we wanted to do here.  The only luck has been
that LMDB ignores this parameter when MDB_DUPSORT isn't in use,
but for example the mdbx for does not.

Merge branch 'zone_backup_filters' into 'master'

add new filters to "zone-backup"/"zone-restore" that allow detailed selection of backup/restore components

See merge request knot/knot-dns!1293

libknot/lmdb: further make_dir() simplification

src: replace mkdir() with make_dir() where suitable

A cleaner solution, but not necessary since the code will fail later for ENOTDIR anyway.

ctl: unify the flags ordering

zone backup: add new filters (detailed selection of backup components)

The filters are now called "filters" in zone-backup/zone-restore, not "flags" as recently.

ctl: add new filters to "zone-backup" and opposite filters evaluation

Abusing of CTL_FILTER_FLUSH_OUTDIR and CTL_FILTER_PURGE_* for backup purposes stops here and
dedicated filter values for backup/restore are created.

libknot/error: add a new error code KNOT_EXPARAM (conflicting parameters)

Merge branch 'nsec_bind_fixes' into 'master'

nameserver: prove wildcards in referral answers

See merge request knot/knot-dns!1292

tests: deal with Bind bug

nameserver: prove all wildcards in wildcard-cname loop

nameserver: dont include wildcard proofs for nonauth data

Revert "internet: don't prove wildcards in referral answers"

This reverts commit 747869b5f806402f70b54bab982b301318ed6b55.

nameserver: dont prove wildcard nonex on empty-non-term query

tests: basic/query compare also with NSEC and NSEC3

Merge branch 'libknot_lf_optimi' into 'master'

libknot: optimize knot_dname_lf for many short labels

See merge request knot/knot-dns!1250

dname: optimize knot_dname_lf() by combination of memset and direct asignment for 1-octet labels

dname: optimize knot_dname_lf() for many short labels

Merge branch 'ede_in_knotd' into 'master'

knotd: implemented extended error in EDNS (EDE)

See merge request knot/knot-dns!1220

knotd: don't return EDE_NOT_READY if zone contents not available

This change reduces code complexity and information leakage.

knotd: remove EDE where redundant or incorrect, refactoring

knotd: implemented extended error in EDNS (EDE)

Merge branch 'load_dnssec_avoid_flush' into 'master'

Don't flush up-to-date signed zone after start

See merge request knot/knot-dns!1290

zone-update: tiny refactoring

zone: don't flush upon start if DNSSEC is up-to-date

zscanner: set unspecified TTL in compliance with RFC-1035

"Omitted class and TTL values are default to the last explicitly stated values."

Merge branch 'cdnskey_public_only' into 'master'

CDNSKEY public only

See merge request knot/knot-dns!1289

DS query: enable checking multi-rr RRset

DNSSEC: allow public-only KSK in READY state (CDS published)

files: fix memory leaks in same_path()

backup: don't create a backup directory for restore

With this change, restore from a non-existent directory stops quickly,
with clear error and without overwriting current zone files.

catalog: make sure the target path exists for catalog_copy() -- needed by zone-restore

kjournalprint: fix some typos

zone-backup: fixes for some corner cases

In the restore mode, don't create the target storage directory if there's nothing to restore.
Don't forget to remove the original zone file, if it's to be removed.

ctl: zone-backup -- don't allow backup to/restore from the DB storage directory

At least, try to detect the most common condition, (database.) journal-db, kasp-db, timer-db,
and catalog-db aren't checked for. Values of zone.storage aren't tested for either.

contrib: utility for comparing two paths

contrib: independent absolute path string utility

Separated from conf_abs_path() in src/knot/conf/conf.c .

zone backup: create the backup directory in the proper way

tests: don't pollute default Knot install directories with test data

zone backup: treat non-existing zone files properly

contrib: copy_file() -- make it recognisable, that it is the source file that doesn't exist

Not being used so far.

tests: add unit test for libknot/probe

libknot/probe: use fchmod(777) on Linux only (doesn't work on BSD*)

doc/knsupdate: improve description of the zone parameter

conf: add notice log if policy.nsec3-iterations > 20

Merge branch 'ksk_publish_active' into 'master'

dnssec: BUGFIX sign DNSKEY with all published KSKs...

See merge request knot/knot-dns!1288

doc: update key rollover example

tests: align with new behaviour of KSK rollover

doc: align with new KSK retirement

dnssec: bugfix: remove key when remove pre-planned

dnssec: remove rolled-over KSK immediately...

...since is not used nor published anyway

dnssec: don't publish retired KSK, it's useless

dnssec: BUGFIX sign DNSKEY with all published KSKs...

...automatic KSK roll must ensure that new KSK has been signing
the DNSKEY for dnskey-ttl seconds before submission updates DS

doc/probe: fix typo

notify: remove unused constant NOTIFY_TIMEOUT

net: simplify initialization of msghdr

libknot/probe: use recvmsg if recvmmsg not available

distro/deb: add symbols for libknot/probe

mod-probe: add maximum rate limiting

modules: add module DNS probe

libknot: add DNS probe API

clang analyzer: double free fix

Merge branch 'keymgr_pregenerate_from' into 'master'

keymgr: offline KSK: may specify pregenerate from

See merge request knot/knot-dns!1287

keymgr: offline KSK: may specify pregenerate from

Merge branch 'ci-distro-update' into 'master'

distro/tests: update fedora, ubuntu releases

See merge request knot/knot-dns!1286

distro/tests: update fedora, ubuntu releases

fedora32 -> fedora34
ubuntu2010 -> ubuntu2104
remove ubuntu1604

Merge branch 'keymgr_list_zones' into 'master'

Keymgr list zones

See merge request knot/knot-dns!1285

keymgr: enable listing zones (based on stored keys)

KASP db: refactor/fix zone-related record classes

keymgr: proper error handling within del-all-old

m4/sanitizer: remove broken Clang version check

tcp-handler: avoid garbage value access (Clang analyzer)

catalog: fix uninitialized function argument (Clang analyzer)

tests: mute Coverity

tests/contrib/test_net: Use IPv4 loopback socket instead of IPv6

In some build environments it may not be possible to bind to ::1. Bind
to INADDR_LOOPBACK instead, which prevents this test suite from failing
or crashing.

tests/contrib/test_net: Don't assert on net_stream_recv() failure

This commit removes the assert() in _sync() and propagates the error to
the caller.

It was possible for the test_dns_tcp() test to set up an invalid socket
descriptor and not notice that the socket was not usable, which would
cause socket reads to fail, which would trigger the assert. This can
occur in some CI environments where IPv6 networking is not available.
These tests should probably just fail instead of crashing the build.

Here is what the strace output looked like when this test asserted:

    write(1, "# DNS messages over TCP\n", 24) = 24
    socket(PF_INET6, SOCK_STREAM|SOCK_NONBLOCK, IPPROTO_IP) = 3
    setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0
    setsockopt(3, SOL_IPV6, IPV6_V6ONLY, [1], 4) = 0
    bind(3, {sa_family=AF_INET6, sin6_port=htons(0), inet_pton(AF_INET6, "::1", &sin6_addr), sin6_flowinfo=0, sin6_scope_id=0}, 28) = -1 EADDRNOTAVAIL (Cannot assign requested address)
    close(3)                                = 0
    write(1, "not ok 1 - single DNS, server, create socket\n", 45) = 45
    listen(-99, 5)                          = -1 EBADF (Bad file descriptor)
    write(1, "not ok 2 - single DNS, server, start listening\n", 47) = 47
    mmap(NULL, 8392704, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS|MAP_STACK, -1, 0) = 0x7f340dc41000
    mprotect(0x7f340dc41000, 4096, PROT_NONE) = 0
    clone(child_stack=0x7f340e440ff0, flags=CLONE_VM|CLONE_FS|CLONE_FILES|CLONE_SIGHAND|CLONE_THREAD|CLONE_SYSVSEM|CLONE_SETTLS|CLONE_PARENT_SETTID|CLONE_CHILD_CLEARTID, parent_tidptr=0x7f340e4419d0, tls=0x7f340e441700, child_tidptr=0x7f340e4419d0) = 80049
    write(1, "ok 3 - single DNS, server, start handler\n", 41) = 41
    getsockname(-99, 0x7fffc7378760, 0x7fffc737867c) = -1 EBADF (Bad file descriptor)
    write(1, "not ok 4 - check getsockname return\n", 36) = 36
    socket(PF_UNSPEC, SOCK_STREAM|SOCK_NONBLOCK, 0) = -1 EAFNOSUPPORT (Address family not supported by protocol)
    write(1, "not ok 5 - single DNS, client, create connected socket\n", 55) = 55
    write(2, "test_net: contrib/test_net.c:410: _sync: Assertion `r == sizeof(buf)' failed.\n", 78) = 78
    mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f340ec4d000
    rt_sigprocmask(SIG_UNBLOCK, [ABRT], NULL, 8) = 0
    tgkill(80048, 80048, SIGABRT)           = 0
    --- SIGABRT {si_signo=SIGABRT, si_code=SI_TKILL, si_pid=80048, si_uid=0} ---
    +++ killed by SIGABRT (core dumped) +++

(I commented out all of the test_*() calls in main() except the call to
test_dns_tcp(), so the test numbering on stdout is off.)

This keeps the test_net program from assert()'ing on IPv4-only and
IPv4/IPv6 machines, but perhaps a better solution would be to have
the loop in test_dns_tcp() skip to the next test case if required
socket operations fail.

Merge branch 'catalog_conf_groups' into 'master'

Catalog conf groups

Closes #704

See merge request knot/knot-dns!1279

zone-update: use update->new_cont instead of update->zone->contents for full catalog update

catalog: implemented interpreting member groups

zonedb: commit catalog before reusing members...

...so that new config applies on them

server: don't log socket capabilities in xdp-only mode

server: add warning if no interface is configured

distro/tests: uncomment default listen configuration

samples: move default server.listen to a comment

tsig: fix wrong errcode when 'time signed' equals 0 to return BADTIME

fixes #730

doc: specify unix socket

tcp-handler: don't print broken address

fdset: polling using kqueue on BSD systems

knotd: print error when unable to change owner of logging file

fixes #728

kzonecheck: allows '-' as substitution for '/dev/stdin'

relates #727

Doxygen: add missing module descriptions

modules: fix knotd_conf() with NULL id

queryacl: remove redundant pointer check

Merge branch 'qdata_addr_getters' into 'master'

Fix getting remote and local addresses in the XDP mode

See merge request knot/knot-dns!1284

knot+modules: fix getting remote and local addresses in the XDP mode