[prev in list] [next in list] [prev in thread] [next in thread]
List: subversion-commits
Subject: svn commit: r1771905 - /subversion/trunk/subversion/libsvn_client/conflicts.c
From: stsp () apache ! org
Date: 2016-11-29 13:54:41
Message-ID: 20161129135442.9ADCA3A0EB1 () svn01-us-west ! apache ! org
[Download RAW message or body]
Author: stsp
Date: Tue Nov 29 13:54:41 2016
New Revision: 1771905
URL: http://svn.apache.org/viewvc?rev=1771905&view=rev
Log:
Prevent the conflict resolver from searching paths in revision zero.
Problem uncovered by the tree conflict raised at the end of conflict_tests 25
which causes the resolver to search for a related path in r0 while trying to
figure out a nested move committed in r2.
* subversion/libsvn_client/conflicts.c
(rev_below): New helper function for decrementing revision numbers.
If we're already at the very bottom (r1), stay there.
Assert that callers don't try to search paths at invalid revisions or r0.
(check_move_ancestry, find_nested_move, find_deleted_rev,
conflict_tree_get_details_local_missing,
get_incoming_delete_details_for_reverse_addition,
describe_incoming_reverse_deletion_upon_merge,
resolve_merge_incoming_added_dir_merge,
merge_incoming_added_dir_replace): Use the new helper function instead of
hardcoding the pattern "rev - 1".
Modified:
subversion/trunk/subversion/libsvn_client/conflicts.c
Modified: subversion/trunk/subversion/libsvn_client/conflicts.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/conflicts.c?rev=1771905&r1=1771904&r2=1771905&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/conflicts.c (original)
+++ subversion/trunk/subversion/libsvn_client/conflicts.c Tue Nov 29 13:54:41 2016
@@ -278,6 +278,15 @@ struct repos_move_info {
apr_array_header_t *next;
};
+static svn_revnum_t
+rev_below(svn_revnum_t rev)
+{
+ SVN_ERR_ASSERT_NO_RETURN(rev != SVN_INVALID_REVNUM);
+ SVN_ERR_ASSERT_NO_RETURN(rev > 0);
+
+ return rev == 1 ? 1 : rev - 1;
+}
+
/* Set *RELATED to true if the deleted node DELETED_REPOS_RELPATH@DELETED_REV
* is an ancestor of the copied node COPYFROM_PATH@COPYFROM_REV.
* If CHECK_LAST_CHANGED_REV is non-zero, also ensure that the copied node
@@ -310,7 +319,7 @@ check_move_ancestry(svn_boolean_t *relat
SVN_ERR(svn_client__ensure_ra_session_url(&old_session_url, ra_session,
deleted_url, scratch_pool));
SVN_ERR(svn_ra_get_locations(ra_session, &locations, "",
- deleted_rev - 1, location_revisions,
+ rev_below(deleted_rev), location_revisions,
scratch_pool));
deleted_location = apr_hash_get(locations, ©from_rev,
@@ -337,7 +346,7 @@ check_move_ancestry(svn_boolean_t *relat
/* Verify that copyfrom_rev >= last-changed revision of the
* deleted node. */
- SVN_ERR(svn_ra_stat(ra_session, "", deleted_rev - 1, &dirent,
+ SVN_ERR(svn_ra_stat(ra_session, "", rev_below(deleted_rev), &dirent,
scratch_pool));
if (dirent == NULL || copyfrom_rev < dirent->created_rev)
{
@@ -684,7 +693,7 @@ find_nested_move(const char **moved_from
copy = APR_ARRAY_IDX(copies_with_same_source_path, 0, struct copy_info *);
SVN_ERR(check_move_ancestry(&related, ra_session, repos_root_url,
moved_along_repos_relpath,
- revision - 1,
+ rev_below(revision),
copy->copyfrom_path,
copy->copyfrom_rev,
TRUE, iterpool));
@@ -822,7 +831,7 @@ find_deleted_rev(void *baton,
b->related_repos_relpath,
b->related_repos_peg_rev,
b->deleted_repos_relpath,
- log_entry->revision - 1,
+ rev_below(log_entry->revision),
b->repos_root_url, b->repos_uuid,
b->ctx, iterpool, iterpool);
if (err)
@@ -1645,7 +1654,7 @@ conflict_tree_get_details_local_missing(
return SVN_NO_ERROR;
/* The node should exist in the revision before it was deleted. */
- related_peg_rev = related_deleted_rev - 1;
+ related_peg_rev = rev_below(related_deleted_rev);
}
}
@@ -3481,7 +3490,8 @@ get_incoming_delete_details_for_reverse_
{
svn_node_kind_t replaced_node_kind;
- SVN_ERR(svn_ra_check_path(ra_session, "", (*details)->added_rev - 1,
+ SVN_ERR(svn_ra_check_path(ra_session, "",
+ rev_below((*details)->added_rev),
&replaced_node_kind, scratch_pool));
if (replaced_node_kind != svn_node_none)
SVN_ERR(svn_ra_check_path(ra_session, "", (*details)->added_rev,
@@ -4099,7 +4109,7 @@ describe_incoming_reverse_deletion_upon_
_("A new directory appeared during reverse-merge "
"of\n'^/%s:%ld-%ld'.\n"
"It was deleted by %s in r%ld."),
- old_repos_relpath, new_rev, old_rev - 1,
+ old_repos_relpath, new_rev, rev_below(old_rev),
details->deleted_rev_author,
details->deleted_rev);
}
@@ -6302,7 +6312,7 @@ resolve_merge_incoming_added_dir_merge(s
"added the repository"),
svn_dirent_local_style(local_abspath,
scratch_pool));
- rev1 = details->added_rev - 1;
+ rev1 = rev_below(details->added_rev);
source2 = svn_path_url_add_component2(repos_root_url,
incoming_new_repos_relpath,
scratch_pool);
@@ -6581,8 +6591,8 @@ merge_incoming_added_dir_replace(svn_cli
* this merge was part of the merge target working copy, not a branch
* in the repository. */
err = merge_newly_added_dir(base_repos_relpath,
- url, b.added_rev - 1, url, base_revision,
- local_abspath, FALSE,
+ url, rev_below(b.added_rev), url,
+ base_revision, local_abspath, FALSE,
ctx, scratch_pool, scratch_pool);
if (err)
goto unlock_wc;
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic