Re: core dump with svn r6178

[Patrick Mayweg <ma...@qint.de>]

Hi Ben,

Ben Collins-Sussman wrote:

> Patrick Mayweg <ma...@qint.de> writes:
>
> > Hi,
> > I had a NULL-pointer access at the same place last week. I think I have it
> > tracked down to where it realy went wrong.
> > In the file subversion/libsvn_ra_dav/fetch.c is the function
> > static svn_error_t * reporter_finish_report(void *report_baton)
> >
> > [...]
> >
> > The retry after the error is the problem. In my case the REPORT
> > request was send ok by the server but the client network code
> > returned "connection reset" error after most of the report request
> > was processed. In this case you cannot restart the request on the
> > same report_beacon, because there is state in the beacon.  Like that
> > rb->dirs->nelts is not 0. The code in static int start_element(void
> > *userdata, const struct ne_xml_elm *elm, const char **atts) assummes
> > that only on the first OPEN-DIRECTORY element there is no "name"
> > attribute. Unfortunately I do not known enough about the
> > report_beacon, to provide a patch which tests if it has already be
> > used.  Regrads, Patrick
>
> Patrick, I think this bug is now fixed in r6377.

that looks much better. I am not when I will be able to test it.
Thanks,
Patrick


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org