Fix memory access in Search::clear()

Fixes a bug in Search::clear, where the filling of CounterMoveStats&, overwrote (currently presumably unused) memory because sizeof(cm) returns the size in bytes, whereas elements was needed.

No functional change

Closes #1119

diff --git a/src/search.cpp b/src/search.cpp
index e8e15ef2cf58826edff4c82b9a9d97ce948fe384..de487e57b2d000768842b98384402921e8a95261 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -197,9 +197,10 @@ void Search::clear() {
       th->history.clear();
       th->counterMoveHistory.clear();
       th->resetCalls = true;
       th->history.clear();
       th->counterMoveHistory.clear();
       th->resetCalls = true;

+

       CounterMoveStats& cm = th->counterMoveHistory[NO_PIECE][0];
       CounterMoveStats& cm = th->counterMoveHistory[NO_PIECE][0];

-      int* t = &cm[NO_PIECE][0];
-      std::fill(t, t + sizeof(cm), CounterMovePruneThreshold - 1);
+      auto* t = &cm[NO_PIECE][0];
+      std::fill(t, t + sizeof(cm)/sizeof(*t), CounterMovePruneThreshold - 1);

   }
 
   Threads.main()->previousScore = VALUE_INFINITE;
   }
 
   Threads.main()->previousScore = VALUE_INFINITE;