[S] Change in libosmocore[master]: select: Prevent negative index lookup on osmo_fd_lookup.table

pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/33748 ) Change subject: select: Prevent negative index lookup on osmo_fd_lookup.table ...................................................................... select: Prevent negative index lookup on osmo_fd_lookup.table Marked as WIP because I'm not entirely sure if negative index lookups were allowed on purpose (although I doubt it) Change-Id: Iba81873cbc435229599b1a8139cf4b627f8e6924 --- M src/core/select.c 1 file changed, 13 insertions(+), 1 deletion(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/core/select.c b/src/core/select.c index 69fa763..70047f0 100644 --- a/src/core/select.c +++ b/src/core/select.c @@ -522,7 +522,7 @@ * \returns \ref osmo_fd for \ref fd; NULL in case it doesn't exist */ struct osmo_fd *osmo_fd_get_by_fd(int fd) { - if (fd > maxfd) + if (fd > maxfd || fd < 0) return NULL; return osmo_fd_lookup.table[fd]; } -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/33748 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Iba81873cbc435229599b1a8139cf4b627f8e6924 Gerrit-Change-Number: 33748 Gerrit-PatchSet: 3 Gerrit-Owner: arehbein &lt;arehbein(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged