kate/BadFish
1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-04-30 16:53:09 +00:00
Code Issues Releases Wiki Activity

Fix memory access in Search::clear()

Browse source 
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
This commit is contained in:
Joost VandeVondele 2017-05-17 18:15:01 -07:00 • committed by Joona Kiiski
parent 862934d7ae
commit 732aa34e3d
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/search.cpp
View file

@ -197,9 +197,10 @@ void Search::clear() {
th->history.clear(); th->history.clear();
th->counterMoveHistory.clear(); th->counterMoveHistory.clear();
th->resetCalls = true; th->resetCalls = true;
CounterMoveStats& cm = th->counterMoveHistory[NO_PIECE][0]; CounterMoveStats& cm = th->counterMoveHistory[NO_PIECE][0];
int* t = &cm[NO_PIECE][0]; auto* t = &cm[NO_PIECE][0];
std::fill(t, t + sizeof(cm), CounterMovePruneThreshold - 1); std::fill(t, t + sizeof(cm)/sizeof(*t), CounterMovePruneThreshold - 1);
} }
Threads.main()->previousScore = VALUE_INFINITE; Threads.main()->previousScore = VALUE_INFINITE;