On Thu, Jan 5, 2012 at 2:37 PM, Stefan Küng <tortoise...@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

Reply via email to