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

am 2d5f41e4: Merge "Better handling of 0-length format conversions in convertAudioPortFromNative()." into mnc-dev

Browse Source 
* commit '2d5f41e4c1733ff3e606390a433a6a52be6e862a':
  Better handling of 0-length format conversions in convertAudioPortFromNative().
This commit is contained in:
Paul McLean
2015-06-18 22:12:30 +00:00
committed by Android Git Automerger
parent ed113f1848 2d5f41e4c1
commit fcff4c382d
1 changed files with 12 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
core/jni/android_media_AudioSystem.cpp
View File

@@ -856,7 +856,8 @@ static jint convertAudioPortFromNative(JNIEnv *env,
bool useInMask;
size_t numPositionMasks = 0;
size_t numIndexMasks = 0;
size_t numUniqueFormats;
size_t numUniqueFormats = 0;
ALOGV("convertAudioPortFromNative id %d role %d type %d name %s",
nAudioPort->id, nAudioPort->role, nAudioPort->type, nAudioPort->name);
@@ -907,12 +908,13 @@ static jint convertAudioPortFromNative(JNIEnv *env,
}
// formats
cFormats = new int[nAudioPort->num_formats];
numUniqueFormats = 0;
for (size_t index = 0; index < nAudioPort->num_formats; index++) {
int format = audioFormatFromNative(nAudioPort->formats[index]);
if (!hasFormat(cFormats, numUniqueFormats, format)) {
cFormats[numUniqueFormats++] = format;
if (nAudioPort->num_formats != 0) {
cFormats = new int[nAudioPort->num_formats];
for (size_t index = 0; index < nAudioPort->num_formats; index++) {
int format = audioFormatFromNative(nAudioPort->formats[index]);
if (!hasFormat(cFormats, numUniqueFormats, format)) {
cFormats[numUniqueFormats++] = format;
}
}
}
jFormats = env->NewIntArray(numUniqueFormats);
@@ -920,7 +922,9 @@ static jint convertAudioPortFromNative(JNIEnv *env,
jStatus = (jint)AUDIO_JAVA_ERROR;
goto exit;
}
env->SetIntArrayRegion(jFormats, 0, numUniqueFormats, cFormats);
if (numUniqueFormats != 0) {
env->SetIntArrayRegion(jFormats, 0, numUniqueFormats, cFormats);
}
// gains
jGains = env->NewObjectArray(nAudioPort->num_gains,