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

am ad285430: Make stagefright a little less verbose. Fix heap corruption for real.

Browse Source 
Merge commit 'ad28543022101d0b01845e3f53dcd57e524816ef' into eclair-mr2-plus-aosp

* commit 'ad28543022101d0b01845e3f53dcd57e524816ef':
  Make stagefright a little less verbose. Fix heap corruption for real.
This commit is contained in:
Andreas Huber
2009-10-23 09:43:54 -07:00
committed by Android Git Automerger
parent 871e4ee81d ad28543022
commit f57b07dfa8
3 changed files with 9 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
media/libmediaplayerservice/StagefrightMetadataRetriever.cpp
View File

@@ -86,7 +86,7 @@ VideoFrame *StagefrightMetadataRetriever::captureFrame() {
LOGV("captureFrame");
if (mExtractor.get() == NULL) {
LOGE("no extractor.");
LOGV("no extractor.");
return NULL;
}
@@ -104,7 +104,7 @@ VideoFrame *StagefrightMetadataRetriever::captureFrame() {
}
if (i == n) {
LOGE("no video track found.");
LOGV("no video track found.");
return NULL;
}
@@ -114,7 +114,7 @@ VideoFrame *StagefrightMetadataRetriever::captureFrame() {
sp<MediaSource> source = mExtractor->getTrack(i);
if (source.get() == NULL) {
LOGE("unable to instantiate video track.");
LOGV("unable to instantiate video track.");
return NULL;
}
@@ -126,7 +126,7 @@ VideoFrame *StagefrightMetadataRetriever::captureFrame() {
NULL, OMXCodec::kPreferSoftwareCodecs);
if (decoder.get() == NULL) {
LOGE("unable to instantiate video decoder.");
LOGV("unable to instantiate video decoder.");
return NULL;
}
@@ -157,13 +157,13 @@ VideoFrame *StagefrightMetadataRetriever::captureFrame() {
if (err != OK) {
CHECK_EQ(buffer, NULL);
LOGE("decoding frame failed.");
LOGV("decoding frame failed.");
decoder->stop();
return NULL;
}
LOGI("successfully decoded video frame.");
LOGV("successfully decoded video frame.");
meta = decoder->getFormat();

2
media/libstagefright/MediaExtractor.cpp
View File

@@ -41,7 +41,7 @@ sp<MediaExtractor> MediaExtractor::Create(
}
mime = tmp.string();
LOGI("Autodetected media content as '%s' with confidence %.2f",
LOGV("Autodetected media content as '%s' with confidence %.2f",
mime, confidence);
}

7
media/libstagefright/omx/OMXNodeInstance.cpp
View File

@@ -107,8 +107,6 @@ static status_t StatusFromOMXError(OMX_ERRORTYPE err) {
}
status_t OMXNodeInstance::freeNode() {
Mutex::Autolock autoLock(mLock);
OMX_ERRORTYPE err = OMX_MasterFreeHandle(mHandle);
mHandle = NULL;
@@ -119,9 +117,8 @@ status_t OMXNodeInstance::freeNode() {
mOwner->invalidateNodeID(mNodeID);
mNodeID = NULL;
LOGI("OMXNodeInstance going away.");
mObserver.clear();
// delete this; // leads to heap-corruption???
LOGV("OMXNodeInstance going away.");
delete this;
return StatusFromOMXError(err);
}