From d946f2a2cfac469b04e1328e5e1be682be2d0440 Mon Sep 17 00:00:00 2001 From: tb Date: Wed, 5 Jan 2022 07:29:47 +0000 Subject: [PATCH] Readability tweaks in addr_contains() Assign to local variables to avoid ugly line wrapping. ok inoguchi jsing --- lib/libcrypto/x509/x509_addr.c | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/lib/libcrypto/x509/x509_addr.c b/lib/libcrypto/x509/x509_addr.c index 92d540dbe56..80260dca108 100644 --- a/lib/libcrypto/x509/x509_addr.c +++ b/lib/libcrypto/x509/x509_addr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: x509_addr.c,v 1.59 2022/01/05 07:28:41 tb Exp $ */ +/* $OpenBSD: x509_addr.c,v 1.60 2022/01/05 07:29:47 tb Exp $ */ /* * Contributed to the OpenSSL Project by the American Registry for * Internet Numbers ("ARIN"). @@ -1631,10 +1631,13 @@ X509v3_addr_inherits(IPAddrBlocks *addr) /* * Figure out whether parent contains child. + * + * This only works correctly if both parent and child are in canonical form. */ static int addr_contains(IPAddressOrRanges *parent, IPAddressOrRanges *child, int length) { + IPAddressOrRange *aorc, *aorp; unsigned char p_min[ADDR_RAW_BUF_LEN], p_max[ADDR_RAW_BUF_LEN]; unsigned char c_min[ADDR_RAW_BUF_LEN], c_max[ADDR_RAW_BUF_LEN]; int p, c; @@ -1646,15 +1649,20 @@ addr_contains(IPAddressOrRanges *parent, IPAddressOrRanges *child, int length) p = 0; for (c = 0; c < sk_IPAddressOrRange_num(child); c++) { - if (!extract_min_max(sk_IPAddressOrRange_value(child, c), - c_min, c_max, length)) + aorc = sk_IPAddressOrRange_value(child, c); + + if (!extract_min_max(aorc, c_min, c_max, length)) return 0; + for (;; p++) { if (p >= sk_IPAddressOrRange_num(parent)) return 0; - if (!extract_min_max(sk_IPAddressOrRange_value(parent, - p), p_min, p_max, length)) + + aorp = sk_IPAddressOrRange_value(parent, p); + + if (!extract_min_max(aorp, p_min, p_max, length)) return 0; + if (memcmp(p_max, c_max, length) < 0) continue; if (memcmp(p_min, c_min, length) > 0) -- 2.20.1