[Openmp-commits] [PATCH] D72637: openmp: fix memcpy memory leak

[Johannes Doerfert via Phabricator via Openmp-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rGa23d7282ca71: openmp: fix memcpy memory leak (authored by trws, committed by jdoerfert).
Herald added a project: OpenMP.

Changed prior to commit:
  https://reviews.llvm.org/D72637?vs=237744&id=250124#toc

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72637/new/

https://reviews.llvm.org/D72637

Files:
  openmp/libomptarget/src/api.cpp


Index: openmp/libomptarget/src/api.cpp
===================================================================
--- openmp/libomptarget/src/api.cpp
+++ openmp/libomptarget/src/api.cpp
@@ -174,6 +174,7 @@
     rc = SrcDev.data_retrieve(buffer, srcAddr, length);
     if (rc == OFFLOAD_SUCCESS)
       rc = DstDev.data_submit(dstAddr, buffer, length);
+    free(buffer);
   }
 
   DP("omp_target_memcpy returns %d\n", rc);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72637.250124.patch
Type: text/x-patch
Size: 416 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20200313/7dfbd12b/attachment.bin>