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

Merge "Fix system service crash on radio session death. DO NOT MERGE" into qt-qpr1-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2019-12-21 00:38:00 +00:00
committed by Android (Google) Code Review
parent 21cc2e5f7d 1b16c57b57
commit af229217da
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
services/core/java/com/android/server/broadcastradio/hal2/RadioModule.java
View File

@@ -233,18 +233,24 @@ class RadioModule {
} }
private void fanoutAidlCallbackLocked(AidlCallbackRunnable runnable) { private void fanoutAidlCallbackLocked(AidlCallbackRunnable runnable) {
List<TunerSession> deadSessions = null;
for (TunerSession tunerSession : mAidlTunerSessions) { for (TunerSession tunerSession : mAidlTunerSessions) {
try { try {
runnable.run(tunerSession.mCallback); runnable.run(tunerSession.mCallback);
} catch (DeadObjectException ex) { } catch (DeadObjectException ex) {
// The other side died without calling close(), so just purge it from our // The other side died without calling close(), so just purge it from our records.
// records.
Slog.e(TAG, "Removing dead TunerSession"); Slog.e(TAG, "Removing dead TunerSession");
mAidlTunerSessions.remove(tunerSession); if (deadSessions == null) {
deadSessions = new ArrayList<>();
}
deadSessions.add(tunerSession);
} catch (RemoteException ex) { } catch (RemoteException ex) {
Slog.e(TAG, "Failed to invoke ITunerCallback: ", ex); Slog.e(TAG, "Failed to invoke ITunerCallback: ", ex);
} }
} }
if (deadSessions != null) {
mAidlTunerSessions.removeAll(deadSessions);
}
} }
public android.hardware.radio.ICloseHandle addAnnouncementListener(@NonNull int[] enabledTypes, public android.hardware.radio.ICloseHandle addAnnouncementListener(@NonNull int[] enabledTypes,