[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Crash when updating

From: Paul Burba <ptburba_at_gmail.com>
Date: Mon, 9 Jan 2012 10:16:48 -0500

On Thu, Jan 5, 2012 at 2:37 PM, Stefan Küng <tortoisesvn_at_gmail.com> wrote:
> Hi,
>
> From a crash report dump sent for TSVN for an update the attached stack
> trace happened.
>
> libsvn_wc\props.c, svn_wc__merge_props()
>      to_val = incoming_change->value
>        ? svn_string_dup(incoming_change->value, result_pool) : NULL;
> ....
>      if (! from_val)  /* adding a new property */
>        SVN_ERR(apply_single_prop_add(state, &conflict_remains,
>                                      db, local_abspath,
>                                      left_version, right_version,
>                                      is_dir, actual_props,
>                                      propname, base_val, to_val,
>                                      conflict_func, conflict_baton,
>                                      dry_run, result_pool, iterpool));
> and then in apply_single_prop_add():
> ...
>      if (svn_string_compare(working_val, new_val))
> crashes, because new_val is NULL.
>
> the property is "svn:mergeinfo".

Hi Stefan,

I saw 'mergeinfo' so took a quick look (happily it has nothing to do
with mergeinfo per se, any property will do ;-)

> Could this happen if someone manually removes the svn:mergeinfo property and
> then the next update (from another working copy) gets the to_val as NULL?

I'm not sure if this is what you mean, but I see one way to replicate
this problem: Create a file external mapping to a file with some
arbitrary property. Remap the same external to a file without that
property. Boom.

I filed an issue (issue #4093) which gives the full replication
recipe. I also added a regression test in r1228340.
Paul
Received on 2012-01-09 16:17:35 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.