Merge pull request #794 from nicholasbishop/bishop-fix-vec-with

uefi: Clear the Vec in DevicePathBuilder::with_vec

Author: phip1611

CHANGELOG.md

@@ -50,6 +50,7 @@
 - The `Revision` struct's one field is now public.
 - Renamed `CStr8::to_bytes` to `CStr8::as_bytes` and changed the semantics:
   The trailing null character is now always included in the returned slice.
+- `DevicePathBuilder::with_vec` now clears the `Vec` before use.
 
 ## uefi-macros - [Unreleased]
 

uefi/src/proto/device_path/build.rs

@@ -83,8 +83,11 @@ impl<'a> DevicePathBuilder<'a> {
     }
 
     /// Create a builder backed by a `Vec`.
+    ///
+    /// The `Vec` is cleared before use.
     #[cfg(feature = "alloc")]
     pub fn with_vec(v: &'a mut Vec<u8>) -> Self {
+        v.clear();
         Self {
             storage: BuilderStorage::Vec(v),
         }