diff options
author | Arkadiusz Hiler <arek@hiler.eu> | 2018-09-30 15:11:24 +0300 |
---|---|---|
committer | Arkadiusz Hiler <arek@hiler.eu> | 2018-09-30 15:37:01 +0300 |
commit | eed0bc3ebd15bf94a28ca019ab656a3a0bc0d258 (patch) | |
tree | 63e09dad69399fe4470830b3103301a97ce0ab7f /sway/main.c | |
parent | f1dbdce0b2cfb6a71f972dd58180337c8ad27096 (diff) |
Add support for installing binaries with DT_RPATH
It's better to use DT_RPATH dynamic section of the elf binary to store
the paths of libraries to load instead of overwriting LD_LIBRARY_PATH
for the whole environment, causing surprises. This solution is much more
transparent and perfectly suitable for running contained installations
of wayland/wlroots/sway.
The code unsetting the LD_LIBRARY_PATH/LD_PRELOAD was also deleted as
it's a placebo security at best - we should trust the execution path
that leads us to running sway, and it's way too late to care about those
variables since we already started executing our compositor, thus we
would be compromised anyway.
Diffstat (limited to 'sway/main.c')
-rw-r--r-- | sway/main.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/sway/main.c b/sway/main.c index 50b05b21..dea4a31c 100644 --- a/sway/main.c +++ b/sway/main.c @@ -241,14 +241,6 @@ int main(int argc, char **argv) { " --get-socketpath Gets the IPC socket path and prints it, then exits.\n" "\n"; - // Security: - unsetenv("LD_PRELOAD"); -#ifdef _LD_LIBRARY_PATH - setenv("LD_LIBRARY_PATH", _LD_LIBRARY_PATH, 1); -#else - unsetenv("LD_LIBRARY_PATH"); -#endif - int c; while (1) { int option_index = 0; |