ARROW-4488: [Rust] From AsRef<[u8]> for Buffer does not ensure correc…

…t padding Padding for all memory allocations needs to be a multiple of 64 bytes to make SIMD implementations easier and more efficient to implement. Author: Paddy Horan <[email protected]> Closes apache#3568 from paddyhoran/asref-padding and squashes the following commits: 05cb0e9 <Paddy Horan> Ensures padding to a multiple of 64 bytes
kszucs · Feb 8, 2019 · 5863a9f · 5863a9f
1 parent 842fdd8
commit 5863a9f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/rust/arrow/src/buffer.rs b/rust/arrow/src/buffer.rs
@@ -132,7 +132,8 @@ impl<T: AsRef<[u8]>> From<T> for Buffer {
         // allocate aligned memory buffer
         let slice = p.as_ref();
         let len = slice.len() * mem::size_of::<u8>();
-        let buffer = memory::allocate_aligned(len).unwrap();
+        let capacity = bit_util::round_upto_multiple_of_64(len);
+        let buffer = memory::allocate_aligned(capacity).unwrap();
         unsafe {
             memory::memcpy(buffer, slice.as_ptr(), len);
         }