see if this makes a difference on windows (#386)

Author: Byron

git-sec/src/lib.rs

@@ -52,8 +52,8 @@ pub mod identity {
         use std::borrow::Cow;
         use std::path::Path;
 
-        fn err(msg: &str) -> std::io::Error {
-            std::io::Error::new(std::io::ErrorKind::Other, msg)
+        fn err(msg: impl Into<String>) -> std::io::Error {
+            std::io::Error::new(std::io::ErrorKind::Other, msg.into())
         }
 
         pub fn is_path_owned_by_current_user(path: Cow<'_, Path>) -> std::io::Result<bool> {
@@ -75,12 +75,12 @@ pub mod identity {
                     .map_err(|_| err("Failed to open process token"))?;
 
                 let mut len = 0_u32;
-                if Security::GetTokenInformation(&handle, Security::TokenUser, std::ptr::null_mut(), 0, &mut len)
+                if !Security::GetTokenInformation(handle, Security::TokenUser, std::ptr::null_mut(), 0, &mut len)
                     .as_bool()
                 {
                     let mut token_user = Security::TOKEN_USER::default();
                     if Security::GetTokenInformation(
-                        &handle,
+                        handle,
                         Security::TokenUser,
                         &mut token_user as *mut _ as *mut std::ffi::c_void,
                         len,
@@ -94,6 +94,7 @@ pub mod identity {
                         if Security::IsValidSid(token_user.User.Sid).as_bool() {
                             use std::os::windows::ffi::OsStrExt;
                             let mut wide_path: Vec<_> = path.as_ref().as_os_str().encode_wide().collect();
+                            wide_path.push(0);
                             // err = GetNamedSecurityInfoW(wpath, SE_FILE_OBJECT,
                             //                             OWNER_SECURITY_INFORMATION |
                             //                                 DACL_SECURITY_INFORMATION,
@@ -113,16 +114,16 @@ pub mod identity {
                             if res == ERROR_SUCCESS.0 && Security::IsValidSid(path_sid).as_bool() {
                                 is_owned = Security::EqualSid(path_sid, token_user.User.Sid).as_bool();
                             } else {
-                                err_msg = "couldn't get owner for path or it wasn't valid".into();
+                                err_msg = format!("couldn't get owner for path or it wasn't valid: {}", res).into();
                             }
                         } else {
-                            err_msg = "owner id of current process wasn't set or valid".into();
+                            err_msg = String::from("owner id of current process wasn't set or valid").into();
                         }
                     } else {
-                        err_msg = "Could not get information about the token user".into();
+                        err_msg = String::from("Could not get information about the token user").into();
                     }
                 } else {
-                    err_msg = "Could not get token information for length of token user".into();
+                    err_msg = String::from("Could not get token information for length of token user").into();
                 }
                 CloseHandle(handle);
                 if !descriptor.is_invalid() {