diff options
| author | Andreas Huber <andih@google.com> | 2012-02-21 14:38:23 -0800 |
|---|---|---|
| committer | Andreas Huber <andih@google.com> | 2012-02-22 16:20:59 -0800 |
| commit | 5c850396b39a57baabd37a9c0c8324f1bee408ca (patch) | |
| tree | f96a99dcb625214edb4262fc26e7f4cb6a18dcfa /media/jni | |
| parent | 592b213608b67b6c9a72ed080db56fa9fb2cdcf8 (diff) | |
| download | frameworks_base-5c850396b39a57baabd37a9c0c8324f1bee408ca.zip frameworks_base-5c850396b39a57baabd37a9c0c8324f1bee408ca.tar.gz frameworks_base-5c850396b39a57baabd37a9c0c8324f1bee408ca.tar.bz2 | |
Add new APIs AMessage::(set|find)Buffer to make it safer to pass
ABuffer objects through messages.
Change-Id: I9f8b4e4c4767d0d70a0105e0c0813b754379b49d
Diffstat (limited to 'media/jni')
| -rw-r--r-- | media/jni/android_media_Utils.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/media/jni/android_media_Utils.cpp b/media/jni/android_media_Utils.cpp index 7dacdcd..8b2321c 100644 --- a/media/jni/android_media_Utils.cpp +++ b/media/jni/android_media_Utils.cpp @@ -176,13 +176,10 @@ status_t ConvertMessageToMap( break; } - case AMessage::kTypeObject: + case AMessage::kTypeBuffer: { - sp<RefBase> obj; - CHECK(msg->findObject(key, &obj)); - - // XXX dangerous, object is not guaranteed to be a buffer. - sp<ABuffer> buffer = static_cast<ABuffer *>(obj.get()); + sp<ABuffer> buffer; + CHECK(msg->findBuffer(key, &buffer)); valueObj = makeByteBufferObject( env, buffer->data(), buffer->size()); |