[PATCH 1/1] common: event: check event_type_name() argument

In event_type_name() we should avoid possible buffer overruns by checking the type argument.
Addresses-Coverity-ID: 478862 Out-of-bounds access Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com --- common/event.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/common/event.c b/common/event.c index dc61b9672f3..16c2ba6cc92 100644 --- a/common/event.c +++ b/common/event.c @@ -56,7 +56,10 @@ _Static_assert(ARRAY_SIZE(type_name) == EVT_COUNT, "event type_name size"); const char *event_type_name(enum event_t type) { #if CONFIG_IS_ENABLED(EVENT_DEBUG) - return type_name[type]; + if (type < ARRAY_SIZE(type_name)) + return type_name[type]; + else + return "(unknown)"; #else return "(unknown)"; #endif

On Sun, Jan 28, 2024 at 08:58:55AM +0100, Heinrich Schuchardt wrote:
In event_type_name() we should avoid possible buffer overruns by checking the type argument.
Addresses-Coverity-ID: 478862 Out-of-bounds access Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com
Applied to u-boot/master, thanks!
participants (2)
-
Heinrich Schuchardt
-
Tom Rini