From abe2c84b1d73272a339daef71a50e27b370f5c0b Mon Sep 17 00:00:00 2001 From: Mark Andrews Date: Wed, 24 Jun 2020 14:31:00 +1000 Subject: [PATCH] Suppress cppcheck warnings: cppcheck-suppress objectIndex cppcheck-suppress nullPointerRedundantCheck --- bin/dnssec/dnssec-keyfromlabel.c | 4 ++++ bin/dnssec/dnssec-keygen.c | 2 ++ lib/ns/client.c | 2 ++ 3 files changed, 8 insertions(+) diff --git a/bin/dnssec/dnssec-keyfromlabel.c b/bin/dnssec/dnssec-keyfromlabel.c index dc3b8d2428..e12169043a 100644 --- a/bin/dnssec/dnssec-keyfromlabel.c +++ b/bin/dnssec/dnssec-keyfromlabel.c @@ -362,6 +362,7 @@ main(int argc, char **argv) { setup_logging(mctx, &log); if (predecessor == NULL) { + /* cppcheck-suppress nullPointerRedundantCheck */ if (label == NULL) { fatal("the key label was not specified"); } @@ -383,6 +384,7 @@ main(int argc, char **argv) { isc_result_totext(ret)); } + /* cppcheck-suppress nullPointerRedundantCheck */ if (strchr(label, ':') == NULL) { char *l; int len; @@ -394,11 +396,13 @@ main(int argc, char **argv) { label = l; } + /* cppcheck-suppress nullPointerRedundantCheck */ if (algname == NULL) { fatal("no algorithm specified"); } r.base = algname; + /* cppcheck-suppress nullPointerRedundantCheck */ r.length = strlen(algname); ret = dns_secalg_fromtext(&alg, &r); if (ret != ISC_R_SUCCESS) { diff --git a/bin/dnssec/dnssec-keygen.c b/bin/dnssec/dnssec-keygen.c index 996b3ac6a6..849947f08d 100644 --- a/bin/dnssec/dnssec-keygen.c +++ b/bin/dnssec/dnssec-keygen.c @@ -1180,10 +1180,12 @@ main(int argc, char **argv) { } if (ctx.predecessor == NULL && ctx.policy == NULL) { + /* cppcheck-suppress nullPointerRedundantCheck */ if (algname == NULL) { fatal("no algorithm specified"); } r.base = algname; + /* cppcheck-suppress nullPointerRedundantCheck */ r.length = strlen(algname); ret = dns_secalg_fromtext(&ctx.alg, &r); if (ret != ISC_R_SUCCESS) { diff --git a/lib/ns/client.c b/lib/ns/client.c index 00badb4ac5..d91380aed0 100644 --- a/lib/ns/client.c +++ b/lib/ns/client.c @@ -1365,7 +1365,9 @@ process_ecs(ns_client_t *client, isc_buffer_t *buf, size_t optlen) { if ((addrlen % 8) != 0) { uint8_t bits = ~0U << (8 - (addrlen % 8)); + /* cppcheck-suppress objectIndex */ bits &= paddr[addrbytes - 1]; + /* cppcheck-suppress objectIndex */ if (bits != paddr[addrbytes - 1]) { return (DNS_R_OPTERR); }