Make sure the return value of malloc_allocator::allocate is zero-initialized so...

	Make sure the return value of malloc_allocator::allocate
	is zero-initialized so that checking it for non-zero works
	later.
	* include/ext/malloc_allocator.h (malloc_allocator::allocate):
	Initialize the return value.
	* testsuite/ext/malloc_allocator/sanity.cc: New.

From-SVN: r241212

diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog
index 1537793bef09a7579c261793d1c717f190ccc5bd..fcb6a29bfd82e1fb743aa0680cdc5b6157680820 100644 (file)
--- a/libstdc++-v3/ChangeLog
+++ b/libstdc++-v3/ChangeLog
@@ -1,3 +1,12 @@
+2016-10-16  Ville Voutilainen  <ville.voutilainen@gmail.com>
+
+       Make sure the return value of malloc_allocator::allocate
+       is zero-initialized so that checking it for non-zero works
+       later.
+       * include/ext/malloc_allocator.h (malloc_allocator::allocate):
+       Initialize the return value.
+       * testsuite/ext/malloc_allocator/sanity.cc: New.
+
 2016-10-14  Eelis van der Weegen  <eelis@eelis.net>
 
        * include/bits/stl_algo.h (shuffle): Extract two random numbers from

diff --git a/libstdc++-v3/include/ext/malloc_allocator.h b/libstdc++-v3/include/ext/malloc_allocator.h
index acb60a2a3cf7bb1ed3bf9e0a825546abdea2be91..113c1dcfe6cc626042ea9cafc6e0158460262771 100644 (file)
--- a/libstdc++-v3/include/ext/malloc_allocator.h
+++ b/libstdc++-v3/include/ext/malloc_allocator.h
@@ -101,7 +101,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
        if (__n > this->max_size())
          std::__throw_bad_alloc();
 
-       pointer __ret;
+       pointer __ret = 0;
 #if __cpp_aligned_new
 #if __cplusplus > 201402L && _GLIBCXX_HAVE_ALIGNED_ALLOC
        if (alignof(_Tp) > alignof(std::max_align_t))

diff --git a/libstdc++-v3/testsuite/ext/malloc_allocator/sanity.cc b/libstdc++-v3/testsuite/ext/malloc_allocator/sanity.cc
new file mode 100644 (file)
index 0000000..b59e709
--- /dev/null
+++ b/libstdc++-v3/testsuite/ext/malloc_allocator/sanity.cc
@@ -0,0 +1,31 @@
+// Copyright (C) 2016 Free Software Foundation, Inc.
+//
+// This file is part of the GNU ISO C++ Library.  This library is free
+// software; you can redistribute it and/or modify it under the
+// terms of the GNU General Public License as published by the
+// Free Software Foundation; either version 3, or (at your option)
+// any later version.
+
+// This library is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+
+// You should have received a copy of the GNU General Public License along
+// with this library; see the file COPYING3.  If not see
+// <http://www.gnu.org/licenses/>.
+
+// { dg-do run { target c++11 } }
+
+#include <map>
+#include <vector>
+#include <ext/malloc_allocator.h>
+
+int main()
+{
+  std::map<int, int, std::less<int>,
+          __gnu_cxx::malloc_allocator<std::pair<int, int> > > allocs;
+  allocs[9] = 3;
+  std::vector<int, __gnu_cxx::malloc_allocator<int>> vec(10);
+  vec[5] = 42;
+}