From: Lucian Grijincu <lucian@fb.com>
Date: Sat, 14 May 2016 07:09:41 +0000 (-0700)
Subject: folly: ubsan: elias-fano avoid memset(nullptr)
X-Git-Tag: 2016.07.26~235
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=3ce293faff5749f9935de1fdc7506c75296c4e6c;p=folly.git

folly: ubsan: elias-fano avoid memset(nullptr)

Reviewed By: ot

Differential Revision: D3300437

fbshipit-source-id: 7e0add5dab6dc1c891b742b4bcedea656c8284a9
---

diff --git a/folly/experimental/EliasFanoCoding.h b/folly/experimental/EliasFanoCoding.h
index 9f3b8744..00a59fcb 100644
--- a/folly/experimental/EliasFanoCoding.h
+++ b/folly/experimental/EliasFanoCoding.h
@@ -23,6 +23,7 @@
 
 #pragma once
 
+#include <algorithm>
 #include <cstdlib>
 #include <limits>
 #include <type_traits>
@@ -140,7 +141,7 @@ struct EliasFanoEncoderV2 {
         forwardPointers_(reinterpret_cast<SkipValueType*>(
               result.forwardPointers)),
         result_(result) {
-    memset(result.data.data(), 0, result.data.size());
+    std::fill(result.data.begin(), result.data.end(), 0);
   }
 
   EliasFanoEncoderV2(size_t size, ValueType upperBound)