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

ls: ignore invalid block size from env vars

Browse source
This commit is contained in:
Daniel Hofstetter 2023-12-19 09:32:32 +01:00
parent 52af36d80b
commit a5606613bd
2 changed files with 16 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/uu/ls/src/ls.rs
View file

@ -840,9 +840,14 @@ impl Config {
match parse_size_u64(&raw_block_size.to_string_lossy()) {
Ok(size) => Some(size),
Err(_) => {
return Err(Box::new(LsError::BlockSizeParseError(
opt_block_size.unwrap().clone(),
)));
// only fail if invalid block size was specified with --block-size,
// ignore invalid block size from env vars
if let Some(invalid_block_size) = opt_block_size {
return Err(Box::new(LsError::BlockSizeParseError(
invalid_block_size.clone(),
)));
}
None
}
}
} else if env_var_posixly_correct.is_some() {

8
tests/by-util/test_ls.rs
View file

@ -3874,6 +3874,14 @@ fn test_ls_invalid_block_size() {
.stderr_is("ls: invalid --block-size argument 'invalid'\n");
}
// TODO ensure the correct block size is used when using -l because
// the output of "ls -l" and "BLOCK_SIZE=invalid ls -l" is different
#[test]
fn test_ls_invalid_block_size_in_env_var() {
new_ucmd!().env("LS_BLOCK_SIZE", "invalid").succeeds();
new_ucmd!().env("BLOCK_SIZE", "invalid").succeeds();
}
#[cfg(all(unix, feature = "dd"))]
#[test]
fn test_ls_block_size_override() {