From 61f04d195194f4444aeed1963b5f626e291dd9c9 Mon Sep 17 00:00:00 2001 From: Egor Shestakov Date: Sat, 11 Apr 2026 16:03:43 +0000 Subject: [PATCH] MINOR: errors: remove excessive errmsg checks I noticed some strange checks for presence of errmsg. Called functions generate non-empty error message in case of failure, so a non-NULL address of the error message is enough. No backport needed. --- src/acme.c | 4 ++-- src/cfgparse-listen.c | 22 +++++++++++----------- src/cfgparse-peers.c | 4 ++-- src/log.c | 6 +++--- 4 files changed, 18 insertions(+), 18 deletions(-) diff --git a/src/acme.c b/src/acme.c index eb7137b4c..437528c08 100644 --- a/src/acme.c +++ b/src/acme.c @@ -800,8 +800,8 @@ static int cfg_postsection_acme() /* tries to open the account key */ if (stat(path, &st) == 0) { if (ssl_sock_load_key_into_ckch(path, NULL, store->data, &errmsg)) { - memprintf(&errmsg, "%s'%s' is present but cannot be read or parsed.\n", errmsg && *errmsg ? errmsg : NULL, path); - if (errmsg && *errmsg) + memprintf(&errmsg, "%s'%s' is present but cannot be read or parsed.\n", errmsg, path); + if (errmsg) indent_msg(&errmsg, 8); err_code |= ERR_ALERT | ERR_FATAL | ERR_ABORT; ha_alert("acme: %s\n", errmsg); diff --git a/src/cfgparse-listen.c b/src/cfgparse-listen.c index af670e5d0..053be72e2 100644 --- a/src/cfgparse-listen.c +++ b/src/cfgparse-listen.c @@ -586,7 +586,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) * will have to be applied to all listeners created after last_listen. */ if (!str2listener(args[1], curproxy, bind_conf, file, linenum, &errmsg)) { - if (errmsg && *errmsg) { + if (errmsg) { indent_msg(&errmsg, 2); ha_alert("parsing [%s:%d] : '%s' : %s\n", file, linenum, args[0], errmsg); } @@ -1366,7 +1366,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) if (curproxy->cap & PR_CAP_BE) where |= SMP_VAL_BE_HRQ_HDR; err_code |= warnif_cond_conflicts(rule->cond, where, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); LIST_APPEND(&curproxy->http_req_rules, &rule->list); @@ -1401,7 +1401,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) if (curproxy->cap & PR_CAP_BE) where |= SMP_VAL_BE_HRS_HDR; err_code |= warnif_cond_conflicts(rule->cond, where, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); LIST_APPEND(&curproxy->http_res_rules, &rule->list); @@ -1435,7 +1435,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) if (curproxy->cap & PR_CAP_BE) where |= SMP_VAL_BE_HRS_HDR; err_code |= warnif_cond_conflicts(rule->cond, where, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); LIST_APPEND(&curproxy->http_after_res_rules, &rule->list); @@ -1500,7 +1500,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) if (curproxy->cap & PR_CAP_BE) where |= SMP_VAL_BE_HRQ_HDR; err_code |= warnif_cond_conflicts(rule->cond, where, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); } else if (strcmp(args[0], "use_backend") == 0) { @@ -1530,7 +1530,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) } err_code |= warnif_cond_conflicts(cond, SMP_VAL_FE_SET_BCK, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); } else if (*args[2]) { @@ -1593,7 +1593,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) } err_code |= warnif_cond_conflicts(cond, SMP_VAL_BE_SET_SRV, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); rule = calloc(1, sizeof(*rule)); @@ -1648,7 +1648,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) * where force-persist is applied. */ err_code |= warnif_cond_conflicts(cond, SMP_VAL_BE_REQ_CNT, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); rule = calloc(1, sizeof(*rule)); @@ -1816,7 +1816,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) err_code |= warnif_cond_conflicts(cond, SMP_VAL_BE_STO_RUL, &errmsg); else err_code |= warnif_cond_conflicts(cond, SMP_VAL_BE_SET_SRV, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); rule = calloc(1, sizeof(*rule)); @@ -1874,7 +1874,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) if (curproxy->cap & PR_CAP_BE) where |= SMP_VAL_BE_HRQ_HDR; err_code |= warnif_cond_conflicts(cond, where, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); rule = calloc(1, sizeof(*rule)); @@ -1954,7 +1954,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int kwm) if (curproxy->cap & PR_CAP_BE) where |= SMP_VAL_BE_HRQ_HDR; err_code |= warnif_cond_conflicts(rule->cond, where, &errmsg); - if (errmsg && *errmsg) + if (errmsg) ha_warning("parsing [%s:%d] : '%s.\n'", file, linenum, errmsg); LIST_APPEND(&curproxy->uri_auth->http_req_rules, &rule->list); diff --git a/src/cfgparse-peers.c b/src/cfgparse-peers.c index c90cefd90..bf1f9a543 100644 --- a/src/cfgparse-peers.c +++ b/src/cfgparse-peers.c @@ -187,7 +187,7 @@ int cfg_parse_peers(const char *file, int linenum, char **args, int kwm) } if (!str2listener(args[1], curpeers->peers_fe, bind_conf, file, linenum, &errmsg)) { - if (errmsg && *errmsg) { + if (errmsg) { indent_msg(&errmsg, 2); ha_alert("parsing [%s:%d] : '%s %s' : %s\n", file, linenum, args[0], args[1], errmsg); } @@ -453,7 +453,7 @@ int cfg_parse_peers(const char *file, int linenum, char **args, int kwm) } if (!str2listener(args[2], curpeers->peers_fe, bind_conf, file, linenum, &errmsg)) { - if (errmsg && *errmsg) { + if (errmsg) { indent_msg(&errmsg, 2); ha_alert("parsing [%s:%d] : '%s %s' : %s\n", file, linenum, args[0], args[1], errmsg); } diff --git a/src/log.c b/src/log.c index 671eaee01..3e56d512d 100644 --- a/src/log.c +++ b/src/log.c @@ -6174,7 +6174,7 @@ int cfg_parse_log_forward(const char *file, int linenum, char **args, int kwm) bind_conf->accept = session_accept_fd; if (!str2listener(args[1], cfg_log_forward, bind_conf, file, linenum, &errmsg)) { - if (errmsg && *errmsg) { + if (errmsg) { indent_msg(&errmsg, 2); ha_alert("parsing [%s:%d] : '%s %s' : %s\n", file, linenum, args[0], args[1], errmsg); } @@ -6216,7 +6216,7 @@ int cfg_parse_log_forward(const char *file, int linenum, char **args, int kwm) bind_conf->maxaccept = global.tune.maxaccept ? global.tune.maxaccept : MAX_ACCEPT; if (!str2receiver(args[1], cfg_log_forward, bind_conf, file, linenum, &errmsg)) { - if (errmsg && *errmsg) { + if (errmsg) { indent_msg(&errmsg, 2); ha_alert("parsing [%s:%d] : '%s %s' : %s\n", file, linenum, args[0], args[1], errmsg); } @@ -6240,7 +6240,7 @@ int cfg_parse_log_forward(const char *file, int linenum, char **args, int kwm) ret = kw->parse(args, cur_arg, cfg_log_forward, bind_conf, &errmsg); err_code |= ret; if (ret) { - if (errmsg && *errmsg) { + if (errmsg) { indent_msg(&errmsg, 2); ha_alert("parsing [%s:%d] : %s\n", file, linenum, errmsg); }