Skip to content

Commit 135092c

Browse files
committed
[lldb-dap][test] Set disableASLR to False for tests
When running in constrained environments like docker, disable ASLR might fail with errors like: ``` AssertionError: False is not true : launch failed (Cannot launch '/__w/.../lldb-dap/stackTrace/subtleFrames/TestDAP_subtleFrames.test_subtleFrames/a.out': personality set failed: Operation not permitted) ``` E.g., llvm#110303 Hence we already run `settings set target.disable-aslr false` as part of the init-commands for the non-DAP tests (see llvm#88312 and https://discourse.llvm.org/t/running-lldb-in-a-container/76801). But we never adjusted it for the DAP tests. Hence we get conflicting test houtput like: ``` { "arguments": { "commandEscapePrefix": null, "disableASLR": true, .... "initCommands": [ ... "settings set target.disable-aslr false", ``` Disabling ASLR by default in tests isn't useulf (it's only really a debugging aid for users). So this patch sets `disableASLR=False` by default.
1 parent c47df3e commit 135092c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lldb/packages/Python/lldbsuite/test/tools/lldb-dap/lldbdap_testcase.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -367,7 +367,7 @@ def launch(
367367
cwd=None,
368368
env=None,
369369
stopOnEntry=False,
370-
disableASLR=True,
370+
disableASLR=False,
371371
disableSTDIO=False,
372372
shellExpandArguments=False,
373373
trace=False,

0 commit comments

Comments
 (0)