also removed unnecessary 'msg != NULL' check
*** CID 352815: Control flow issues (DEADCODE) /lib/dns/xfrin.c: 1363 in xfrin_send_request()
1357 isc_nmhandle_attach(send_xfr->handle, &xfr->sendhandle);
1358 isc_refcount_increment0(&send_xfr->sends);
1359 isc_nm_send(xfr->handle, ®ion, xfrin_send_done, send_xfr);
1360
1361 failure:
1362 if (qname != NULL) {
>>> CID 352815: Control flow issues (DEADCODE)
>>> Execution cannot reach this statement: "dns_message_puttempname(msg...".
1363 dns_message_puttempname(msg, &qname);
1364 }
1365 if (qrdataset != NULL) {
1366 dns_message_puttemprdataset(msg, &qrdataset);
1367 }
1368 if (msg != NULL) {
*** CID 352819: Control flow issues (DEADCODE) /lib/dns/xfrin.c: 1366 in xfrin_send_request()
1360
1361 failure:
1362 if (qname != NULL) {
1363 dns_message_puttempname(msg, &qname);
1364 }
1365 if (qrdataset != NULL) {
>>> CID 352819: Control flow issues (DEADCODE)
>>> Execution cannot reach this statement: "dns_message_puttemprdataset...".
1366 dns_message_puttemprdataset(msg, &qrdataset);
1367 }
1368 if (msg != NULL) {
1369 dns_message_detach(&msg);
1370 }
1371 if (soatuple != NULL) {
isc_nm_send(xfr->handle, ®ion, xfrin_send_done, send_xfr);
failure:
- if (qname != NULL) {
- dns_message_puttempname(msg, &qname);
- }
- if (qrdataset != NULL) {
- dns_message_puttemprdataset(msg, &qrdataset);
- }
- if (msg != NULL) {
- dns_message_detach(&msg);
- }
+ dns_message_detach(&msg);
if (soatuple != NULL) {
dns_difftuple_free(&soatuple);
}