Unmark list_accounts as unsafe

Author: allan2

crates/devolutions-pedm/src/account.rs

@@ -209,7 +209,7 @@ impl PartialEq<Account> for AccountWithId {
 ///
 /// `LookupAccountNameW` must be called to enable `ConvertSidToStringSidW` to work.
 #[cfg(target_os = "windows")]
-pub(crate) unsafe fn list_accounts() -> Result<Vec<Account>, ListAccountsError> {
+pub(crate) fn list_accounts() -> Result<Vec<Account>, ListAccountsError> {
     use windows::core::PWSTR;
     use windows::Win32::NetworkManagement::NetManagement::{
         NERR_Success, NetApiBufferFree, NetUserEnum, FILTER_NORMAL_ACCOUNT, MAX_PREFERRED_LENGTH, USER_INFO_0,

crates/devolutions-pedm/src/api/mod.rs

@@ -183,7 +183,7 @@ pub async fn serve(config: Config, shutdown_signal: ShutdownSignal) -> Result<()
     // Update the list of accounts in the database.
 
     // SAFETY: uses `NetUserEnum` and `LookupAccountNameW` from `windows`
-    let accounts = unsafe { list_accounts()? };
+    let accounts = list_accounts()?;
 
     let db_accounts = db.get_accounts().await?;
     info!("Accounts retrieved successfully");