[Bugfix] Use the correct taskData when retrieving descriptors

Author: h2zero

src/NimBLERemoteCharacteristic.cpp

@@ -27,9 +27,9 @@
 # include <climits>
 
 struct NimBLEDescriptorFilter {
-    NimBLERemoteDescriptor* dsc;
     const NimBLEUUID*       uuid;
-    void*                   taskData;
+    NimBLERemoteDescriptor* dsc{nullptr};
+    void*                   taskData{nullptr};
 };
 
 static const char* LOG_TAG = "NimBLERemoteCharacteristic";
@@ -120,6 +120,8 @@ bool NimBLERemoteCharacteristic::retrieveDescriptors(NimBLEDescriptorFilter* pFi
     NimBLEDescriptorFilter defaultFilter{nullptr, nullptr, &taskData};
     if (pFilter == nullptr) {
         pFilter = &defaultFilter;
+    } else {
+        pFilter->taskData = &taskData;
     }
 
     int rc = ble_gattc_disc_all_dscs(getClient()->getConnHandle(),
@@ -134,7 +136,7 @@ bool NimBLERemoteCharacteristic::retrieveDescriptors(NimBLEDescriptorFilter* pFi
 
     auto prevDscCount = m_vDescriptors.size();
     NimBLEUtils::taskWait(taskData, BLE_NPL_TIME_FOREVER);
-    rc = ((NimBLEUtils::TaskData*)pFilter->taskData)->m_flags;
+    rc = taskData.m_flags;
     if (rc != BLE_HS_EDONE) {
         NIMBLE_LOGE(LOG_TAG, "<< retrieveDescriptors(): failed: rc=%d %s", rc, NimBLEUtils::returnCodeToString(rc));
         return false;
@@ -156,8 +158,7 @@ bool NimBLERemoteCharacteristic::retrieveDescriptors(NimBLEDescriptorFilter* pFi
 NimBLERemoteDescriptor* NimBLERemoteCharacteristic::getDescriptor(const NimBLEUUID& uuid) const {
     NIMBLE_LOGD(LOG_TAG, ">> getDescriptor: uuid: %s", uuid.toString().c_str());
     NimBLEUUID             uuidTmp{uuid};
-    NimBLEUtils::TaskData  taskData(const_cast<NimBLERemoteCharacteristic*>(this));
-    NimBLEDescriptorFilter filter{nullptr, &uuidTmp, &taskData};
+    NimBLEDescriptorFilter filter{&uuidTmp};
 
     for (const auto& dsc : m_vDescriptors) {
         if (dsc->getUUID() == uuid) {