Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit db5075c9 authored by Luis A. Lozano's avatar Luis A. Lozano Committed by android-build-merger
Browse files

Merge "Fix "use of memory after freed" warning."

am: 3f5160ca

Change-Id: I3b431990684662b4a504b35ff72c673e53fa7e03
parents fb23ba21 3f5160ca
Loading
Loading
Loading
Loading
+9 −1
Original line number Diff line number Diff line
@@ -289,7 +289,15 @@ SimpleBestFitAllocator::SimpleBestFitAllocator(size_t size)
SimpleBestFitAllocator::~SimpleBestFitAllocator()
{
    while(!mList.isEmpty()) {
        delete mList.remove(mList.head());
        chunk_t* removed = mList.remove(mList.head());
#ifdef __clang_analyzer__
        // Clang static analyzer gets confused in this loop
        // and generates a false positive warning about accessing
        // memory that is already freed.
        // Add an "assert" to avoid the confusion.
        LOG_ALWAYS_FATAL_IF(mList.head() == removed);
#endif
        delete removed;
    }
}