Ticket #349 (closed defect: fixed)

Opened 17 months ago

Last modified 16 months ago

nearsubqueries1 fails under valgrind

Reported by: richard Owned by: olly
Priority: normal Milestone: 1.0.12
Component: Library API Version: SVN trunk
Severity: normal Keywords:
Cc: Blocked By:
Operating System: All Blocking:

Description

Running apitest under valgrind fails on my (linux, i686, ubuntu hardy) box with the attached errors.

According to Charlie, this test crashes on windows in the 1.0 branch - I'd guess that it's probably the same problem.

Attachments

out Download (61.7 KB) - added by richard 17 months ago.
Output of ./runtest ./apitest nearsubqueries1 -v

Change History

Changed 17 months ago by richard

  • attachment out Download added

Output of ./runtest ./apitest nearsubqueries1 -v

Changed 17 months ago by olly

Yeah, this was uncovered when the valgrind logging problem was fixed and we suddenly saw all our valgrind errors - you even commented on it on #xapian-devel:

@1236013181 <richardb> ojwb: only nearsubqueries1 is failing for me (under valgrind) now. @1236013196 <richardb> Not sure I'm brave enough to try tackling it, though.

Given that the code there is really hard to follow, and hasn't changed for ages, I felt it was probably safer not to rush a fix for 1.0.11 - after all nobody has hit this in the wild.

Changed 17 months ago by olly

  • status changed from new to assigned

Ticket #201 is related. And I was wrong - this changed (albeit only assert -> exception) in 1.0.10.

I have a fix (which also fixes #201) but it fails to suck up sub-OR queries into a parent OR, etc so it really needs further work.

Changed 17 months ago by richard

Would be worth knowing if your fix fixes the crash which Charlie observes on windows in the 1.0 branch - can you attach it as a patch here?

Changed 17 months ago by olly

  • milestone changed from 1.1.0 to 1.0.12

Fixed in trunk r12228.

I'm virtually certain that this is the cause of Charlie's crash. The old code was rather ... odd. Be nice to have confirmation though.

Setting milestone:1.0.12

Changed 16 months ago by olly

  • status changed from assigned to closed
  • resolution set to fixed

Fixed for 1.0.12 in branches/1.0 r12258.

Note: See TracTickets for help on using tickets.