From aae3eaa20489daab7640ce9369fe0a472e4e2675 Mon Sep 17 00:00:00 2001 From: Edward Firmo <94725493+edwardtfn@users.noreply.github.com> Date: Fri, 8 Aug 2025 14:54:56 +0200 Subject: [PATCH] Validate `group_index` and add symmetric logging in `restore_dual` --- ESPHome/TX-Ultimate-Easy-ESPHome_core_hw_leds.yaml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/ESPHome/TX-Ultimate-Easy-ESPHome_core_hw_leds.yaml b/ESPHome/TX-Ultimate-Easy-ESPHome_core_hw_leds.yaml index 64fa2be..9878e4f 100644 --- a/ESPHome/TX-Ultimate-Easy-ESPHome_core_hw_leds.yaml +++ b/ESPHome/TX-Ultimate-Easy-ESPHome_core_hw_leds.yaml @@ -563,6 +563,10 @@ script: ESP_LOGE("${TAG_CORE_HW_LEDS}", "Invalid attributes pointer"); return; } + if (group_index != 1 && group_index != 2) { + ESP_LOGE("${TAG_CORE_HW_LEDS}", "Invalid group_index=%d (expected 1 or 2)", group_index); + return; + } auto attrs_pair = tx_ultimate_easy::unpack_dual_light_attributes(*attr_global_ptr); tx_ultimate_easy::LightAttributes attrs = (group_index == 1) ? attrs_pair.first : attrs_pair.second; @@ -570,6 +574,8 @@ script: call.set_brightness(attrs.brightness / 100.0f); call.set_rgb(attrs.red / 255.0f, attrs.green / 255.0f, attrs.blue / 255.0f); call.perform(); + ESP_LOGI("${TAG_CORE_HW_LEDS}", "Restored (group=%d) attributes: brightness=%d%%, RGB=(%d,%d,%d)", + group_index, attrs.brightness, attrs.red, attrs.green, attrs.blue); - id: light_attributes_save mode: parallel