From: David Goulet <dgoulet@efficios.com>
Date: Mon, 10 Dec 2012 23:24:42 +0000 (-0500)
Subject: Fix: overlap bash escaping for wildcard event name
X-Git-Tag: v2.1.0~65
X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=commitdiff_plain;h=907b42de38da8455f22f2b4e03fbc8e75ecbd38a

Fix: overlap bash escaping for wildcard event name

Signed-off-by: David Goulet <dgoulet@efficios.com>
---

diff --git a/tests/ust/overlap/overlap.sh b/tests/ust/overlap/overlap.sh
index 6baf7c878..ebe6cfd1a 100755
--- a/tests/ust/overlap/overlap.sh
+++ b/tests/ust/overlap/overlap.sh
@@ -43,8 +43,6 @@ fi
 
 run_demo_app()
 {
-	local dir=`pwd`
-
 	cd $CURDIR/demo
 
 	# Start test
@@ -59,25 +57,25 @@ run_demo_app()
 # Ease our life a bit ;)
 trace_match_demo1_events()
 {
-	trace_matches $DEMO_EVENT1 $NUM_DEMO1_EVENT $TRACE_PATH
-	trace_matches $DEMO_EVENT1_2 $NUM_DEMO1_EVENT $TRACE_PATH
+	trace_matches "$DEMO_EVENT1" $NUM_DEMO1_EVENT $TRACE_PATH
+	trace_matches "$DEMO_EVENT1_2" $NUM_DEMO1_EVENT $TRACE_PATH
 }
 
 # Ease our life a bit ;)
 trace_match_all_demo_events()
 {
 	trace_match_demo1_events
-	trace_matches $DEMO_EVENT2 $NUM_DEMO2_EVENT $TRACE_PATH
-	trace_matches $DEMO_EVENT3 $NUM_DEMO3_EVENT $TRACE_PATH
+	trace_matches "$DEMO_EVENT2" $NUM_DEMO2_EVENT $TRACE_PATH
+	trace_matches "$DEMO_EVENT3" $NUM_DEMO3_EVENT $TRACE_PATH
 }
 
 # Ease our life a bit ;)
 trace_match_no_demo_events()
 {
-	trace_matches $DEMO_EVENT1 0 $TRACE_PATH
-	trace_matches $DEMO_EVENT1_2 0 $TRACE_PATH
-	trace_matches $DEMO_EVENT2 0 $TRACE_PATH
-	trace_matches $DEMO_EVENT3 0 $TRACE_PATH
+	trace_matches "$DEMO_EVENT1" 0 $TRACE_PATH
+	trace_matches "$DEMO_EVENT1_2" 0 $TRACE_PATH
+	trace_matches "$DEMO_EVENT2" 0 $TRACE_PATH
+	trace_matches "$DEMO_EVENT3" 0 $TRACE_PATH
 }
 
 # Expect all "demo" events, no duplicate.
@@ -89,8 +87,8 @@ test_enable_simple_wildcard()
 	echo ""
 	echo "=== Simple wildcard overlap"
 
-	enable_ust_lttng_event $SESSION_NAME $event_wild1
-	enable_ust_lttng_event $SESSION_NAME $event_wild2
+	enable_ust_lttng_event $SESSION_NAME "$event_wild1"
+	enable_ust_lttng_event $SESSION_NAME "$event_wild2"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -112,8 +110,8 @@ test_enable_wildcard_filter()
 	echo ""
 	echo "=== Wildcard overlap with filter"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==0"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -134,8 +132,8 @@ test_enable_wildcard_filter_2()
 	echo ""
 	echo "=== Wildcard overlap with filter 2"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==0"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -156,8 +154,8 @@ test_enable_wildcard_filter_3()
 	echo ""
 	echo "=== Wildcard overlap with filter 3"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -178,8 +176,8 @@ test_enable_wildcard_filter_4()
 	echo ""
 	echo "=== Wildcard overlap with filter 4"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==0"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==0"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -200,8 +198,8 @@ test_enable_wildcard_filter_5()
 	echo ""
 	echo "=== Wildcard overlap with filter 5"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==0"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -222,8 +220,8 @@ test_enable_wildcard_filter_6()
 	echo ""
 	echo "=== Wildcard overlap with filter 6"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==0"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -247,8 +245,8 @@ test_enable_wildcard_filter_7()
 	echo ""
 	echo "=== Wildcard overlap with filter 7"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -269,8 +267,8 @@ test_enable_wildcard_filter_8()
 	echo ""
 	echo "=== Wildcard overlap with filter 8"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==0"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==0"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -291,8 +289,8 @@ test_enable_same_wildcard_filter()
 	echo ""
 	echo "=== Same wildcard overlap with filter"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1&&1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1&&1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -313,8 +311,8 @@ test_enable_same_wildcard_filter_2()
 	echo ""
 	echo "=== Same wildcard overlap with filter 2"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 	if [ $? -eq 1 ]; then
 		echo -n "FAIL is normal. Same event with same filter is denied by the sessiond "
 		print_ok
@@ -341,8 +339,8 @@ test_enable_same_wildcard_filter_3()
 	echo ""
 	echo "=== Same wildcard overlap with filter 3"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==0"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -363,8 +361,8 @@ test_enable_same_wildcard_filter_4()
 	echo ""
 	echo "=== Same wildcard overlap with filter 4"
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==0&&1==0"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==0&&1==0"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==0"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -385,8 +383,8 @@ test_enable_same_event_filter()
 	echo ""
 	echo "=== Enable same event with filter."
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1&&1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1&&1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 
 	disable_ust_lttng_event $SESSION_NAME "ust*"
 
@@ -412,8 +410,8 @@ test_disable_same_wildcard_filter()
 	echo ""
 	echo "=== Disable same wildcard with filter."
 
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild1 "1==1&&1==1"
-	enable_ust_lttng_event_filter $SESSION_NAME $event_wild2 "1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild1" "1==1&&1==1"
+	enable_ust_lttng_event_filter $SESSION_NAME "$event_wild2" "1==1"
 
 	disable_ust_lttng_event $SESSION_NAME "ust*"
 
@@ -438,9 +436,9 @@ test_enable_bad_wildcard()
 	echo ""
 	echo "=== Enable bad wildcard"
 
-	enable_ust_lttng_event $SESSION_NAME $event_wild1
-	enable_ust_lttng_event $SESSION_NAME $event_wild2
-	enable_ust_lttng_event $SESSION_NAME $event_wild3
+	enable_ust_lttng_event $SESSION_NAME "$event_wild1"
+	enable_ust_lttng_event $SESSION_NAME "$event_wild2"
+	enable_ust_lttng_event $SESSION_NAME "$event_wild3"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -461,8 +459,8 @@ test_enable_simple_wildcard_2()
 	echo ""
 	echo "=== Simple wildcard 2"
 
-	enable_ust_lttng_event $SESSION_NAME $event_wild1
-	enable_ust_lttng_event $SESSION_NAME $event_wild2
+	enable_ust_lttng_event $SESSION_NAME "$event_wild1"
+	enable_ust_lttng_event $SESSION_NAME "$event_wild2"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -596,11 +594,11 @@ test_disable_simple_wildcard()
 	echo ""
 	echo "=== Disable simple wildcard"
 
-	enable_ust_lttng_event $SESSION_NAME $event_wild1
-	enable_ust_lttng_event $SESSION_NAME $event_wild2
+	enable_ust_lttng_event $SESSION_NAME "$event_wild1"
+	enable_ust_lttng_event $SESSION_NAME "$event_wild2"
 
-	disable_ust_lttng_event $SESSION_NAME $event_wild1
-	disable_ust_lttng_event $SESSION_NAME $event_wild2
+	disable_ust_lttng_event $SESSION_NAME "$event_wild1"
+	disable_ust_lttng_event $SESSION_NAME "$event_wild2"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -622,10 +620,10 @@ test_disable_wildcard_overlap()
 	echo ""
 	echo "=== Disable wildcard overlap"
 
-	enable_ust_lttng_event $SESSION_NAME $event_wild1
-	enable_ust_lttng_event $SESSION_NAME $event_wild2
+	enable_ust_lttng_event $SESSION_NAME "$event_wild1"
+	enable_ust_lttng_event $SESSION_NAME "$event_wild2"
 
-	disable_ust_lttng_event $SESSION_NAME $event_wild1
+	disable_ust_lttng_event $SESSION_NAME "$event_wild1"
 
 	start_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
@@ -634,10 +632,10 @@ test_disable_wildcard_overlap()
 	stop_lttng_tracing $SESSION_NAME >/dev/null 2>&1
 
 	# Expect only "ust_tests_demo" events.
-	trace_matches $DEMO_EVENT1 $NUM_DEMO1_EVENT $TRACE_PATH
-	trace_matches $DEMO_EVENT1_2 0 $TRACE_PATH
-	trace_matches $DEMO_EVENT2 0 $TRACE_PATH
-	trace_matches $DEMO_EVENT3 0 $TRACE_PATH
+	trace_matches "$DEMO_EVENT1" $NUM_DEMO1_EVENT $TRACE_PATH
+	trace_matches "$DEMO_EVENT1_2" 0 $TRACE_PATH
+	trace_matches "$DEMO_EVENT2" 0 $TRACE_PATH
+	trace_matches "$DEMO_EVENT3" 0 $TRACE_PATH
 	return $?
 }
 
