From 3e4dcaa06e7fd555628f1c3a65f4b7967d9708cd Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Fri, 30 Aug 2013 16:40:11 +0200 Subject: [PATCH] Fix a bogus assert in allows() Becuase castle is coded as "king captures the rook" the to_sq(move), A1/8 or H1/8 is empty after the move, leading to assert assert(p != NO_PIECE) in color_of(). Teach allows() asserts about castle and fix the crash. Bug reported by Ryan Takker and tracked down by Tom Vijlbrief. No functional change. --- src/search.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/search.cpp b/src/search.cpp index c85cdb5d..7b43f64a 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -1400,7 +1400,7 @@ moves_loop: // When in check and at SpNode search starts from here assert(is_ok(first)); assert(is_ok(second)); assert(color_of(pos.piece_on(from_sq(second))) == ~pos.side_to_move()); - assert(color_of(pos.piece_on(to_sq(first))) == ~pos.side_to_move()); + assert(type_of(first) == CASTLE || color_of(pos.piece_on(to_sq(first))) == ~pos.side_to_move()); Square m1from = from_sq(first); Square m2from = from_sq(second); -- 2.39.2