Fixed bug dereferencing garbage, causing crash.
authorEric Mullins <emwine@earthlink.net>
Thu, 1 Oct 2009 22:50:28 +0000 (16:50 -0600)
committerEric Mullins <emwine@earthlink.net>
Thu, 1 Oct 2009 22:50:28 +0000 (16:50 -0600)
commit7e9ca02d2427c59cf04b2e9a61010ed5658c2016
treefb886d8066cb53d56661d467cef93266b01b8b48
parent4f827f6365be213cfd93979c562662445dec53e5
Fixed bug dereferencing garbage, causing crash.

Commit fa8be4a3fc5d81e9ec1f6c218fcf55c95d34fd10 included code to free the
resultDetails of the gameInfo structure.  That exact code exists a few lines
above where it was added.  None of these free() calls finish by setting the
member to NULL afterword, therefore repeated attempts to test/free() the
members result in an attempt to free() memory winboard no longer owns, and
an eventual crash.

link to problematic commit: http://git.savannah.gnu.org/cgit/xboard.git/commit/?id=fa8be4a3fc5d81e9ec1f6c218fcf55c95d34fd10
gamelist.c [changed mode: 0644->0755]