You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by cm...@apache.org on 2010/09/22 18:09:26 UTC

svn commit: r1000060 - /subversion/trunk/subversion/mod_dav_svn/authz.c

Author: cmpilato
Date: Wed Sep 22 16:09:26 2010
New Revision: 1000060

URL: http://svn.apache.org/viewvc?rev=1000060&view=rev
Log:
Fix issue #3695 ("SVNPathAuthz short_circuit ignores the authz path
rules with repo prefix") by passing the correct value to
mod_authz_svn, which wants a repository's basename, not it's
(typically undefined) nickname.

* subversion/mod_dav_svn/authz.c
  (dav_svn__allow_read): Pass repo_basename to the authz checker
    function, not repo_name.

Modified:
    subversion/trunk/subversion/mod_dav_svn/authz.c

Modified: subversion/trunk/subversion/mod_dav_svn/authz.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/authz.c?rev=1000060&r1=1000059&r2=1000060&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/authz.c (original)
+++ subversion/trunk/subversion/mod_dav_svn/authz.c Wed Sep 22 16:09:26 2010
@@ -60,7 +60,7 @@ dav_svn__allow_read(request_rec *r,
   allow_read_bypass = dav_svn__get_pathauthz_bypass(r);
   if (allow_read_bypass != NULL)
     {
-      if (allow_read_bypass(r, path, repos->repo_name) == OK)
+      if (allow_read_bypass(r, path, repos->repo_basename) == OK)
         return TRUE;
       else
         return FALSE;