svn commit: r1847555 - in /subversion/branches/1.10.x: STATUS subversion/mod_dav_svn/reports/list.c

[rhuijben]

Author: rhuijben
Date: Tue Nov 27 15:20:09 2018
New Revision: 1847555

URL: http://svn.apache.org/viewvc?rev=1847555&view=rev
Log:
* STATUS: Cast some votes. Approve 2 groups.

Modified:
    subversion/branches/1.10.x/STATUS
    subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c

Modified: subversion/branches/1.10.x/STATUS
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.10.x/STATUS?rev=1847555&r1=1847554&r2=1847555&view=diff
==============================================================================
--- subversion/branches/1.10.x/STATUS (original)
+++ subversion/branches/1.10.x/STATUS Tue Nov 27 15:20:09 2018
@@ -22,6 +22,12 @@ Candidate changes:
    Votes:
      +1: brane
 
+Veto-blocked changes:
+=====================
+
+Approved changes:
+=================
+
  * r1846299
    Add resolver support for unversioned directories during update/switch.
    Justification:
@@ -30,17 +36,11 @@ Candidate changes:
    Branch:
      ^/subversion/branches/1.10.x-r1846299
    Votes:
-     +1: stsp, brane
+     +1: stsp, brane, rhuijben
 
  * r1847181, r1847182, r1847188, r1847264
    Fix issue SVN-4792: Foreign repo copy of file adding mergeinfo.
    Justification:
      We don't want bogus mergeinfo.
    Votes:
-     +1: julianfoad, brane
-
-Veto-blocked changes:
-=====================
-
-Approved changes:
-=================
+     +1: julianfoad, brane, rhuijben
\ No newline at end of file

Modified: subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c
URL: 
http://svn.apache.org/viewvc/subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c?rev=1847555&r1=1847554&r2=1847555&view=diff
==============================================================================
--- subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c (original)
+++ subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c Tue Nov 27 
15:20:09 2018
@@ -201,7 +201,7 @@ dav_svn__list_report(const dav_resource
   dav_svn__authz_read_baton arb;
   const dav_svn_repos *repos = resource->info->repos;
   int ns;
-  const char *full_path;
+  const char *full_path = NULL;
   svn_boolean_t path_info_only;
   svn_fs_root_t *root;
   svn_depth_t depth = svn_depth_unknown;
@@ -280,6 +280,12 @@ dav_svn__list_report(const dav_resource
       /* else unknown element; skip it */
     }
 
+  if (! full_path)
+    {
+      return dav_svn__new_error_svn(resource->pool, HTTP_BAD_REQUEST, 0, 0,
+                                    "Request was missing the path argument");
+    }
+
   /* Build authz read baton */
   arb.r = resource->info->r;
   arb.repos = resource->info->repos;