From mboxrd@z Thu Jan  1 00:00:00 1970
From: compudj@krystal.dyndns.org (Mathieu Desnoyers)
Date: Wed, 16 Nov 2011 07:17:46 -0500
Subject: [ltt-dev] [PATCH] usttrace: allow spaces in
	command-line	arguments
In-Reply-To: <1321437618-1416-1-git-send-email-stefanha@linux.vnet.ibm.com>
References: <1321437618-1416-1-git-send-email-stefanha@linux.vnet.ibm.com>
Message-ID: <20111116121745.GA15398@Krystal>

* Stefan Hajnoczi (stefanha at linux.vnet.ibm.com) wrote:
> The usttrace uses $*, which expands arguments without preserving their
> boundaries.  This causes arguments that include spaces to be mangled:
> 
> [0] "echo"
> [1] "hello world"
> 
> becomes:
> 
> [0] "echo"
> [1] "hello"
> [2] "world"
> 
> Use "$@" instead so that argument boundaries are preserved.

Merged into UST 0.x.

Thanks,

Mathieu

> 
> Signed-off-by: Stefan Hajnoczi <stefanha at linux.vnet.ibm.com>
> ---
>  usttrace |    7 ++-----
>  1 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/usttrace b/usttrace
> index e76dae7..d1a6e5f 100755
> --- a/usttrace
> +++ b/usttrace
> @@ -85,16 +85,13 @@ if [ -n "$where" ]; then
>  	exit 0
>  fi
>  
> -# Prepare vars
> -CMD=$*
> -
>  # Validate input
>  if [ -z "$HOME" ];
>  then
>  	error "no home specified"
>  fi
>  
> -if [ -z "$CMD" ];
> +if [ -z "$*" ];
>  then
>  	error "no command specified"
>  	usage;
> @@ -196,7 +193,7 @@ fi
>      fi
>  
>  # Execute the command
> -    $CMD 2>&1
> +    "$@" 2>&1
>  ) | tee "$OUTDIR/app.log"
>  
>  ## Because of the keepalive mechanism, we're sure that by the time
> -- 
> 1.7.7.1
> 
> 
> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com