diff options
author | Deepak Kushwah <dkushwah@codeaurora.org> | 2016-01-07 18:34:24 +0530 |
---|---|---|
committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2016-04-26 15:14:40 -0700 |
commit | 73699c1931234002157fb99169694a43ce61924a (patch) | |
tree | 7aae8529a010c45bee2a9d727a8c80673a2ab25a | |
parent | 12a1b35118f86b54d1673aec86a987053c59dced (diff) | |
download | android_hardware_qcom_media-73699c1931234002157fb99169694a43ce61924a.tar.gz android_hardware_qcom_media-73699c1931234002157fb99169694a43ce61924a.tar.bz2 android_hardware_qcom_media-73699c1931234002157fb99169694a43ce61924a.zip |
mm-video-v4l2: Pointer null check before deferencing it
Pointer needs to be checked for sanity before dereferencing
it.
Change-Id: I4734b581cb6b506ebad9c99b8c9f407bccb4c3d1
-rw-r--r-- | mm-video-v4l2/vidc/venc/src/omx_video_base.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/mm-video-v4l2/vidc/venc/src/omx_video_base.cpp b/mm-video-v4l2/vidc/venc/src/omx_video_base.cpp index dbafdaeb..e1488502 100644 --- a/mm-video-v4l2/vidc/venc/src/omx_video_base.cpp +++ b/mm-video-v4l2/vidc/venc/src/omx_video_base.cpp @@ -3334,6 +3334,10 @@ OMX_ERRORTYPE omx_video::allocate_output_buffer( //secure handle fd struct native_handle_t* m_pOutput_pmem[i].buffer = malloc(sizeof(OMX_U32) + sizeof(native_handle_t*)); native_handle_t *handle = native_handle_create(1, 0); + if (!handle) { + DEBUG_PRINT_ERROR("ERROR: native handle creation failed"); + return OMX_ErrorInsufficientResources; + } handle->data[0] = m_pOutput_pmem[i].fd; char *data = (char*) m_pOutput_pmem[i].buffer; OMX_U32 type = 1; |