Do not ignore named_server_dumpdb() result code

The named_server_dumpdb() function, which is called when a 'rndc dumpdb'
command is issued, returns a 'isc_result_t' result code and it has been
always ignored since its introduction in
eb8713ed947fdf22a41dad673d561896dd6fe4a2, where it was still called
ns_server_dumpdb(). The orignal reasoning is not preserved, but it could
have been also a simple copy-paste mistake, as there are commands, which
return 'void' and require manually setting 'result = ISC_R_SUCCESS;', as
it was done here. Anyway, named will now return the actual result, and
'rndc' will report an error, when the 'dumpdb' command fails.

diff --git a/bin/named/control.c b/bin/named/control.c
index a1a66a4eb30b10f68886b40d067113e2e3dedf0f..a99ce1b850852b7c119cfe6de2cec1035536de99 100644 (file)
--- a/bin/named/control.c
+++ b/bin/named/control.c
@@ -214,8 +214,7 @@ named_control_docommand(isccc_sexpr_t *message, bool readonly,
        {
                result = named_server_dnstap(named_g_server, lex, text);
        } else if (command_compare(command, NAMED_COMMAND_DUMPDB)) {
-               named_server_dumpdb(named_g_server, lex, text);
-               result = ISC_R_SUCCESS;
+               result = named_server_dumpdb(named_g_server, lex, text);
        } else if (command_compare(command, NAMED_COMMAND_DUMPSTATS)) {
                result = named_server_dumpstats(named_g_server);
        } else if (command_compare(command, NAMED_COMMAND_FETCHLIMIT)) {