[compiler-rt] a305d25 - asan_device_setup's wrapper scripts not handling args with spaces correctly

Evgenii Stepanov via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 12:55:49 PDT 2020


Author: Andrew Grieve
Date: 2020-07-22T12:55:28-07:00
New Revision: a305d2502948697f21cea0fd152566e473f88a6a

URL: https://github.com/llvm/llvm-project/commit/a305d2502948697f21cea0fd152566e473f88a6a
DIFF: https://github.com/llvm/llvm-project/commit/a305d2502948697f21cea0fd152566e473f88a6a.diff

LOG: asan_device_setup's wrapper scripts not handling args with spaces correctly

Summary: Came up in Chromium: https://bugs.chromium.org/p/chromium/issues/detail?id=1103108#c21

Reviewers: eugenis

Reviewed By: eugenis

Subscribers: #sanitizers

Tags: #sanitizers

Differential Revision: https://reviews.llvm.org/D84237

Added: 
    

Modified: 
    compiler-rt/lib/asan/scripts/asan_device_setup

Removed: 
    


################################################################################
diff  --git a/compiler-rt/lib/asan/scripts/asan_device_setup b/compiler-rt/lib/asan/scripts/asan_device_setup
index 041bf92e6253..27fedea3f863 100755
--- a/compiler-rt/lib/asan/scripts/asan_device_setup
+++ b/compiler-rt/lib/asan/scripts/asan_device_setup
@@ -330,7 +330,7 @@ function generate_zygote_wrapper { # from, to
 ASAN_OPTIONS=$ASAN_OPTIONS \\
 ASAN_ACTIVATION_OPTIONS=include_if_exists=/data/local/tmp/asan.options.%b \\
 LD_PRELOAD=$_ld_preload \\
-exec $_to \$@
+exec $_to "\$@"
 
 EOF
 }


        


More information about the llvm-commits mailing list