Wait for spawn child in order to avoit concurrency problems.

DataDog · Sep 26, 2024 · ba13c1c · ba13c1c
1 parent 1d90886
commit ba13c1c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/builder/build/arch/linux.rs b/builder/build/arch/linux.rs
@@ -58,10 +58,12 @@ pub fn strip_libraries(lib_path: &str) {
 }
 
 pub fn fix_soname(lib_path: &str) {
-    Command::new("patchelf")
+    let mut patch_soname = Command::new("patchelf")
         .arg("--set-soname")
         .arg(PROF_DYNAMIC_LIB)
         .arg(lib_path.to_owned() + "/" + PROF_DYNAMIC_LIB)
         .spawn()
-        .expect("failed to change the soname");
+        .expect("failed to span patchelf");
+
+    patch_soname.wait().expect("failed to change the soname");
 }
diff --git a/builder/build/arch/musl.rs b/builder/build/arch/musl.rs
@@ -58,10 +58,12 @@ pub fn strip_libraries(lib_path: &str) {
 }
 
 pub fn fix_soname(lib_path: &str) {
-    Command::new("patchelf")
+    let mut patch_soname = Command::new("patchelf")
         .arg("--set-soname")
         .arg(PROF_DYNAMIC_LIB)
         .arg(lib_path)
         .spawn()
-        .expect("failed to change the soname");
+        .expect("failed to spawn patchelf");
+
+    patch_soname.wait().expect("failed to change the soname");
 }