From 2cdebaf7027e4f32d70dea9fea373ba5f052b548 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20Men=C5=A1=C3=ADk?= Date: Wed, 19 Jan 2022 14:47:13 +0100 Subject: [PATCH] Initialize printed buffer - var_decl: Declaring variable "tbuf" without initializer - assign: Assigning: "target.base" = "tbuf", which points to uninitialized data - assign: Assigning: "r.base" = "target.base", which points to uninitialized data I expect it would correctly initialize length always. Add simple initialization to silent coverity. (cherry picked from commit 59132bd3ec8a9f648a1e1cf59f5f3b2d59f17927) --- bin/dig/host.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/bin/dig/host.c b/bin/dig/host.c index 9befd12e7e..7b6acade27 100644 --- a/bin/dig/host.c +++ b/bin/dig/host.c @@ -211,15 +211,9 @@ printsection(dns_message_t *msg, dns_section_t sectionid, isc_result_t result, loopresult; isc_region_t r; dns_name_t empty_name; - char tbuf[4096]; + char tbuf[4096] = { 0 }; bool first; - bool no_rdata; - - if (sectionid == DNS_SECTION_QUESTION) { - no_rdata = true; - } else { - no_rdata = false; - } + bool no_rdata = (sectionid == DNS_SECTION_QUESTION); if (headers) { printf(";; %s SECTION:\n", section_name);