diff options
| author | Leena Winterrowd <lenhardw@codeaurora.org> | 2015-09-10 15:41:08 -0700 |
|---|---|---|
| committer | Linux Build Service Account <lnxbuild@localhost> | 2015-10-06 03:25:07 -0600 |
| commit | 8f18da70cf611d6a2d066b0a7ef43ccc2a3d968f (patch) | |
| tree | 8403b1e950907c6844217fa8b3a417c74430e526 /media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp | |
| parent | 5b0ded9bf189d8e6a22ab1b22316302b1499bb53 (diff) | |
| download | frameworks_av-8f18da70cf611d6a2d066b0a7ef43ccc2a3d968f.zip frameworks_av-8f18da70cf611d6a2d066b0a7ef43ccc2a3d968f.tar.gz frameworks_av-8f18da70cf611d6a2d066b0a7ef43ccc2a3d968f.tar.bz2 | |
NuPlayerRenderer: Handle AMessage post error in openAudioSink
AMessage::postAndAwaitResponse can fail if the target looper no
longer exists or if allocation of the reply token fails. Check for
error in these cases to avoid deferencing a NULL response.
CRs-Fixed: 902117
Change-Id: I52d838e93768ba5b431a0ed8c74a67a5575f6685
Diffstat (limited to 'media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp')
| -rw-r--r-- | media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp index dd9ba6f..87e49bd 100644 --- a/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp +++ b/media/libmediaplayerservice/nuplayer/NuPlayerRenderer.cpp @@ -361,10 +361,10 @@ status_t NuPlayer::Renderer::openAudioSink( msg->setInt32("isStreaming", isStreaming); sp<AMessage> response; - msg->postAndAwaitResponse(&response); + status_t postStatus = msg->postAndAwaitResponse(&response); int32_t err; - if (!response->findInt32("err", &err)) { + if (postStatus != OK || !response->findInt32("err", &err)) { err = INVALID_OPERATION; } else if (err == OK && isOffloaded != NULL) { int32_t offload; |