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

Commit 39ca2792 authored by Hans Boehm's avatar Hans Boehm Committed by android-build-merger
Browse files

Merge "Check sp<>::clear() for data races" 

am: 59cd8237

Change-Id: I8b7b4a60aa3f56ae0931b8b8edffd3913a638f3a
parents ae18d553 59cd8237

libutils/include/utils/StrongPointer.h

+4 −2
Original line number Diff line number Diff line
@@ -228,8 +228,10 @@ void sp<T>::force_set(T* other) { 


template<typename T>

void sp<T>::clear() {

    if (m_ptr) {

        m_ptr->decStrong(this);

    T* oldPtr(*const_cast<T* volatile*>(&m_ptr));

    if (oldPtr) {

        oldPtr->decStrong(this);

        if (oldPtr != *const_cast<T* volatile*>(&m_ptr)) sp_report_race();

        m_ptr = nullptr;

    }

}