Opened 16 years ago

Closed 16 years ago

#349 closed defect (fixed)

nearsubqueries1 fails under valgrind

Reported by: Richard Boulton Owned by: Olly Betts
Priority: normal Milestone: 1.0.12
Component: Library API Version: SVN trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Operating System: All

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 (1)

out (61.7 KB ) - added by Richard Boulton 16 years ago.
Output of ./runtest ./apitest nearsubqueries1 -v

Download all attachments as: .zip

Change History (6)

by Richard Boulton, 16 years ago

Attachment: out added

Output of ./runtest ./apitest nearsubqueries1 -v

comment:1 by Olly Betts, 16 years ago

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.

comment:2 by Olly Betts, 16 years ago

Status: newassigned

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.

comment:3 by Richard Boulton, 16 years ago

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?

comment:4 by Olly Betts, 16 years ago

Milestone: 1.1.01.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

comment:5 by Olly Betts, 16 years ago

Resolution: fixed
Status: assignedclosed

Fixed for 1.0.12 in branches/1.0 r12258.

Note: See TracTickets for help on using tickets.