From mboxrd@z Thu Jan  1 00:00:00 1970
From: <bogus@does.not.exist.com>
Date: Fri, 23 Jan 2009 10:48:37 -0000
Subject: No subject
Message-ID: <mailman.258.1308611100.8188.u-boot@lists.denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

run_command() or parse_string_outer() depending on whether or not hush
is enabled. Should the simple/hush command execution always be safe with
a command like this?

> > +	printf("running: %s\n", dupcmd);
> 
> This should probably be a debug() ?

I don't think so - it's the one place a user watching the screen would
get to see what is being executed, which might be very useful to "end
users" who aren't eager to recompile to get some verbosity.

Thanks,
Jason