nice: -n huge_num true

Author: oech3

src/uu/nice/src/nice.rs

@@ -127,10 +127,26 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
             match nstr.parse::<i32>() {
                 Ok(num) => num,
                 Err(e) => {
-                    return Err(USimpleError::new(
-                        125,
-                        translate!("nice-error-invalid-number", "value" => nstr.clone(), "error" => e),
-                    ));
+                    // clamp huge values
+                    let trimmed = nstr.trim();
+                    let mut chars = trimmed.chars();
+                    let first = chars.next();
+                    let is_valid_number = (first.is_some_and(|c| c.is_ascii_digit())
+                        || first == Some('-')
+                        || first == Some('+'))
+                        && chars.all(|c| c.is_ascii_digit());
+                    if is_valid_number {
+                        if first == Some('-') {
+                            i32::MIN
+                        } else {
+                            i32::MAX
+                        }
+                    } else {
+                        return Err(USimpleError::new(
+                            125,
+                            translate!("nice-error-invalid-number", "value" => nstr.clone(), "error" => e),
+                        ));
+                    }
                 }
             }
         }

tests/by-util/test_nice.rs

@@ -90,3 +90,20 @@ fn test_trailing_empty_adjustment() {
         "error: The argument '--adjustment <adjustment>' requires a value but none was supplied",
     );
 }
+
+#[test]
+fn test_nice_huge() {
+    new_ucmd!()
+        .args(&[
+            "-n",
+            "99999999999999999999999999999999999999999999999999999999999999999999999999999999999999999",
+            "true",
+        ])
+        .succeeds()
+        .no_stdout();
+}
+
+#[test]
+fn test_nice_huge_negative() {
+    new_ucmd!().args(&["-n", "-9999999999", "true"]).succeeds(); // permission denied
+}