From b594e8defacae06fee1f01dff4fbb6ff44f8a7c9 Mon Sep 17 00:00:00 2001 From: Kevin Darbyshire-Bryant Date: Fri, 6 Mar 2020 10:31:17 +0000 Subject: [PATCH] rfc3315: fix incorrect logical '&&' warning rfc3315.c:1711:28: warning: use of logical '&&' with constant operand [-Wconstant-logical-operand] if (!(addr_list->flags && ADDRLIST_DECLINED) || ^ ~~~~~~~~~~~~~~~~~ It's a flag bit so should be bitwise '&' operator Signed-off-by: Kevin Darbyshire-Bryant --- src/rfc3315.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/rfc3315.c b/src/rfc3315.c index eec8776..43ed4f8 100644 --- a/src/rfc3315.c +++ b/src/rfc3315.c @@ -1708,7 +1708,7 @@ static int config_valid(struct dhcp_config *config, struct dhcp_context *context return 0; for (addr_list = config->addr6; addr_list; addr_list = addr_list->next) - if (!(addr_list->flags && ADDRLIST_DECLINED) || + if (!(addr_list->flags & ADDRLIST_DECLINED) || difftime(now, addr_list->decline_time) >= (float)DECLINE_BACKOFF) { addrpart = addr6part(&addr_list->addr.addr6);