diff options
| author | SeungBeom Kim <sbcrux.kim@samsung.com> | 2011-11-16 19:00:06 +0900 |
|---|---|---|
| committer | James Dong <jdong@google.com> | 2011-11-21 10:03:53 -0800 |
| commit | 41e03a94962b6aec7938762b0820928f90ff3502 (patch) | |
| tree | c4bb4af6049b7b944efe83c82d300105d9b4a5d6 /sec_mm/sec_omx | |
| parent | cba9655c68bc9c787333f63b18c564b70ffc40b2 (diff) | |
| download | device_samsung_crespo-41e03a94962b6aec7938762b0820928f90ff3502.zip device_samsung_crespo-41e03a94962b6aec7938762b0820928f90ff3502.tar.gz device_samsung_crespo-41e03a94962b6aec7938762b0820928f90ff3502.tar.bz2 | |
Patch for SEC-OMX error handling.
This patch makes the SEC-OMX terminate normally.
There will be no error messages(related to OMX) after applying this patch.
This patch solves the OMX abnormal termination (and mediaserver crash) issue.
Change-Id: I5010ffcfff6ff112c08f0a157ea290d691190134
Signed-off-by: SeungBeom Kim <sbcrux.kim@samsung.com>
related-to-bug: 5364248
Diffstat (limited to 'sec_mm/sec_omx')
| -rw-r--r-- | sec_mm/sec_omx/sec_omx_component/common/SEC_OMX_Basecomponent.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/sec_mm/sec_omx/sec_omx_component/common/SEC_OMX_Basecomponent.c b/sec_mm/sec_omx/sec_omx_component/common/SEC_OMX_Basecomponent.c index 170600a..89e9aeb 100644 --- a/sec_mm/sec_omx/sec_omx_component/common/SEC_OMX_Basecomponent.c +++ b/sec_mm/sec_omx/sec_omx_component/common/SEC_OMX_Basecomponent.c @@ -329,6 +329,10 @@ OMX_ERRORTYPE SEC_OMX_ComponentStateSet(OMX_COMPONENTTYPE *pOMXComponent, OMX_U3 case OMX_StateLoaded: for (i = 0; i < pSECComponent->portParam.nPorts; i++) { pSECPort = (pSECComponent->pSECPort + i); + if (pSECPort == NULL) { + ret = OMX_ErrorBadParameter; + goto EXIT; + } if (CHECK_PORT_TUNNELED(pSECPort) && CHECK_PORT_BUFFER_SUPPLIER(pSECPort)) { if (CHECK_PORT_ENABLED(pSECPort)) { ret = pSECComponent->sec_AllocateTunnelBuffer(pSECPort, i); |