]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Remove dead code
authorMark Andrews <marka@isc.org>
Mon, 18 Jul 2022 07:34:30 +0000 (17:34 +1000)
committerMark Andrews <marka@isc.org>
Tue, 6 Sep 2022 12:47:08 +0000 (12:47 +0000)
    *** CID 352812:  Control flow issues  (DEADCODE) /lib/ns/query.c: 8584 in query_filter64()
    8578     cleanup:
    8579      if (buffer != NULL) {
    8580      isc_buffer_free(&buffer);
    8581      }
    8582
    8583      if (myrdata != NULL) {
    >>>     CID 352812:  Control flow issues  (DEADCODE)
    >>>     Execution cannot reach this statement: "dns_message_puttemprdata(cl...".
    8584      dns_message_puttemprdata(client->message, &myrdata);
    8585      }
    8586
    8587      if (myrdataset != NULL) {
    8588      dns_message_puttemprdataset(client->message, &myrdataset);
    8589      }

lib/ns/query.c

index d1c969ca161a75b768e588259f3e09790860c6ff..ea676dd537eadec29573a7e496c1d54f286e4e8d 100644 (file)
@@ -8462,10 +8462,6 @@ cleanup:
                isc_buffer_free(&buffer);
        }
 
-       if (myrdata != NULL) {
-               dns_message_puttemprdata(client->message, &myrdata);
-       }
-
        if (myrdataset != NULL) {
                dns_message_puttemprdataset(client->message, &myrdataset);
        }
@@ -8480,6 +8476,7 @@ cleanup:
                }
                dns_message_puttemprdatalist(client->message, &myrdatalist);
        }
+
        if (qctx->dbuf != NULL) {
                ns_client_releasename(client, &name);
        }