Merge pull request #957 from GuillaumeGomez/fix-windows-panic

Fix potential windows panic when getting process data

src/linux/process.rs

@@ -675,10 +675,10 @@ fn get_uid_and_gid(file_path: &Path) -> Option<(uid_t, gid_t)> {
     let mut gid = None;
     for line in status_data.lines() {
         if let Some(u) = f(line, "Uid:") {
-            assert!(uid.is_none());
+            debug_assert!(uid.is_none());
             uid = Some(u);
         } else if let Some(g) = f(line, "Gid:") {
-            assert!(gid.is_none());
+            debug_assert!(gid.is_none());
             gid = Some(g);
         } else {
             continue;

src/windows/process.rs

@@ -690,8 +690,10 @@ unsafe fn get_process_data(
         return Err("Unable to read process data");
     }
 
-    // Documentation states that the function fails if not all data is accessible
-    assert_eq!(bytes_read, size);
+    // Documentation states that the function fails if not all data is accessible.
+    if bytes_read != size {
+        return Err("ReadProcessMemory returned unexpected number of bytes read");
+    }
 
     buffer.set_len(size / 2);
     buffer.push(0);