Tracking provider cannot change behavior of upstream provider

[ldorau]

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

[vinser52]

Could you please also fix the trackingAlloc function and align it with this PR. This part looks strange:

ret = umfMemoryTrackerAdd(p->hTracker, p->pool, *ptr, size);
    if (ret != UMF_RESULT_SUCCESS && p->hUpstream) {
        if (umfMemoryProviderFree(p->hUpstream, *ptr, size)) {
            // TODO: LOG
        }
    }

if we are not able to add new block to the tracker we should not call umfMemoryProviderFree, right?

[ldorau]

Could you please also fix the trackingAlloc function and align it with this PR. This part looks strange:
if we are not able to add new block to the tracker we should not call umfMemoryProviderFree, right?

Right. Done.