]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
correct order checks
authorAlan T. DeKok <aland@freeradius.org>
Thu, 4 Jun 2026 12:23:04 +0000 (08:23 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Thu, 4 Jun 2026 12:23:04 +0000 (08:23 -0400)
src/lib/eap_aka_sim/id.c

index 800cc85d08a85d958a3ee7970fe32931667b840e..003520dc43b9bdef2df667f852dbda34dd73cfb3 100644 (file)
@@ -101,10 +101,10 @@ ssize_t fr_aka_sim_3gpp_root_nai_domain_mcc_mnc(uint16_t *mnc, uint16_t *mcc,
        char *q;
        unsigned long num;
 
-       if (((p + 8) < end) || (CRYPTO_memcmp(p, "wlan.mnc", 8) != 0)) return -1;
+       if (((p + 8) > end) || (CRYPTO_memcmp(p, "wlan.mnc", 8) != 0)) return -1;
        p += 8;
 
-       if (((p + 3) < end)) {
+       if (((p + 3) > end)) {
                fr_strerror_const("Missing MNC component");
                return (domain - p);
        }
@@ -116,7 +116,7 @@ ssize_t fr_aka_sim_3gpp_root_nai_domain_mcc_mnc(uint16_t *mnc, uint16_t *mcc,
        if (mnc) *mnc = (uint16_t)num;
        p = q + 1;
 
-       if (((p + 3) < end) || (CRYPTO_memcmp(p, "mcc", 3) != 0)) {
+       if (((p + 3) > end) || (CRYPTO_memcmp(p, "mcc", 3) != 0)) {
                fr_strerror_const("Missing MCC component");
                return (domain - p);
        }
@@ -128,7 +128,7 @@ ssize_t fr_aka_sim_3gpp_root_nai_domain_mcc_mnc(uint16_t *mnc, uint16_t *mcc,
        if (mcc) *mcc = (uint16_t)num;
 
        p = q + 1;
-       if (((p + 15) < end) || (CRYPTO_memcmp(p, "3gppnetwork.org", 15) != 0)) {
+       if (((p + 15) > end) || (CRYPTO_memcmp(p, "3gppnetwork.org", 15) != 0)) {
                fr_strerror_const("Missing 3gppnetwork.org suffix");
                return (domain - p);
        }