Bugfix: add missing taskData when retrieving all descriptors

Author: h2zero

src/NimBLERemoteCharacteristic.cpp

@@ -115,7 +115,13 @@ bool NimBLERemoteCharacteristic::retrieveDescriptors(NimBLEDescriptorFilter* fil
         return false;
     }
 
-    NimBLEUtils::taskWait(filter->taskData, BLE_NPL_TIME_FOREVER);
+    if (filter == nullptr) {
+        NimBLETaskData         taskData(const_cast<NimBLERemoteCharacteristic*>(this));
+        NimBLEDescriptorFilter filter{nullptr, nullptr, &taskData};
+        NimBLEUtils::taskWait(filter.taskData, BLE_NPL_TIME_FOREVER);
+    } else {
+        NimBLEUtils::taskWait(filter->taskData, BLE_NPL_TIME_FOREVER);
+    }
     rc = ((NimBLETaskData*)filter->taskData)->m_flags;
     if (rc != BLE_HS_EDONE) {
         NIMBLE_LOGE(LOG_TAG, "<< retrieveDescriptors(): failed: rc=%d %s", rc, NimBLEUtils::returnCodeToString(rc));
@@ -135,7 +141,7 @@ bool NimBLERemoteCharacteristic::retrieveDescriptors(NimBLEDescriptorFilter* fil
 NimBLERemoteDescriptor* NimBLERemoteCharacteristic::getDescriptor(const NimBLEUUID& uuid) const {
     NIMBLE_LOGD(LOG_TAG, ">> getDescriptor: uuid: %s", uuid.toString().c_str());
     NimBLETaskData         taskData(const_cast<NimBLERemoteCharacteristic*>(this));
-    NimBLEDescriptorFilter filter = {nullptr, &uuid, &taskData};
+    NimBLEDescriptorFilter filter{nullptr, &uuid, &taskData};
     NimBLEUUID             uuidTmp;
 
     for (const auto& dsc : m_vDescriptors) {