[banshee/stable-1.6] Fix insecure LD_LIBRARY_PATH (bnc#642505)



commit 456d7e961ce3a7781e44ea20d539cd5c157d7982
Author: Aaron Bockover <abockover novell com>
Date:   Wed Oct 20 12:22:40 2010 -0400

    Fix insecure LD_LIBRARY_PATH (bnc#642505)
    
    A vulnerability existed where if LD_LIBRARY_PATH were set but empty, a
    trailing : as a path separator would still be appended to the path,
    exposing an insecure/invalid search path. GST_PLUGINS_PATH was similarly
    vulnerable.
    
    Using :+: instead of +: prevents this as ${X:+:$X} returns X iff X is
    set and not empty whereas ${X+:$X} returns X iff X is set (it may be
    empty).

 src/Clients/Booter/banshee-1.linux.in |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/src/Clients/Booter/banshee-1.linux.in b/src/Clients/Booter/banshee-1.linux.in
index af85d8f..a62d95f 100644
--- a/src/Clients/Booter/banshee-1.linux.in
+++ b/src/Clients/Booter/banshee-1.linux.in
@@ -7,8 +7,8 @@ MONO_EXE="@expanded_libdir@/@PACKAGE@/$exec_asm"
 BANSHEE_EXEC_NAME=$(basename $0)
 BANSHEE_CONFIG_DIR="${XDG_CONFIG_HOME:-$HOME/.config}/banshee-1"
 
-export LD_LIBRARY_PATH= expanded_libdir@:@expanded_libdir@/@PACKAGE@:@expanded_libdir@/@PACKAGE@/Extensions:@expanded_libdir@/@PACKAGE@/Backends${LD_LIBRARY_PATH+:$LD_LIBRARY_PATH}
-export GST_PLUGIN_PATH= expanded_libdir@/@PACKAGE@/gstreamer-0.10${GST_PLUGIN_PATH+:$GST_PLUGIN_PATH}
+export LD_LIBRARY_PATH= expanded_libdir@:@expanded_libdir@/@PACKAGE@:@expanded_libdir@/@PACKAGE@/Extensions:@expanded_libdir@/@PACKAGE@/Backends${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH}
+export GST_PLUGIN_PATH= expanded_libdir@/@PACKAGE@/gstreamer-0.10${GST_PLUGIN_PATH:+:$GST_PLUGIN_PATH}
 if [ $BANSHEE_EXEC_NAME = "muinshee" ]; then
     BANSHEE_CLIENT="Muinshee"
     export MONO_PATH= expanded_libdir@/@PACKAGE@/Extensions



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]