svn commit: r1730421 - /subversion/trunk/subversion/libsvn_repos/log.c

[st...@apache.org]

Author: stefan2
Date: Sun Feb 14 22:32:21 2016
New Revision: 1730421

URL: http://svn.apache.org/viewvc?rev=1730421&view=rev
Log:
Follow-up to r1730388:
Empty revisions shall report "full revprop access" instead of "no access".

* subversion/libsvn_repos/log.c
  (svn_repos_check_revision_access): Re-instantiate special casing for
                                     empty revisions.

Modified:
    subversion/trunk/subversion/libsvn_repos/log.c

Modified: subversion/trunk/subversion/libsvn_repos/log.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_repos/log.c?rev=1730421&r1=1730420&r2=1730421&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_repos/log.c (original)
+++ subversion/trunk/subversion/libsvn_repos/log.c Sun Feb 14 22:32:21 2016
@@ -94,6 +94,13 @@ svn_repos_check_revision_access(svn_repo
   SVN_ERR(svn_fs_paths_changed3(&iterator, rev_root, pool, pool));
   SVN_ERR(svn_fs_path_change_get(&change, iterator));
 
+  /* No changed paths?  We're done.
+
+     Note that the check at "decision:" assumes that at least one
+     path has been processed.  So, this actually affects functionality. */
+  if (!change)
+    return SVN_NO_ERROR;
+
   /* Otherwise, we have to check the readability of each changed
      path, or at least enough to answer the question asked. */
   iterpool = svn_pool_create(pool);