]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s4/rpc_server/dnsserver: cppcheck: Fix Uninitialized variable error.
authorNoel Power <noel.power@suse.com>
Wed, 22 May 2019 09:37:40 +0000 (09:37 +0000)
committerAndreas Schneider <asn@cryptomilk.org>
Tue, 4 Jun 2019 22:13:07 +0000 (22:13 +0000)
source4/rpc_server/dnsserver/dcerpc_dnsserver.c:715: error: uninitvar: Uninitialized variable: answer_integer <--[cppcheck]

This error is benign and somewhat false because the code pointed to does
not run (due to a different check) if answer_integer is not actually
initialsed. It is easy to squash it though by just initialising the var.

Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: Andreas Schneider <asn@samba.org>
source4/rpc_server/dnsserver/dcerpc_dnsserver.c

index 841557814a0290d4fbc39ee41a950285f3f679d3..1bb46a58339a6343870cb1522124c24e651074f7 100644 (file)
@@ -374,6 +374,7 @@ static WERROR dnsserver_query_server(struct dnsserver_state *dsstate,
        }
 
        is_integer = 0;
+       answer_integer = 0;
 
        if (strcasecmp(operation, "AddressAnswerLimit") == 0) {
                answer_integer = serverinfo->cAddressAnswerLimit;
@@ -834,7 +835,7 @@ static WERROR dnsserver_query_zone(struct dnsserver_state *dsstate,
                                        union DNSSRV_RPC_UNION *r)
 {
        uint8_t is_integer, is_addresses, is_string;
-       uint32_t answer_integer;
+       uint32_t answer_integer = 0;
        struct IP4_ARRAY *answer_iparray;
        struct DNS_ADDR_ARRAY *answer_addrarray;
        char *answer_string;