nice -n . 2>/dev/full does not abort

Author: oech3

src/uu/nice/src/nice.rs

@@ -8,14 +8,14 @@
 use clap::{Arg, ArgAction, Command};
 use libc::PRIO_PROCESS;
 use std::ffi::OsString;
-use std::io::{Error, ErrorKind, Write, stdout};
+use std::io::{Error, ErrorKind, Write, stderr, stdout};
 use std::num::IntErrorKind;
 use std::os::unix::process::CommandExt;
 use std::process;
 
 use uucore::translate;
 use uucore::{
-    error::{UResult, USimpleError, UUsageError, set_exit_code},
+    error::{UResult, USimpleError, set_exit_code},
     format_usage, show_error,
 };
 
@@ -121,10 +121,13 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
 
     let adjustment = if let Some(nstr) = matches.get_one::<String>(options::ADJUSTMENT) {
         if !matches.contains_id(options::COMMAND) {
-            return Err(UUsageError::new(
-                125,
-                translate!("nice-error-command-required-with-adjustment"),
-            ));
+            let _ = writeln!(
+                stderr(),
+                "nice: {}",
+                translate!("nice-error-command-required-with-adjustment")
+            );
+            set_exit_code(125);
+            return Ok(());
         }
         match nstr.parse::<i32>() {
             Ok(num) => num,
@@ -155,7 +158,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
     if unsafe { libc::setpriority(PRIO_PROCESS, 0, niceness) } == -1 {
         let warning_msg = translate!("nice-warning-setpriority", "util_name" => uucore::util_name(), "error" => Error::last_os_error());
 
-        if write!(std::io::stderr(), "{warning_msg}").is_err() {
+        if write!(stderr(), "{warning_msg}").is_err() {
             set_exit_code(125);
             return Ok(());
         }