folly: fbvector: ubsan: avoid memcpy(dest, nullptr, 0)

Reviewed By: meyering

Differential Revision: D3277138

fbshipit-source-id: 7050d18c256ac15c8122d4af59aa31208b1466fd

diff --git a/folly/FBVector.h b/folly/FBVector.h
index e1809f3179370ed5455c7ef93d6d1f5b5547a199..fb611e56c1e4759b8a610d289f55fa6671449d05 100644 (file)
--- a/folly/FBVector.h
+++ b/folly/FBVector.h
@@ -636,7 +636,9 @@ private:
   }
 
   void relocate_move_or_memcpy(T* dest, T* first, T* last, std::true_type) {
-    std::memcpy((void*)dest, (void*)first, (last - first) * sizeof(T));
+    if (first != nullptr) {
+      std::memcpy((void*)dest, (void*)first, (last - first) * sizeof(T));
+    }
   }
 
   void relocate_move_or_memcpy(T* dest, T* first, T* last, std::false_type) {