Invalidating parameter buffers for codec_get_version

Previously version_buf pointer is set to buffer pointer (data.buf)
of (IVIDDEC3_Status *) status.
The pointer dyn_params and status are not cache invalidated first which
can cause the status->data.buf to be accessed without cache invalidation.
Due to this corruption, it can cause exception on IPUMM.

The solution is to cache invalidate the dyn_params and status pointer
first to get the valid pointer of dyn_params and status.
Then set the version_buf with status->data.buf.
Perform cache invalidation on the version_buf to get the valid pointer.

Change-Id: I3258cca0ee191c488fc6457391215b8bfcde3b85
Signed-off-by: Subash Lakkimsetti <x0091084@ti.com>

1 files changed, 4 insertions, 2 deletions

diff --git a/src/ti/framework/dce/dce.c b/src/ti/framework/dce/dce.c
index 78848d4..f33b772 100644
--- a/src/ti/framework/dce/dce.c
+++ b/src/ti/framework/dce/dce.c
@@ -919,14 +919,16 @@ static int codec_get_version(UInt32 size, UInt32 *data)
         ERROR("invalid number of params sent");
         return (-1);
     }
+
+    dce_inv(dyn_params);
+    dce_inv(status);
+
     if( codec_id == OMAP_DCE_VIDDEC3 ) {
         version_buf = (void *)(H2P((MemHeader *)((IVIDDEC3_Status *)status)->data.buf));
     } else if( codec_id == OMAP_DCE_VIDENC2 ) {
         version_buf = (void *)(H2P((MemHeader *)((IVIDENC2_Status *)status)->data.buf));
     }
 
-    dce_inv(dyn_params);
-    dce_inv(status);
     dce_inv(version_buf);
 
     ivahd_acquire();