RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-28 11:37:44 +00:00
Code Issues Activity

sort: prevent -o/--output to be specified multiple times

Browse source
This commit is contained in:
Dorian Peron 2025-05-25 23:36:36 +02:00
parent 04e7de1546
commit 3b4226c48c
2 changed files with 23 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -156,6 +156,9 @@ pub enum SortError {
#[error("{error}")] #[error("{error}")]
Uft8Error { error: Utf8Error }, Uft8Error { error: Utf8Error },
#[error("multiple output files specified")]
MultipleOutputFiles,
} }
impl UError for SortError { impl UError for SortError {
@ -1034,6 +1037,14 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
} }
}; };
// Prevent -o/--output to be specified multiple times
if matches
.get_occurrences::<String>(options::OUTPUT)
.is_some_and(|out| out.len() > 1)
{
return Err(SortError::MultipleOutputFiles.into());
}
settings.debug = matches.get_flag(options::DEBUG); settings.debug = matches.get_flag(options::DEBUG);
// check whether user specified a zero terminated list of files for input, otherwise read files from args // check whether user specified a zero terminated list of files for input, otherwise read files from args
@ -1427,7 +1438,9 @@ pub fn uu_app() -> Command {
.long(options::OUTPUT) .long(options::OUTPUT)
.help("write output to FILENAME instead of stdout") .help("write output to FILENAME instead of stdout")
.value_name("FILENAME") .value_name("FILENAME")
.value_hint(clap::ValueHint::FilePath), .value_hint(clap::ValueHint::FilePath)
// To detect multiple occurrences and raise an error
.action(ArgAction::Append),
) )
.arg( .arg(
Arg::new(options::REVERSE) Arg::new(options::REVERSE)

9
tests/by-util/test_sort.rs
View file

@ -1345,3 +1345,12 @@ fn test_failed_write_is_reported() {
.fails() .fails()
.stderr_is("sort: write failed: 'standard output': No space left on device\n"); .stderr_is("sort: write failed: 'standard output': No space left on device\n");
} }
#[test]
// Test for GNU tests/sort/sort.pl "o2"
fn test_multiple_output_files() {
new_ucmd!()
.args(&["-o", "foo", "-o", "bar"])
.fails_with_code(2)
.stderr_is("sort: multiple output files specified\n");
}