Tracking provider cannot change behavior of upstream provider

Tracking provider cannot change behavior of the upstream provider.
Tracking provider should be transparent for the upstream provider.

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

src/provider/provider_tracking.c

@@ -110,10 +110,10 @@ static umf_result_t trackingAlloc(void *hProvider, size_t size,
     }
 
     ret = umfMemoryTrackerAdd(p->hTracker, p->pool, *ptr, size);
-    if (ret != UMF_RESULT_SUCCESS && p->hUpstream) {
-        if (umfMemoryProviderFree(p->hUpstream, *ptr, size)) {
-            // TODO: LOG
-        }
+    if (ret != UMF_RESULT_SUCCESS) {
+        // DO NOT call umfMemoryProviderFree() here, because the tracking provider
+        // cannot change behaviour of the upstream provider.
+        // TODO: LOG
     }
 
     return ret;
@@ -131,7 +131,9 @@ static umf_result_t trackingFree(void *hProvider, void *ptr, size_t size) {
     if (ptr) {
         ret = umfMemoryTrackerRemove(p->hTracker, ptr, size);
         if (ret != UMF_RESULT_SUCCESS) {
-            return ret;
+            // DO NOT return an error here, because the tracking provider
+            // cannot change behaviour of the upstream provider.
+            // TODO: LOG
         }
     }