svn commit: r1453988 - /subversion/branches/1.7.x-issue4332/subversion/libsvn_ra_neon/props.c

[cm...@apache.org]

Author: cmpilato
Date: Thu Mar  7 18:27:11 2013
New Revision: 1453988

URL: http://svn.apache.org/r1453988
Log:
On the '1.7.x-issue4332', follow up to r1453491 by not sending a full
URL to a function that only wants the path portion.

* subversion/libsvn_ra_neon/props.c
  (svn_ra_neon__do_check_path): Send the path portion of the public
    URL to svn_ra_neon__get_starting_props(), not the whole URL.

Modified:
    subversion/branches/1.7.x-issue4332/subversion/libsvn_ra_neon/props.c

Modified: subversion/branches/1.7.x-issue4332/subversion/libsvn_ra_neon/props.c
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x-issue4332/subversion/libsvn_ra_neon/props.c?rev=1453988&r1=1453987&r2=1453988&view=diff
==============================================================================
--- subversion/branches/1.7.x-issue4332/subversion/libsvn_ra_neon/props.c (original)
+++ subversion/branches/1.7.x-issue4332/subversion/libsvn_ra_neon/props.c Thu Mar  7 18:27:11 2013
@@ -1360,6 +1360,23 @@ svn_ra_neon__do_check_path(svn_ra_sessio
       if (! err)
         url = svn_path_url_add_component2(bc_url, bc_relative, pool);
     }
+  else
+    {
+      ne_uri parsed_url;
+
+      /* Split the url into its component pieces (scheme, host, path,
+         etc).  We svn_ra_neon__get_starting_props() wants only the
+         path part. */
+      ne_uri_parse(url, &parsed_url);
+      if (parsed_url.path == NULL)
+        {
+          ne_uri_free(&parsed_url);
+          return svn_error_createf(SVN_ERR_RA_ILLEGAL_URL, NULL,
+                                   _("Neon was unable to parse URL '%s'"), url);
+        }
+      url = apr_pstrdup(pool, parsed_url.path);
+      ne_uri_free(&parsed_url);
+    }
 
   if (! err)
     {