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/16 17:03:16 UTC

svn commit: r997777 - /subversion/trunk/subversion/libsvn_client/cleanup.c

Author: cmpilato
Date: Thu Sep 16 15:03:15 2010
New Revision: 997777

URL: http://svn.apache.org/viewvc?rev=997777&view=rev
Log:
Follow-up to r965551 on issue #3620: svn add ^/ triggers assertion
failure

* subversion/libsvn_client/cleanup.c
  (svn_client_upgrade): Check that path isn't a URL.

Patch by: Uwe Stuehler <uw...@bsdx.de>

Modified:
    subversion/trunk/subversion/libsvn_client/cleanup.c

Modified: subversion/trunk/subversion/libsvn_client/cleanup.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/cleanup.c?rev=997777&r1=997776&r2=997777&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/cleanup.c (original)
+++ subversion/trunk/subversion/libsvn_client/cleanup.c Thu Sep 16 15:03:15 2010
@@ -129,6 +129,11 @@ svn_client_upgrade(const char *path,
   info_baton.last_repos = NULL;
   info_baton.last_uuid = NULL;
 
+  if (svn_path_is_url(path))
+    return svn_error_return(svn_error_createf(SVN_ERR_ILLEGAL_TARGET, NULL,
+                                              _("'%s' is not a local path"),
+                                              path));
+
   SVN_ERR(svn_dirent_get_absolute(&local_abspath, path, scratch_pool));
   SVN_ERR(svn_wc_upgrade(ctx->wc_ctx, local_abspath,
                          fetch_repos_info, &info_baton,