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 5094c556a..643d2caac 100644
--- a/src/log.c
+++ b/src/log.c
@@ -6172,7 +6172,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);
}
@@ -6214,7 +6214,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);
}
@@ -6238,7 +6238,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);
}
--
Egor Shestakov
egor ascii(0x40) ved1 ascii(0x2E) me
