diff options
author | c_sridur <sridur@codeaurora.org> | 2015-04-09 17:14:48 +0530 |
---|---|---|
committer | Arne Coucheron <arco68@gmail.com> | 2015-04-22 00:38:52 +0200 |
commit | 4ce4595c9bf29b11eb89ef259666e9297923da9e (patch) | |
tree | cea8775854fd838b4f40d1798b22ad472822dff8 | |
parent | 73af4b5b0f0a93589c9d2dbb85b7cbe70df6d750 (diff) | |
download | android_hardware_qcom_media-stable/cm-12.1-caf-8960-YOG7D.tar.gz android_hardware_qcom_media-stable/cm-12.1-caf-8960-YOG7D.tar.bz2 android_hardware_qcom_media-stable/cm-12.1-caf-8960-YOG7D.zip |
mm-video: venc: Fix reconbuffer free crash.stable/cm-12.1-caf-8960-YOG7Dstable/cm-12.1-caf-8960-YOG4Pstable/cm-12.1-caf-8960-YOG3C
- ION_IOC_FREE for reconbuffer is called with invalid fd -1.
Modified code to check proper fd value before freeing.
Change-Id: I669d545bbdd9fe894a66c0e6f0279692c1460980
-rw-r--r-- | mm-video/vidc/venc/src/video_encoder_device.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/mm-video/vidc/venc/src/video_encoder_device.cpp b/mm-video/vidc/venc/src/video_encoder_device.cpp index 96facccf..d8f6f942 100644 --- a/mm-video/vidc/venc/src/video_encoder_device.cpp +++ b/mm-video/vidc/venc/src/video_encoder_device.cpp @@ -1,5 +1,5 @@ /*-------------------------------------------------------------------------- -Copyright (c) 2010-2013, The Linux Foundation. All rights reserved. +Copyright (c) 2010-2013,2015 The Linux Foundation. All rights reserved. Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following conditions are met: @@ -1554,7 +1554,7 @@ OMX_U32 venc_dev::pmem_free() struct venc_recon_addr recon_addr; for (cnt = 0; cnt < recon_buffers_count; cnt++) { - if(recon_buff[cnt].pmem_fd) + if(recon_buff[cnt].pmem_fd > 0) { recon_addr.pbuffer = recon_buff[cnt].virtual_address; recon_addr.offset = recon_buff[cnt].offset; |