diff --git a/tests/utils.sh b/tests/utils.sh
index 23b2dfd07..80e384281 100644
--- a/tests/utils.sh
+++ b/tests/utils.sh
@@ -119,7 +119,7 @@ function lttng_enable_kernel_event
 	fi
 
 	echo -n "Enabling kernel event $event_name for session $sess_name"
-	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event $event_name -s $sess_name -k >/dev/null 2>&1
+	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event "$event_name" -s $sess_name -k >/dev/null 2>&1
 	if [ $? -eq 1 ]; then
 		print_fail
 		return 1
@@ -283,7 +283,7 @@ function enable_ust_lttng_event_filter()
 	filter="$3"
 	echo -n "Enabling lttng event with filtering "
 
-	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event $event_name -s $sess_name -u --filter "$filter" 2>&1 >/dev/null
+	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event "$event_name" -s $sess_name -u --filter "$filter" 2>&1 >/dev/null
 	if [ $? -eq 0 ]; then
 		print_ok
 		return 0
@@ -300,7 +300,7 @@ function enable_ust_lttng_event_loglevel()
 	loglevel="$3"
 	echo -n "Enabling lttng event $event_name with loglevel $loglevel"
 
-	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event $event_name -s $sess_name -u --loglevel $loglevel 2>&1 >/dev/null
+	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event "$event_name" -s $sess_name -u --loglevel $loglevel 2>&1 >/dev/null
 	if [ $? -eq 0 ]; then
 		print_ok
 		return 0
@@ -317,7 +317,7 @@ function enable_ust_lttng_event_loglevel_only()
 	loglevel="$3"
 	echo -n "Enabling lttng event $event_name with loglevel-only $loglevel"
 
-	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event $event_name -s $sess_name -u --loglevel-only $loglevel 2>&1 >/dev/null
+	$TESTDIR/../src/bin/lttng/$LTTNG_BIN enable-event "$event_name" -s $sess_name -u --loglevel-only $loglevel 2>&1 >/dev/null
 	if [ $? -eq 0 ]; then
 		print_ok
 		return 0
@@ -329,11 +329,11 @@ function enable_ust_lttng_event_loglevel_only()
 
 function disable_ust_lttng_event ()
 {
-	sess_name=$1
-	event_name=$2
+	sess_name="$1"
+	event_name="$2"
 
 	echo -n "Disabling lttng event $event_name for session $sess_name "
-	$TESTDIR/../src/bin/lttng/$LTTNG_BIN disable-event $event_name -s $sess_name -u >/dev/null 2>&1
+	$TESTDIR/../src/bin/lttng/$LTTNG_BIN disable-event "$event_name" -s $sess_name -u >/dev/null 2>&1
 	if [ $? -eq 1 ]; then
 		print_fail
 		return 1