Fixed ffi_call return code type which was causing stack smashing error (

Xilinx#6959) * Fixed ffi_call return code type which was causing stack smashing error * Updated kernel_return conversion * refactor based on PR comment Co-authored-by: Jeff Lin <[email protected]>
pkcommando · Sep 23, 2022 · 69e5f32 · 69e5f32
1 parent c3a48cc
commit 69e5f32
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/runtime_src/core/edge/skd/xrt_skd.cpp b/src/runtime_src/core/edge/skd/xrt_skd.cpp
@@ -175,7 +175,7 @@ namespace xrt {
   XCL_DRIVER_DLLESPEC
   void
   skd::run() {
-    int32_t kernel_return = 0;
+    ffi_sarg kernel_return = 0;
     int ret = 0;
     void* ffi_arg_values[num_args];
     // Buffer Objects
@@ -221,7 +221,7 @@ namespace xrt {
       }
 
       ffi_call(&cif,FFI_FN(kernel), &kernel_return, ffi_arg_values);
-      args_from_host[return_offset] = (uint32_t)kernel_return;
+      args_from_host[return_offset] = (kernel_return >= 0) ? static_cast<uint32_t>(kernel_return) : 255; // Exit status out of range if returning negative value
 
       // Unmap Buffers
       for(auto i:bo_list) {