Daniel Shahaf wrote:
> Justin Erenkrantz wrote on Thu, 1 May 2008 at 10:10 -0700:
>> On Thu, May 1, 2008 at 10:04 AM, Daniel Shahaf <d.s_at_daniel.shahaf.co.il> wrote:
>>> Yes, except that I don't know whether the bug is in Serf or in Subversion.
>>> (The abort(), however, is in Serf code.)
>> You should try trunk - not 0.1.2. (Yes, I'm overdue to release the
>> next version.) -- justin
>>
>
> I did, and it segfaulted in 'make check'. Lieven said it was a known
> issue.
>
To be clear, there are two known issues involved here. Known as in 'I
know about them, but that's about it'. The issue that started this
thread is an issue in ra_serf. In short, when you give a bucket with the
request body to serf, it will consume and release that bucket. When the
request has to be sent a second time (due to authentication required or
retry), that bucket can't be used again and must be recreated.
Practically this means that everytime we pass a request to serf we must
not pass a bucket with the body, but a callback that'll create the
bucket on demand, twice if needed.
There are a few locations in ra_serf where we still pass a body bucket,
this need to be converted to use the callback function.
The segfault in 'make check' of serf is purely serf related. The
segfault itself comes from the testsuite after a test failed. I'm not
sure if that test failure is really an issue in serf, but it should be
solved anyhow.
Hope this makes things clearer.
Lieven
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-02 14:43:17 CEST