Ticket #212 (closed defect: fixed)

Opened 18 months ago

Last modified 6 months ago

Don't crash if gtp_aa_confirm_safety is fed an illegal move

Reported by: bump Owned by: gnugo
Priority: normal Milestone: 3.9.1
Component: source Version:
Severity: normal Keywords:
Cc: patch: yes

Description (last modified by gunnar) (diff)

see
 http://lists.gnu.org/archive/html/gnugo-devel/2009-02/msg00000.html
 http://lists.gnu.org/archive/html/gnugo-devel/2009-02/msg00003.html
and thread for details

Attachments

Regression Results

Attachment Rev. PASS FAIL Nodes Status
0001-don-t-crash-if-gtp_aa_confirm_safety-is-fed-an-illeg.patch Download never tested

Change History

Changed 18 months ago by gunnar

  • patch set
  • description modified (diff)
  • milestone changed from 3.9.x to 3.9.1

This patch is not good since it leaves the stone played if the move is legal, promptly causing the reported assertion failure, but now for the valid moves. An else popgo() should solve the problem.

Changed 18 months ago by bump

Changed 18 months ago by bump

replaced attachment. added popgo()

Changed 6 months ago by gunnar

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.