From 25a093494122684143c1e0f48807f637ec9c32af Mon Sep 17 00:00:00 2001 From: Guido Casati Date: Mon, 7 Apr 2025 12:15:19 +0200 Subject: [PATCH] Fix missing deref on pointer types in Bearer Context Management lib Note: the equality check macros are using a PRINT_ERROR function that is enabled only when E1AP_LOG_ERRORS is defined. --- openair2/E1AP/lib/e1ap_bearer_context_management.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openair2/E1AP/lib/e1ap_bearer_context_management.c b/openair2/E1AP/lib/e1ap_bearer_context_management.c index ad5d3dd337..42065c53f2 100644 --- a/openair2/E1AP/lib/e1ap_bearer_context_management.c +++ b/openair2/E1AP/lib/e1ap_bearer_context_management.c @@ -516,7 +516,7 @@ static bool eq_pdu_session_item(const pdu_session_to_setup_t *a, const pdu_sessi return false; _E1_EQ_CHECK_OPTIONAL_PTR(a, b, inactivityTimer); if (a->inactivityTimer && b->inactivityTimer) - _E1_EQ_CHECK_INT(a->inactivityTimer, b->inactivityTimer); + _E1_EQ_CHECK_INT(*a->inactivityTimer, *b->inactivityTimer); _E1_EQ_CHECK_OPTIONAL_IE(a, b, dlAggregateMaxBitRate, _E1_EQ_CHECK_LONG); return true; } @@ -767,7 +767,7 @@ bool eq_bearer_context_setup_request(const e1ap_bearer_setup_req_t *a, const e1a // Check optional IEs _E1_EQ_CHECK_OPTIONAL_PTR(a, b, inactivityTimerUE); if (a->inactivityTimerUE && b->inactivityTimerUE) - _E1_EQ_CHECK_INT(a->inactivityTimerUE, b->inactivityTimerUE); + _E1_EQ_CHECK_INT(*a->inactivityTimerUE, *b->inactivityTimerUE); _E1_EQ_CHECK_OPTIONAL_IE(a, b, ueDlMaxIPBitRate, _E1_EQ_CHECK_LONG); return true; }