From 5a25d8d55a62bb3e0062e7e6062dd2eee40ab882 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Tue, 19 Nov 2013 13:13:22 +0000 Subject: [PATCH] Don't allocate way too much memory in wxVector::reserve(). We multiplied the number of items by the size of each element twice, once in wxVector::reserve() itself and once in Ops::Realloc() it called, so we allocated much more memory than actually needed. Fix this by passing the number of elements to Ops::Realloc(). git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@75246 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- include/wx/vector.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/wx/vector.h b/include/wx/vector.h index dffb19fc9d..e81cafb082 100644 --- a/include/wx/vector.h +++ b/include/wx/vector.h @@ -327,7 +327,7 @@ public: if ( m_capacity + increment > n ) n = m_capacity + increment; - m_values = Ops::Realloc(m_values, n * sizeof(value_type), m_size); + m_values = Ops::Realloc(m_values, n, m_size); m_capacity = n; }