From 0ee3155123c7c4876dcbd4fedbc44947beb2e5b7 Mon Sep 17 00:00:00 2001
From: "J. Nick Koston" <nick@koston.org>
Date: Thu, 14 Aug 2025 22:20:08 -0500
Subject: [PATCH] fix

---
 esphome/components/esp32_ble/ble_event.h | 24 ++++++++++++++++++------
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/esphome/components/esp32_ble/ble_event.h b/esphome/components/esp32_ble/ble_event.h
index ff9f1fcf3f..bb551bd867 100644
--- a/esphome/components/esp32_ble/ble_event.h
+++ b/esphome/components/esp32_ble/ble_event.h
@@ -344,15 +344,23 @@ class BLEEvent {
     switch (e) {
       case ESP_GATTC_NOTIFY_EVT:
         this->event_.gattc.data_len = p->notify.value_len;
-        this->event_.gattc.data = new uint8_t[p->notify.value_len];
-        memcpy(this->event_.gattc.data, p->notify.value, p->notify.value_len);
+        if (p->notify.value_len > 0) {
+          this->event_.gattc.data = new uint8_t[p->notify.value_len];
+          memcpy(this->event_.gattc.data, p->notify.value, p->notify.value_len);
+        } else {
+          this->event_.gattc.data = nullptr;
+        }
         this->event_.gattc.gattc_param->notify.value = this->event_.gattc.data;
         break;
       case ESP_GATTC_READ_CHAR_EVT:
       case ESP_GATTC_READ_DESCR_EVT:
         this->event_.gattc.data_len = p->read.value_len;
-        this->event_.gattc.data = new uint8_t[p->read.value_len];
-        memcpy(this->event_.gattc.data, p->read.value, p->read.value_len);
+        if (p->read.value_len > 0) {
+          this->event_.gattc.data = new uint8_t[p->read.value_len];
+          memcpy(this->event_.gattc.data, p->read.value, p->read.value_len);
+        } else {
+          this->event_.gattc.data = nullptr;
+        }
         this->event_.gattc.gattc_param->read.value = this->event_.gattc.data;
         break;
       default:
@@ -389,8 +397,12 @@ class BLEEvent {
     switch (e) {
       case ESP_GATTS_WRITE_EVT:
         this->event_.gatts.data_len = p->write.len;
-        this->event_.gatts.data = new uint8_t[p->write.len];
-        memcpy(this->event_.gatts.data, p->write.value, p->write.len);
+        if (p->write.len > 0) {
+          this->event_.gatts.data = new uint8_t[p->write.len];
+          memcpy(this->event_.gatts.data, p->write.value, p->write.len);
+        } else {
+          this->event_.gatts.data = nullptr;
+        }
         this->event_.gatts.gatts_param->write.value = this->event_.gatts.data;
         break;
       default: