Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "mCallback defined in BringYourOwnSection should be an object instead of a reference." into rvc-dev

Browse Source
This commit is contained in:
Wenjie Zhou
2020-05-15 02:06:30 +00:00
committed by Android (Google) Code Review
parent 103aed340e 751c7c947a
commit ad24e49e2f
4 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
cmds/incidentd/src/IncidentService.cpp
View File

@@ -351,9 +351,9 @@ Status IncidentService::reportIncidentToDumpstate(unique_fd stream,
Status IncidentService::registerSection(const int id, const String16& name16,
const sp<IIncidentDumpCallback>& callback) {
const char* name = String8(name16).c_str();
const String8 name = String8(name16);
const uid_t callingUid = IPCThreadState::self()->getCallingUid();
ALOGI("Uid %d registers section %d '%s'", callingUid, id, name);
ALOGI("Uid %d registers section %d '%s'", callingUid, id, name.c_str());
if (callback == nullptr) {
return Status::fromExceptionCode(Status::EX_NULL_POINTER);
}
@@ -363,11 +363,11 @@ Status IncidentService::registerSection(const int id, const String16& name16,
ALOGW("Error registering section %d: calling uid does not match", id);
return Status::fromExceptionCode(Status::EX_SECURITY);
}
mRegisteredSections.at(i) = new BringYourOwnSection(id, name, callingUid, callback);
mRegisteredSections.at(i) = new BringYourOwnSection(id, name.c_str(), callingUid, callback);
return Status::ok();
}
}
mRegisteredSections.push_back(new BringYourOwnSection(id, name, callingUid, callback));
mRegisteredSections.push_back(new BringYourOwnSection(id, name.c_str(), callingUid, callback));
return Status::ok();
}

4
cmds/incidentd/src/Section.cpp
View File

@@ -876,7 +876,9 @@ BringYourOwnSection::~BringYourOwnSection() {}
status_t BringYourOwnSection::BlockingCall(unique_fd& pipeWriteFd) const {
android::os::ParcelFileDescriptor pfd(std::move(pipeWriteFd));
mCallback->onDumpSection(pfd);
if(mCallback != nullptr) {
mCallback->onDumpSection(pfd);
}
return NO_ERROR;
}

2
cmds/incidentd/src/Section.h
View File

@@ -207,7 +207,7 @@ public:
virtual status_t BlockingCall(unique_fd& pipeWriteFd) const;
private:
const sp<IIncidentDumpCallback>& mCallback;
const sp<IIncidentDumpCallback> mCallback;
};

3
core/java/android/os/IIncidentDumpCallback.aidl
View File

@@ -25,7 +25,8 @@ import android.os.ParcelFileDescriptor;
*/
oneway interface IIncidentDumpCallback {
/**
* Dumps section data to the given ParcelFileDescriptor.
* Dumps section data to the given ParcelFileDescriptor, which needs to be
* closed properly after writing the data.
*/
void onDumpSection(in ParcelFileDescriptor fd);
}