CLEANUP: server: remove ambiguous check in srv_update_addr_port()

A leftover check was left by recent patch series about server
addr:svc_port propagation: a check on (msg) being set was performed
in srv_update_addr_port(), but msg is always set, so the check is not
needed and confuses coverity (See GH #2399)
This commit is contained in:
Aurelien DARRAGON 2023-12-22 14:42:22 +01:00
parent e6d0b87f7f
commit 3ebe7bef8d
1 changed files with 1 additions and 2 deletions

View File

@ -4167,8 +4167,7 @@ const char *srv_update_addr_port(struct server *s, const char *addr, const char
* s->svc_port is only relevant under inet context
*/
if ((s->addr.ss_family != AF_INET) && (s->addr.ss_family != AF_INET6)) {
if (msg)
chunk_printf(msg, "Update for the current server address family is only supported through configuration file.");
chunk_printf(msg, "Update for the current server address family is only supported through configuration file.");
goto out;
}