[Xapian-tickets] [Xapian] #675: Exception in [Chert/Glass]Cursor::find_entry()

[Xapian]

#675: Exception in [Chert/Glass]Cursor::find_entry()
---------------------------+-----------------------------
 Reporter:  Kronuz         |             Owner:  olly
     Type:  defect         |            Status:  assigned
 Priority:  high           |         Milestone:
Component:  Backend-Chert  |           Version:
 Severity:  normal         |        Resolution:
 Keywords:                 |        Blocked By:
 Blocking:                 |  Operating System:  All
---------------------------+-----------------------------
Changes (by olly):

 * status:  new => assigned
 * component:  Other => Backend-Chert


Comment:

 Thanks for the patch, but I'm not quite sold on the logic behind the
 change.

 `ChertTable::cursor_get()` returns a new cursor, not the table's built-in
 cursor, so I'm not seeing how we end up with a stale cursor here, since
 the cursor which `ChertTable::close()` releases the memory for shouldn't
 be the cursor which `ChertValueManager` has (though valgrind's output does
 seem consistent with that being what's happening).

 I'll have a poke and see if I can either convince myself your patch is
 right, or if not, work out what's actually going on.

 Did you already agree to the patch licensing requirements?

 http://trac.xapian.org/browser/git/xapian-core/HACKING#L1376

--
Ticket URL: <http://trac.xapian.org/ticket/675#comment:6>
Xapian <http://xapian.org/>
Xapian