You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by st...@apache.org on 2011/12/14 13:38:48 UTC

svn commit: r1214216 - /subversion/trunk/subversion/libsvn_repos/load-fs-vtable.c

Author: stsp
Date: Wed Dec 14 12:38:48 2011
New Revision: 1214216

URL: http://svn.apache.org/viewvc?rev=1214216&view=rev
Log:
* subversion/libsvn_repos/load-fs-vtable.c
  (delete_node_property): Don't crash trying to delete properties in skipped
   revisions.

Reported by: Trent Nelson

Modified:
    subversion/trunk/subversion/libsvn_repos/load-fs-vtable.c

Modified: subversion/trunk/subversion/libsvn_repos/load-fs-vtable.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_repos/load-fs-vtable.c?rev=1214216&r1=1214215&r2=1214216&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_repos/load-fs-vtable.c (original)
+++ subversion/trunk/subversion/libsvn_repos/load-fs-vtable.c Wed Dec 14 12:38:48 2011
@@ -787,6 +787,10 @@ delete_node_property(void *baton,
   struct node_baton *nb = baton;
   struct revision_baton *rb = nb->rb;
 
+  /* If we're skipping this revision, we're done here. */
+  if (rb->skipped)
+    return SVN_NO_ERROR;
+
   return change_node_prop(rb->txn_root, nb->path, name, NULL,
                           rb->pb->validate_props, nb->pool);
 }