[runvirt] reworked wrapper log handling

1 files changed, 13 insertions, 5 deletions

diff --git a/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/init_core.inc b/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/init_core.inc
index e551e5e8..b49488cb 100644
--- a/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/init_core.inc
+++ b/core/modules/run-virt/data/opt/openslx/vmchooser/run-virt-includes/init_core.inc
@@ -9,12 +9,20 @@ declare -rg TMPDIR="/tmp/virt/${USER}/$$"
 
 # Check if the path to the logfile was given externally (by the wrapper)
 unset WRAPPED
-if isset LOGFILE; then
-	# given by wrapper
-	declare -rg WRAPPED=aye
-else
-	declare -rg LOGFILE="/var/log/openslx/run-virt.${USER}.$$.log"
+if notempty LOGFILE; then
+	if [ ! -w "${LOGFILE}" ]; then
+		# given logfile is not writeable or non-existant
+		# so handle it ourself (so do not set WRAPPED)
+		rm -f "${LOGFILE}"
+		unset LOGFILE
+	else
+		# given logfile was ok, mark that we are wrapped
+		# (for cleanexit to not slxlog the log since
+		# the wrapper is handling that for us)
+		declare -rg WRAPPED=aye
+	fi
 fi
+isset LOGFILE || declare -rg LOGFILE="/var/log/openslx/run-virt.${USER}.$$.log"
 
 # Create temporary directory for current invocation
 if check_dep mkdir && ! mkdir -p "$TMPDIR"; then