diff options
| author | Lennart Poettering <lennart@poettering.net> | 2022-11-24 18:42:08 +0100 |
|---|---|---|
| committer | Yu Watanabe <watanabe.yu+github@gmail.com> | 2022-11-24 23:49:04 +0100 |
| commit | fb896517aeecc2a8ec16586a34a0249606eb9f66 (patch) | |
| tree | 5892e1d714f49b2125cc659ad71ee1a1a80f62d1 /src/resolve/resolved-dns-rr.c | |
| parent | Merge pull request #25473 from yuwata/mount-tool-cleanups (diff) | |
| download | systemd-fb896517aeecc2a8ec16586a34a0249606eb9f66.tar.xz systemd-fb896517aeecc2a8ec16586a34a0249606eb9f66.zip | |
resolved: remove inappropriate assert()
A NULL Bitmap object is by all our code considered identical to an empty
bitmap. Hence let's remove the entirely unnecessary assert().
The assert() can be triggered if debug monitoring is used an an empty
NSEC or NSEC3 RR is included in an answer resolved returns.
it's not really a security issue since enabling debug monitoring is a
manual step requiring root privileges, that is off by default. Moreover,
it's a "clean" assert(), i.e. the worst that happens is tha a coredump
is generated and resolved restarted.
Fixes: #25449
Diffstat (limited to 'src/resolve/resolved-dns-rr.c')
| -rw-r--r-- | src/resolve/resolved-dns-rr.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/resolve/resolved-dns-rr.c b/src/resolve/resolved-dns-rr.c index 8123ca1f98..d47cdbbd8e 100644 --- a/src/resolve/resolved-dns-rr.c +++ b/src/resolve/resolved-dns-rr.c @@ -1865,7 +1865,6 @@ static int type_bitmap_to_json(Bitmap *b, JsonVariant **ret) { unsigned t; int r; - assert(b); assert(ret); BITMAP_FOREACH(t, b) { |