diff options
author | Lajos Molnar <lajos@google.com> | 2016-03-15 19:22:27 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2016-03-15 19:22:28 +0000 |
commit | bd4acc2f9b64b2694d3dd3d77a5758c91b565f3c (patch) | |
tree | 106ad5ac90883cf03c6cb6139141af20e550a17f /media/jni/android_media_Utils.cpp | |
parent | b5a4d9605f6ee009984165f1c894b3949dc8417a (diff) | |
parent | c04399feab8e51531975a4b404327511b094bbf6 (diff) |
Merge "media: don't expose private keys in MediaFormat" into nyc-dev
Diffstat (limited to 'media/jni/android_media_Utils.cpp')
-rw-r--r-- | media/jni/android_media_Utils.cpp | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/media/jni/android_media_Utils.cpp b/media/jni/android_media_Utils.cpp index 12833f4b357c..62685c94453c 100644 --- a/media/jni/android_media_Utils.cpp +++ b/media/jni/android_media_Utils.cpp @@ -307,6 +307,11 @@ status_t ConvertMessageToMap( AMessage::Type valueType; const char *key = msg->getEntryNameAt(i, &valueType); + if (!strncmp(key, "android._", 9)) { + // don't expose private keys (starting with android._) + continue; + } + jobject valueObj = NULL; switch (valueType) { @@ -460,6 +465,11 @@ status_t ConvertKeyValueArraysToMessage( env->ReleaseStringUTFChars((jstring)keyObj, tmp); tmp = NULL; + if (key.startsWith("android._")) { + // don't propagate private keys (starting with android._) + continue; + } + jobject valueObj = env->GetObjectArrayElement(values, i); if (env->IsInstanceOf(valueObj, stringClass.get())) { |