2022-11-02 - BLE Client Fix deadlock if connection lost while readValue. Releases BLE read semaphore on BLE target disconnect to avoid deadlock on read process

espressif/arduino-esp32#7319

Author: DamianSuess

src/BLEAdvertisedDevice.cpp

@@ -571,7 +571,7 @@ std::string BLEAdvertisedDevice::toString() {
 			res += ", serviceData: " + getServiceData(i);
 		}
 	}
- 	return res;
+	return res;
 } // toString
 
 uint8_t* BLEAdvertisedDevice::getPayload() {

src/BLERemoteCharacteristic.cpp

@@ -247,9 +247,11 @@ void BLERemoteCharacteristic::gattClientEventHandler(esp_gattc_cb_event_t event,
 			break;
 
 		case ESP_GATTC_DISCONNECT_EVT:
+			// Cleanup semaphores to avoid deadlocks.
+			m_semaphoreReadCharEvt.give(1);
 			m_semaphoreWriteCharEvt.give(1);
 			break;
-			
+
 		default:
 			break;
 	} // End switch