mm-video:vdec: Fix random ION memory call failures

c_sridur · khusika · commit 88aaf209947d · 2015-04-30T01:09:18.000+07:00
- Seeing random ION_FREE() and ION_MAP failures during cts
    if both encoder and decoder instances are running parallelly
    since duplicate fd is given for both encoder and decoder.
    use dup(fd) in platform private VideoHeap data to avoid fd leak.

Change-Id: I3c88c165e50e36edf6d2390cdae888eacc2b5272
diff --git a/mm-video/vidc/vdec/src/omx_vdec.cpp b/mm-video/vidc/vdec/src/omx_vdec.cpp
@@ -476,7 +476,7 @@ VideoHeap::VideoHeap(int devicefd, size_t size, void* base,
 {
     m_ion_device_fd = devicefd;
     m_ion_handle = handle;
-    MemoryHeapBase::init(ionMapfd, base, size, 0, MEM_DEVICE);
+    MemoryHeapBase::init(dup(ionMapfd), base, size, 0, MEM_DEVICE);
     //ionInit(devicefd, base, size, 0 , MEM_DEVICE,handle,ionMapfd);
 }
 #else

Original file line number	Diff line number	Diff line change
`@@ -476,7 +476,7 @@ VideoHeap::VideoHeap(int devicefd, size_t size, void* base,`
`476`	`476`	`{`
`477`	`477`	`m_ion_device_fd = devicefd;`
`478`	`478`	`m_ion_handle = handle;`
`479`		`- MemoryHeapBase::init(ionMapfd, base, size, 0, MEM_DEVICE);`
	`479`	`+ MemoryHeapBase::init(dup(ionMapfd), base, size, 0, MEM_DEVICE);`
`480`	`480`	`//ionInit(devicefd, base, size, 0 , MEM_DEVICE,handle,ionMapfd);`
`481`	`481`	`}`
`482`	`482`	`#else`