diff options
author | Gu, Wangyi <wangyi.gu@intel.com> | 2013-02-05 14:10:14 +0800 |
---|---|---|
committer | Patrick Tjin <pattjin@google.com> | 2014-07-21 22:03:40 -0700 |
commit | 274e417839e9335eccada4b08e082e238b0ed7dc (patch) | |
tree | cbd99130d5e6930061cda3001e5dc7e325c2daf9 /base | |
parent | f94dc0afb6e4af367787f758a77e510c0bfe6b8a (diff) | |
download | wrs_omxil_core-274e417839e9335eccada4b08e082e238b0ed7dc.tar.gz |
[PORT FROM R4.1] Enhance the error handle mechanism in OMX Core to avoid video editor carsh
BZ: 75145
1. the dlerror will return the error caused by the main thread, this will lead the
ret equals -1, and this will make the loop not end in CModule destructor;
2. remove the redundant pthread_mutex_unlock;
3. add some error logs to expose some corner cases
Change-Id: Ifab1bb704ae1a65553be1326716558a4e013fada
Signed-off-by: Gu, Wangyi <wangyi.gu@intel.com>
Reviewed-on: http://android.intel.com:8080/90189
Reviewed-by: cactus <cactus@intel.com>
Reviewed-by: Wang, Yi A <yi.a.wang@intel.com>
Reviewed-by: Shi, PingX <pingx.shi@intel.com>
Tested-by: Shi, PingX <pingx.shi@intel.com>
Reviewed-by: Gozalvez Herrero, Juan AntonioX <juan.antoniox.gozalvez.herrero@intel.com>
Tested-by: Gozalvez Herrero, Juan AntonioX <juan.antoniox.gozalvez.herrero@intel.com>
Diffstat (limited to 'base')
-rw-r--r-- | base/src/cmodule.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/base/src/cmodule.cpp b/base/src/cmodule.cpp index ee5b93d..23e4718 100644 --- a/base/src/cmodule.cpp +++ b/base/src/cmodule.cpp @@ -90,6 +90,8 @@ OMX_ERRORTYPE CModule::Load(int flag) return OMX_ErrorInvalidComponent; } + if (module) + LOGE("module %s will be overwrite",module->name); module = m; LOGI("module %s successfully loaded\n", lname); |