You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ph...@apache.org on 2011/11/15 12:34:20 UTC

svn commit: r1202135 - in /subversion/trunk/subversion: include/svn_repos.h libsvn_repos/authz.c

Author: philip
Date: Tue Nov 15 11:34:20 2011
New Revision: 1202135

URL: http://svn.apache.org/viewvc?rev=1202135&view=rev
Log:
Fix issue 4060, spurious authz denied when svnserve root is a repository.

* subversion/include/svn_repos.h
  (svn_repos_authz_read): Document NULL repos_path.

* subversion/libsvn_repos/authz.c
  (svn_repos_authz_check_access): Handle NULL repos_path.

Modified:
    subversion/trunk/subversion/include/svn_repos.h
    subversion/trunk/subversion/libsvn_repos/authz.c

Modified: subversion/trunk/subversion/include/svn_repos.h
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_repos.h?rev=1202135&r1=1202134&r2=1202135&view=diff
==============================================================================
--- subversion/trunk/subversion/include/svn_repos.h (original)
+++ subversion/trunk/subversion/include/svn_repos.h Tue Nov 15 11:34:20 2011
@@ -2991,6 +2991,9 @@ svn_repos_authz_read(svn_authz_t **authz
  * to TRUE if at least one path is accessible with the @a
  * required_access.
  *
+ * For compatibility with 1.6, and earlier, @a repos_name can be NULL
+ * in which case it is equivalent to a @a repos_name of "".
+ *
  * @since New in 1.3.
  */
 svn_error_t *

Modified: subversion/trunk/subversion/libsvn_repos/authz.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_repos/authz.c?rev=1202135&r1=1202134&r2=1202135&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_repos/authz.c (original)
+++ subversion/trunk/subversion/libsvn_repos/authz.c Tue Nov 15 11:34:20 2011
@@ -785,6 +785,9 @@ svn_repos_authz_check_access(svn_authz_t
 {
   const char *current_path;
 
+  if (!repos_name)
+    repos_name = "";
+
   /* If PATH is NULL, check if the user has *any* access. */
   if (!path)
     {