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

Commit 9a9de84b authored by Songchun Fan's avatar Songchun Fan Committed by Automerger Merge Worker
Browse files

Merge "[incremental service] avoid nullptr dereferencing" into rvc-dev am: eee228f6 

Change-Id: I44a7218de67cc35d53711ac6dbdb296e1dd59b96
parents a2b2f3f3 eee228f6

services/incremental/IncrementalService.cpp

+5 −3
Original line number Diff line number Diff line
@@ -1090,7 +1090,9 @@ bool IncrementalService::prepareDataLoader(IncrementalService::IncFsMount& ifs, 
            base::unique_fd(::dup(ifs.control.pendingReads)));

    fsControlParcel.incremental->log.reset(base::unique_fd(::dup(ifs.control.logs)));

    sp<IncrementalDataLoaderListener> listener =

            new IncrementalDataLoaderListener(*this, *externalListener);

            new IncrementalDataLoaderListener(*this,

                                              externalListener ? *externalListener

                                                               : DataLoaderStatusListener());

    bool created = false;

    auto status = mDataLoaderManager->initializeDataLoader(ifs.mountId, *dlp, fsControlParcel,

                                                           listener, &created);
@@ -1230,8 +1232,8 @@ binder::Status IncrementalService::IncrementalDataLoaderListener::onStatusChange 
        std::unique_lock l(incrementalService.mLock);

        const auto& ifs = incrementalService.getIfsLocked(mountId);

        if (!ifs) {

            LOG(WARNING) << "Received data loader status " << int(newStatus) << " for unknown mount "

                         << mountId;

            LOG(WARNING) << "Received data loader status " << int(newStatus)

                         << " for unknown mount " << mountId;

            return binder::Status::ok();

        }

        ifs->dataLoaderStatus = newStatus;