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.

diff --git a/src/acme.c b/src/acme.c
index eb7137b4c890d21ab0bba94731ffbce7bc0d116d..437528c08fd4bbcfaef00b537eba8c7e4325a3de 100644 (file)
--- 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 af670e5d071bf201da2dfede3b895a94114b074b..053be72e2fa0989c08d6866ce9797105cb3a0f16 100644 (file)
--- 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 c90cefd90e8443e858ba42dcbe19241f2eb59faa..bf1f9a54372c8968ee230df68e7a1ef9b4618434 100644 (file)
--- 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 671eaee014a6601a3d32533993ed63eaa01b55f7..3e56d512d5723b2c4839ed7131ee3d8a49eaaa62 100644 (file)
--- 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);
                                }