On Wed, Nov 16, 2011 at 12:24 PM, Stefan Küng <tortoisesvn_at_gmail.com> wrote:
> Hi,
>
> A few crash reports for TSVN show a problem in libsvn_client\merge.c,
> function merge_props_changed:
>
> if (strcmp(prop->name, SVN_PROP_MERGEINFO) == 0)
> {
> Â /* Does LOCAL_ABSPATH have any pristine mergeinfo? */
> Â svn_boolean_t has_pristine_mergeinfo = FALSE;
> Â apr_hash_t *pristine_props;
>
> Â SVN_ERR(svn_wc_get_pristine_props(&pristine_props,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ctx->wc_ctx,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â local_abspath,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â scratch_pool,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â scratch_pool));
>
> Â if (apr_hash_get(pristine_props, SVN_PROP_MERGEINFO,
> Â Â Â Â Â Â Â Â Â APR_HASH_KEY_STRING))
> Â Â has_pristine_mergeinfo = TRUE;
>
> The svn_wc_get_pristine_props() sets pristine_props to NULL, which then
> results in a segfault in apr_hash_get().
>
> svn_wc_get_pristine_props() can set pristine_props to NULL and still return
> SVN_NO_ERROR if status is either incomplete, excluded, server_excluded or
> not_present. So it _can_ happen.
>
> Checked the source of the svn trunk: the issue is still there. Haven't found
> an entry in the issue tracker so I'm reporting this here.
Thanks Stefan,
Fixed in r1202807and nominated for backport to 1.7.2.
Paul
> Stefan
>
>
> --
> Â Â Â ___
>  oo  // \\    "De Chelonian Mobile"
> Â (_,\/ \_/ \ Â Â TortoiseSVN
> Â \ \_/_\_/> Â Â The coolest Interface to (Sub)Version Control
> Â /_/ Â \_\ Â Â http://tortoisesvn.net
>
Received on 2011-11-16 18:57:15 CET