From 7749d98a14359200d6d37028f27f07fd823623a1 Mon Sep 17 00:00:00 2001 From: "W.C.A. Wijngaards" Date: Mon, 28 Feb 2022 14:51:35 +0100 Subject: [PATCH] - Fix for edns client subnet option add fix in removal code, from review. --- doc/Changelog | 1 + edns-subnet/subnetmod.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/doc/Changelog b/doc/Changelog index fe8ef0d83..082f18b99 100644 --- a/doc/Changelog +++ b/doc/Changelog @@ -8,6 +8,7 @@ - Fix edns client subnet to add the option based on the option list, so that it is not state dependent, after the state fix of #605 for double EDNS options. + - Fix for edns client subnet option add fix in removal code, from review. 25 February 2022: Wouter - Fix to detect that no IPv6 support means that IPv6 addresses are diff --git a/edns-subnet/subnetmod.c b/edns-subnet/subnetmod.c index 3c723bcc3..fcea71c31 100644 --- a/edns-subnet/subnetmod.c +++ b/edns-subnet/subnetmod.c @@ -172,7 +172,7 @@ int ecs_whitelist_check(struct query_info* qinfo, else { /* Outgoing ECS option is set, but we don't want to sent it to * this address, remove option. */ - if(!edns_opt_list_find(qstate->edns_opts_back_out, + if(edns_opt_list_find(qstate->edns_opts_back_out, qstate->env->cfg->client_subnet_opcode)) { edns_opt_list_remove(&qstate->edns_opts_back_out, qstate->env->cfg->client_subnet_opcode);