Merge pull request #2675 from blyxxyz/whoami-cleanup

whoami: Cleanup

Cargo.lock

@@ -3222,6 +3222,7 @@ name = "uu_whoami"
 version = "0.0.7"
 dependencies = [
  "clap",
+ "libc",
  "uucore",
  "uucore_procs",
  "winapi 0.3.9",

src/uu/whoami/Cargo.toml

@@ -16,12 +16,15 @@ path = "src/whoami.rs"
 
 [dependencies]
 clap = { version = "2.33", features = ["wrap_help"] }
-uucore = { version=">=0.0.9", package="uucore", path="../../uucore", features=["entries", "wide"] }
+uucore = { version=">=0.0.9", package="uucore", path="../../uucore", features=["entries"] }
 uucore_procs = { version=">=0.0.6", package="uucore_procs", path="../../uucore_procs" }
 
 [target.'cfg(target_os = "windows")'.dependencies]
 winapi = { version = "0.3", features = ["lmcons"] }
 
+[target.'cfg(unix)'.dependencies]
+libc = "0.2.42"
+
 [[bin]]
 name = "whoami"
 path = "src/main.rs"

src/uu/whoami/src/platform/unix.rs

@@ -8,14 +8,14 @@
  * file that was distributed with this source code.
  */
 
-// spell-checker:ignore (ToDO) getusername
+use std::ffi::OsString;
+use std::io;
 
-use std::io::Result;
 use uucore::entries::uid2usr;
-use uucore::libc::geteuid;
 
-pub unsafe fn get_username() -> Result<String> {
+pub fn get_username() -> io::Result<OsString> {
-    // Get effective user id
+    // SAFETY: getuid() does nothing with memory and is always successful.
-    let uid = geteuid();
+    let uid = unsafe { libc::geteuid() };
-    uid2usr(uid)
+    // uid2usr should arguably return an OsString but currently doesn't
+    uid2usr(uid).map(Into::into)
 }

src/uu/whoami/src/platform/windows.rs

@@ -7,22 +7,21 @@
  * file that was distributed with this source code.
  */
 
-extern crate winapi;
+use std::ffi::OsString;
+use std::io;
+use std::os::windows::ffi::OsStringExt;
 
-use self::winapi::shared::lmcons;
+use winapi::shared::lmcons;
-use self::winapi::shared::minwindef;
+use winapi::shared::minwindef::DWORD;
-use self::winapi::um::{winbase, winnt};
+use winapi::um::winbase;
-use std::io::{Error, Result};
-use std::mem;
-use uucore::wide::FromWide;
 
-pub unsafe fn get_username() -> Result<String> {
+pub fn get_username() -> io::Result<OsString> {
-    #[allow(deprecated)]
+    const BUF_LEN: DWORD = lmcons::UNLEN + 1;
-    let mut buffer: [winnt::WCHAR; lmcons::UNLEN as usize + 1] = mem::uninitialized();
+    let mut buffer = [0_u16; BUF_LEN as usize];
-    let mut len = buffer.len() as minwindef::DWORD;
+    let mut len = BUF_LEN;
-    if winbase::GetUserNameW(buffer.as_mut_ptr(), &mut len) == 0 {
+    // SAFETY: buffer.len() == len
-        return Err(Error::last_os_error());
+    if unsafe { winbase::GetUserNameW(buffer.as_mut_ptr(), &mut len) } == 0 {
+        return Err(io::Error::last_os_error());
     }
-    let username = String::from_wide(&buffer[..len as usize - 1]);
+    Ok(OsString::from_wide(&buffer[..len as usize - 1]))
-    Ok(username)
 }

src/uu/whoami/src/whoami.rs

@@ -1,5 +1,3 @@
-use clap::App;
-
 //  * This file is part of the uutils coreutils package.
 //  *
 //  * (c) Jordi Boggiano <j.boggiano@seld.be>
@@ -14,46 +12,25 @@ extern crate clap;
 #[macro_use]
 extern crate uucore;
 
-use uucore::error::{UResult, USimpleError};
+use clap::App;
+
+use uucore::display::println_verbatim;
+use uucore::error::{FromIo, UResult};
 
 mod platform;
 
+static ABOUT: &str = "Print the current username.";
+
 #[uucore_procs::gen_uumain]
 pub fn uumain(args: impl uucore::Args) -> UResult<()> {
-    let app = uu_app();
+    uu_app().get_matches_from(args);
-
+    let username = platform::get_username().map_err_context(|| "failed to get username".into())?;
-    if let Err(err) = app.get_matches_from_safe(args) {
+    println_verbatim(&username).map_err_context(|| "failed to print username".into())?;
-        if err.kind == clap::ErrorKind::HelpDisplayed
+    Ok(())
-            || err.kind == clap::ErrorKind::VersionDisplayed
-        {
-            println!("{}", err);
-            Ok(())
-        } else {
-            return Err(USimpleError::new(1, format!("{}", err)));
-        }
-    } else {
-        exec()
-    }
 }
 
 pub fn uu_app() -> App<'static, 'static> {
-    app_from_crate!()
+    App::new(uucore::util_name())
-}
+        .version(crate_version!())
-
+        .about(ABOUT)
-pub fn exec() -> UResult<()> {
-    unsafe {
-        match platform::get_username() {
-            Ok(username) => {
-                println!("{}", username);
-                Ok(())
-            }
-            Err(err) => match err.raw_os_error() {
-                Some(0) | None => Err(USimpleError::new(1, "failed to get username")),
-                Some(_) => Err(USimpleError::new(
-                    1,
-                    format!("failed to get username: {}", err),
-                )),
-            },
-        }
-    }
 }

tests/by-util/test_whoami.rs

@@ -3,7 +3,6 @@
 //  * For the full copyright and license information, please view the LICENSE
 //  * file that was distributed with this source code.
 
-#[cfg(unix)]
 use crate::common::util::*;
 
 #[test]
@@ -34,7 +33,6 @@ fn test_normal_compare_id() {
 }
 
 #[test]
-#[cfg(unix)]
 fn test_normal_compare_env() {
     let whoami = whoami();
     if whoami == "nobody" {

tests/common/util.rs

@@ -1069,10 +1069,12 @@ pub fn whoami() -> String {
 
     // Use environment variable to get current user instead of
     // invoking `whoami` and fall back to user "nobody" on error.
-    std::env::var("USER").unwrap_or_else(|e| {
+    std::env::var("USER")
-        println!("{}: {}, using \"nobody\" instead", UUTILS_WARNING, e);
+        .or_else(|_| std::env::var("USERNAME"))
-        "nobody".to_string()
+        .unwrap_or_else(|e| {
-    })
+            println!("{}: {}, using \"nobody\" instead", UUTILS_WARNING, e);
+            "nobody".to_string()
+        })
 }
 
 /// Add prefix 'g' for `util_name` if not on linux