- Fix for edns client subnet option add fix in removal code, from review.

diff --git a/doc/Changelog b/doc/Changelog
index fe8ef0d838c547c3c47917a0a00d840376a54a77..082f18b991f9d18cd54cd05d363f22f3fa776b54 100644 (file)
--- 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 3c723bcc3d18c7197555517dac6ae9b54ce63f73..fcea71c31438d568c166b31b5670f7d2ef2265f7 100644 (file)
--- 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);