david/ipxe
david
/
ipxe
Archived
1
0
Fork 0
Code

[cmdline] Fix up "sleep" argument parsing 

Use parse_integer() rather than strtoul() to allow parsing errors to
be reported in a meaningful way.

Signed-off-by: Michael Brown <mcb30@ipxe.org>
This commit is contained in:
Michael Brown 2011-10-24 15:52:14 +01:00
parent bf2da3122b
commit f177a6f09f
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/core/exec.c
View File

@ -548,17 +548,25 @@ static struct command_descriptor sleep_cmd =
*/
static int sleep_exec ( int argc, char **argv ) {
struct sleep_options opts;
unsigned long start, delay;
unsigned int seconds;
unsigned long start;
unsigned long delay;
int rc;
/* Parse options */
if ( ( rc = parse_options ( argc, argv, &sleep_cmd, &opts ) ) != 0 )
return rc;
/* Parse number of seconds */
if ( ( rc = parse_integer ( argv[optind], &seconds ) ) != 0 )
return rc;
/* Delay for specified number of seconds */
start = currticks();
delay = strtoul ( argv[1], NULL, 0 ) * ticks_per_sec();
delay = ( seconds * TICKS_PER_SEC );
while ( ( currticks() - start ) <= delay )
cpu_nap();
return 0;
}