Re: segfault during merge

On 04/15/2009 07:53 PM, Stefan Küng wrote:
> Hi,
>
> A strange one from a crashdump for TSVN:
>
> file libsvn_ra\compat.c, line 802, function log_path_del_receiver():
>
> for (hi = apr_hash_first(pool, log_entry->changed_paths2);
> hi != NULL;
> hi = apr_hash_next(hi))
> {
>
> The 'log_entry->changed_paths2' is NULL.
> (changed_paths and revprops of log_entry are also NULL).
>
> But I have no idea why there would be no changed paths in a revision?

Because the revision is empty, or it contains paths that aren't readable.

> Maybe the callstack might help?
>
> TortoiseProc.exe!log_path_del_receiver(void * baton=0x01ddf8dc,
> svn_log_entry_t * log_entry=0x02278af8, apr_pool_t * pool=0x02282868)
> Line 801 + 0x17 bytes C

This code is only used with server version < 1.6. It's missing the check
for empty paths list.

Fixed in r37363 and proposed for backport to the 1.6.x branch.

Thanks for the report

Lieven

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1807927
Received on 2009-04-19 16:34:56 CEST