]> git.ipfire.org Git - thirdparty/knot-dns.git/commitdiff
knsupdate: don't print two similar errors
authorDaniel Salzman <daniel.salzman@nic.cz>
Thu, 3 Feb 2022 11:39:56 +0000 (12:39 +0100)
committerDaniel Salzman <daniel.salzman@nic.cz>
Fri, 4 Feb 2022 10:23:59 +0000 (11:23 +0100)
src/utils/knsupdate/knsupdate_params.c

index b62cd4d89adbffdb31bf9c8f9c010f151e023010..486bba1d6dddcbb8d38427b81057927bc1e24810 100644 (file)
@@ -1,4 +1,4 @@
-/*  Copyright (C) 2019 CZ.NIC, z.s.p.o. <knot-dns@labs.nic.cz>
+/*  Copyright (C) 2022 CZ.NIC, z.s.p.o. <knot-dns@labs.nic.cz>
 
     This program is free software: you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -58,10 +58,6 @@ static const style_t DEFAULT_STYLE_NSUPDATE = {
        .show_footer = false
 };
 
-static void parse_err(zs_scanner_t *s) {
-       ERR("failed to parse RR: %s\n", zs_strerror(s->error.code));
-}
-
 static int parser_set_default(zs_scanner_t *s, const char *fmt, ...)
 {
        /* Format string. */
@@ -72,13 +68,13 @@ static int parser_set_default(zs_scanner_t *s, const char *fmt, ...)
        va_end(ap);
 
        if (n < 0 || (size_t)n >= sizeof(buf)) {
-               return KNOT_ESPACE;
+               return ZS_EINVAL;
        }
 
        /* Buffer must contain newline */
        if (zs_set_input_string(s, buf, n) != 0 ||
            zs_parse_all(s) != 0) {
-               return KNOT_EPARSEFAIL;
+               return s->error.code;
        }
 
        return KNOT_EOK;
@@ -113,7 +109,7 @@ static int knsupdate_init(knsupdate_params_t *params)
 
        /* Initialize RR parser. */
        if (zs_init(&params->parser, ".", params->class_num, 0) != 0 ||
-           zs_set_processing(&params->parser, NULL, parse_err, NULL) != 0) {
+           zs_set_processing(&params->parser, NULL, NULL, NULL) != 0) {
                zs_deinit(&params->parser);
                return KNOT_ENOMEM;
        }
@@ -292,7 +288,7 @@ int knsupdate_set_ttl(knsupdate_params_t *params, const uint32_t ttl)
        if (ret == KNOT_EOK) {
                params->ttl = ttl;
        } else {
-               ERR("failed to set default TTL, %s\n", knot_strerror(ret));
+               ERR("failed to set default TTL, %s\n", zs_strerror(ret));
        }
        return ret;
 }
@@ -306,7 +302,7 @@ int knsupdate_set_origin(knsupdate_params_t *params, const char *origin)
        free(fqdn);
 
        if (ret != KNOT_EOK) {
-               ERR("failed to set default origin, %s\n", knot_strerror(ret));
+               ERR("failed to set default origin, %s\n", zs_strerror(ret));
        }
        return ret;
 }