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

[cm...@apache.org]

Author: cmpilato
Date: Thu Mar  7 18:39:01 2013
New Revision: 1453991

URL: http://svn.apache.org/r1453991
Log:
On the '1.7.x-issue4332':

* subversion/libsvn_ra_neon/props.c
  (svn_ra_neon__do_check_path):  Follow up to r1453988, honoring the
    "set-err-rather-than-SVN_ERR()"-ness of this function.

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=1453991&r1=1453990&r2=1453991&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:39:01 2013
@@ -1364,17 +1364,17 @@ svn_ra_neon__do_check_path(svn_ra_sessio
     {
       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. */
+      /* svn_ra_neon__get_starting_props() wants only the path part of URL. */
       ne_uri_parse(url, &parsed_url);
-      if (parsed_url.path == NULL)
+      if (parsed_url.path)
         {
-          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);
+        }
+      else
+        {
+          err = 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);
     }