From 81cec6a25ae6447d3ec8b7169d522ea7992cfb5e Mon Sep 17 00:00:00 2001
From: Frank <91616163+softhack007@users.noreply.github.com>
Date: Fri, 6 Sep 2024 19:36:24 +0200
Subject: [PATCH] Update usermod_seven_segment_reloaded.h - prevent array
 bounds violation (solves #4121)

I'm not the maintainer of this usermod, but its obvious that the code might overrun array bounds, so fixing this.
---
 .../usermod_seven_segment_reloaded.h                            | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/usermods/seven_segment_display_reloaded/usermod_seven_segment_reloaded.h b/usermods/seven_segment_display_reloaded/usermod_seven_segment_reloaded.h
index 5c2fac0d4..572d4cebd 100644
--- a/usermods/seven_segment_display_reloaded/usermod_seven_segment_reloaded.h
+++ b/usermods/seven_segment_display_reloaded/usermod_seven_segment_reloaded.h
@@ -236,6 +236,8 @@ private:
   }
 
   void _setLeds(int lednr, int lastSeenLedNr, bool range, int countSegments, int number, bool colon) {
+    if ((lednr < 0) || (lednr >= umSSDRLength)) return;                                   // prevent array bounds violation
+    if ((number < 0) || (countSegments < 0) || (lastSeenLedNr <0)) return; // prevent array out of range access
 
     if ((colon && umSSDRColonblink) || umSSDRNumbers[number][countSegments]) {