Merge pull request rust-lang#225 from Hywan/fix-module-leak

fix(module,values) Fix memory leaks

src/module.rs

@@ -23,7 +23,6 @@ use std::mem::{forget, MaybeUninit};
 use std::path::Path;
 use std::ptr;
 use std::rc::Rc;
-use std::slice::from_raw_parts;
 
 use crate::{AddressSpace, OptimizationLevel};
 #[llvm_versions(7.0..=latest)]
@@ -930,20 +929,18 @@ impl<'ctx> Module<'ctx> {
     /// ```
     pub fn get_global_metadata(&self, key: &str) -> Vec<MetadataValue<'ctx>> {
         let c_string = to_c_str(key);
-        let count = self.get_global_metadata_size(key);
+        let count = self.get_global_metadata_size(key) as usize;
 
-        let mut raw_vec: Vec<LLVMValueRef> = Vec::with_capacity(count as usize);
-        let ptr = raw_vec.as_mut_ptr();
+        let mut vec: Vec<LLVMValueRef> = Vec::with_capacity(count);
+        let ptr = vec.as_mut_ptr();
 
-        forget(raw_vec);
-
-        let slice = unsafe {
+        unsafe {
             LLVMGetNamedMetadataOperands(self.module.get(), c_string.as_ptr(), ptr);
 
-            from_raw_parts(ptr, count as usize)
+            vec.set_len(count);
         };
 
-        slice.iter().map(|val| MetadataValue::new(*val)).collect()
+        vec.iter().map(|val| MetadataValue::new(*val)).collect()
     }
 
     /// Gets the first `GlobalValue` in a module.

src/values/metadata_value.rs

@@ -12,8 +12,6 @@ use crate::values::{BasicMetadataValueEnum, Value};
 
 use std::ffi::CStr;
 use std::fmt;
-use std::mem::forget;
-use std::slice::from_raw_parts;
 
 // TODOC: Varies by version
 #[cfg(feature = "llvm3-6")]
@@ -109,19 +107,19 @@ impl<'ctx> MetadataValue<'ctx> {
             return Vec::new();
         }
 
-        let count = self.get_node_size();
-        let mut raw_vec: Vec<LLVMValueRef> = Vec::with_capacity(count as usize);
-        let ptr = raw_vec.as_mut_ptr();
+        let count = self.get_node_size() as usize;
+        let mut vec: Vec<LLVMValueRef> = Vec::with_capacity(count);
+        let ptr = vec.as_mut_ptr();
 
-        forget(raw_vec);
-
-        let slice = unsafe {
+        unsafe {
             LLVMGetMDNodeOperands(self.as_value_ref(), ptr);
 
-            from_raw_parts(ptr, count as usize)
+            vec.set_len(count)
         };
 
-        slice.iter().map(|val| BasicMetadataValueEnum::new(*val)).collect()
+        vec.iter()
+            .map(|val| BasicMetadataValueEnum::new(*val))
+            .collect()
     }
 
     pub fn print_to_string(self) -> LLVMString {