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

Merge "TIF: fix NPE for TvInputHardwareManager" into lmp-mr1-dev

Browse Source
This commit is contained in:
Jae Seo
2014-11-18 18:01:07 +00:00
committed by Android (Google) Code Review
parent ce66d35fc3 b683e6e4c7
commit f0a8c3e674
1 changed files with 19 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
services/core/java/com/android/server/tv/TvInputHardwareManager.java
View File

@@ -673,28 +673,35 @@ class TvInputHardwareManager implements TvInputHal.Callback {
if (mReleased) {
throw new IllegalStateException("Device already released.");
}
if (surface != null && config == null) {
return false;
}
if (surface == null && mActiveConfig == null) {
return false;
}
int result = TvInputHal.ERROR_UNKNOWN;
if (surface == null) {
result = mHal.removeStream(mInfo.getDeviceId(), mActiveConfig);
mActiveConfig = null;
// The value of config is ignored when surface == null.
if (mActiveConfig != null) {
result = mHal.removeStream(mInfo.getDeviceId(), mActiveConfig);
mActiveConfig = null;
} else {
// We already have no active stream.
return true;
}
} else {
if (!config.equals(mActiveConfig)) {
// It's impossible to set a non-null surface with a null config.
if (config == null) {
return false;
}
// Remove stream only if we have an existing active configuration.
if (mActiveConfig != null && !config.equals(mActiveConfig)) {
result = mHal.removeStream(mInfo.getDeviceId(), mActiveConfig);
if (result != TvInputHal.SUCCESS) {
mActiveConfig = null;
return false;
}
}
result = mHal.addOrUpdateStream(mInfo.getDeviceId(), surface, config);
// Proceed only if all previous operations succeeded.
if (result == TvInputHal.SUCCESS) {
mActiveConfig = config;
result = mHal.addOrUpdateStream(mInfo.getDeviceId(), surface, config);
if (result == TvInputHal.SUCCESS) {
mActiveConfig = config;
}
}
}
updateAudioConfigLocked();