Remove the downwards check for overlapping regions when the subtree flag is

set.

There's a bit of inverted logic in there and this feature will probably get
in the way of the blocklist feature (and maybe others)

OK tb@

diff --git a/usr.sbin/snmpd/application.c b/usr.sbin/snmpd/application.c
index dad8ce4..a4a7da4 100644 (file)
--- a/usr.sbin/snmpd/application.c
+++ b/usr.sbin/snmpd/application.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: application.c,v 1.12 2022/08/29 18:05:08 martijn Exp $        */
+/*     $OpenBSD: application.c,v 1.13 2022/08/30 17:33:20 martijn Exp $        */
 
 /*
  * Copyright (c) 2021 Martijn van Duren <martijn@openbsd.org>
@@ -237,14 +237,6 @@ appl_region(struct appl_context *ctx, uint32_t timeout, uint8_t priority,
            region->ar_backend != backend)
                goto overlap;
 
-       search.ar_oid = *oid;
-       region = RB_NFIND(appl_regions, &(ctx->ac_regions), &search);
-       if (region != NULL && region->ar_subtree && 
-           region->ar_backend != backend && (
-           appl_region_cmp(&search, region) == 0 ||
-           appl_region_cmp(&search, region) == -2))
-               goto overlap;
-
        if ((nregion = malloc(sizeof(*nregion))) == NULL) {
                log_warn("%s: Can't register %s: Processing error",
                    backend->ab_name, oidbuf);