From: James Carter <jwcart2@gmail.com>
To: selinux@vger.kernel.org
Cc: stephen.smalley.work@gmail.com, James Carter <jwcart2@gmail.com>
Subject: [PATCH] libsepol: Use spaceship comparison operator when comparing ibpkeys
Date: Wed, 12 Aug 2026 14:28:07 -0400 [thread overview]
Message-ID: <20260812182807.7311-1-jwcart2@gmail.com> (raw)
The comparsion function for ibpkeys in kernel_to_common subtracts
the subnet_prefixes of the two ibpkeys which are unsigned 64-bit
values and stores the result in an int which might not be able
to contain the result.
Use the spaceship comparison operator to compare the subnet_prefxes
to avoid any issues.
Signed-off-by: James Carter <jwcart2@gmail.com>
---
libsepol/src/kernel_to_common.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libsepol/src/kernel_to_common.c b/libsepol/src/kernel_to_common.c
index 338dcef6..f338b588 100644
--- a/libsepol/src/kernel_to_common.c
+++ b/libsepol/src/kernel_to_common.c
@@ -585,7 +585,8 @@ static int ibpkey_data_cmp(const void *a, const void *b)
struct ocontext *const *aa = a;
struct ocontext *const *bb = b;
- rc = (*aa)->u.ibpkey.subnet_prefix - (*bb)->u.ibpkey.subnet_prefix;
+ rc = spaceship_cmp((*aa)->u.ibpkey.subnet_prefix,
+ (*bb)->u.ibpkey.subnet_prefix);
if (rc)
return rc;
--
2.55.0
next reply other threads:[~2026-08-12 18:28 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-12 18:28 James Carter [this message]
2026-08-12 19:18 ` [PATCH] libsepol: Use spaceship comparison operator when comparing ibpkeys Stephen Smalley
2026-08-12 19:38 ` Stephen Smalley
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260812182807.7311-1-jwcart2@gmail.com \
--to=jwcart2@gmail.com \
--cc=selinux@vger.kernel.org \
--cc=stephen.smalley.work@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.