svn commit: r1033166 - /subversion/trunk/subversion/mod_dav_svn/repos.c

[st...@apache.org]

Author: stsp
Date: Tue Nov  9 19:23:35 2010
New Revision: 1033166

URL: http://svn.apache.org/viewvc?rev=1033166&view=rev
Log:
* subversion/mod_dav_svn/repos.c
  (walk): Do not attempt to walk SVNParentPath collections as this makes
   no sense and will ultimately lead to crashes further down the code path.

Found by: Rob Kooper

Patch by: philip
          me

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

Modified: subversion/trunk/subversion/mod_dav_svn/repos.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/repos.c?rev=1033166&r1=1033165&r2=1033166&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/repos.c (original)
+++ subversion/trunk/subversion/mod_dav_svn/repos.c Tue Nov  9 19:23:35 2010
@@ -4119,6 +4119,12 @@ walk(const dav_walk_params *params, int 
   walker_ctx_t ctx = { 0 };
   dav_error *err;
 
+  if (params->root->info->restype == DAV_SVN_RESTYPE_PARENTPATH_COLLECTION)
+    {
+      /* Cannot walk an SVNParentPath collection, there is no repository. */
+      return NULL;
+    }
+
   ctx.params = params;
 
   ctx.wres.walk_ctx = params->walk_ctx;