No, we shouldn't call mem::forget to free

mullvad · Sep 24, 2024 · 420e4b5 · 420e4b5
1 parent 38aeb2b
commit 420e4b5
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/mullvad-ios/src/encrypted_dns_proxy.rs b/mullvad-ios/src/encrypted_dns_proxy.rs
@@ -148,7 +148,7 @@ pub unsafe extern "C" fn encrypted_dns_proxy_init() -> *mut EncryptedDnsProxySta
 /// by `encrypted_dns_proxy_init`. This function is not thread safe.
 #[no_mangle]
 pub unsafe extern "C" fn encrytped_dns_proxy_free(ptr: *mut EncryptedDnsProxyState) {
-    mem::forget(unsafe { Box::from_raw(ptr) })
+    let _ = unsafe { Box::from_raw(ptr) };
 }
 
 /// # Safety