RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-29 12:07:46 +00:00
Code Issues Activity

Merge pull request #3206 from 353fc443/timeout-empty-string

Browse source 
timeout: avoid panicking for empty string
This commit is contained in:
Sylvestre Ledru 2022-03-06 23:02:46 +01:00 • committed by GitHub
commit 9113594951
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

15
src/uu/timeout/src/timeout.rs
View file

@ -51,7 +51,7 @@ struct Config {
}
impl Config {
fn from(options: &clap::ArgMatches) -> Self {
fn from(options: &clap::ArgMatches) -> UResult<Self> {
let signal = match options.value_of(options::SIGNAL) {
Some(signal_) => {
let signal_result = signal_by_name_or_value(signal_);
@ -69,8 +69,11 @@ impl Config {
.value_of(options::KILL_AFTER)
.map(|time| uucore::parse_time::from_str(time).unwrap());
let duration: Duration =
uucore::parse_time::from_str(options.value_of(options::DURATION).unwrap()).unwrap();
let duration =
match uucore::parse_time::from_str(options.value_of(options::DURATION).unwrap()) {
Ok(duration) => duration,
Err(err) => return Err(USimpleError::new(1, err)),
};
let preserve_status: bool = options.is_present(options::PRESERVE_STATUS);
let foreground = options.is_present(options::FOREGROUND);
@ -82,7 +85,7 @@ impl Config {
.map(String::from)
.collect::<Vec<_>>();
Self {
Ok(Self {
foreground,
kill_after,
signal,
@ -90,7 +93,7 @@ impl Config {
preserve_status,
verbose,
command,
}
})
}
}
@ -104,7 +107,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
let matches = app.get_matches_from(args);
let config = Config::from(&matches);
let config = Config::from(&matches)?;
timeout(
&config.command,
config.duration,