-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Julian Foad wrote:
[..]
> 
> Or late tonight, in fact.
> 
> It's a good patch. Committed revision 887469.
> 
> Thank you, Kannan.
> 
> I think that fixed the last of the URLs being passed to
> svn_dirent_get_absolute(), so we can now assert that its input is not a
> URL.

  Thank you. Seems the 'assert' is not yet moved to the repo, so sending
  the patch herewith.

  [[[
  Log:
  Assert on the input to `svn_dirent_get_absolute()'.

  * subversion/libsvn_subr/dirent_uri.c
   (svn_dirent_get_absolute): Add an assert statement to capture future
    instances of calling this method on a URL.

  Suggested by: julianfoad
  Patch by: Kannan R <kann...@collab.net>
  ]]]

- --
Thanks & Regards,
Kannan
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEVAwUBSxyepnlTqcY7ytmIAQJRJggArGWJUu3n3ggyUtEVPzglp1jbAasE93vk
0MhuN2huFjKKUNzTFiOkOxI02v4bFc+csPNM0WTfMmr0A7x6uKrYAhIkxhBLkOsP
bfQL3opRmHC47F8wv+7Kr84A1PK4/HtYS+TgjyfIJSbbEzxl4rMbKoQwieSzVh6K
XEV/2QYG6lDO3xIlAo+wD+6q5oNZQSOoV69gXSeZSRyuDpOA4dFn28PezohgcWQN
Z+tR6FNba+hgAINZ92dt54HDf0KnmhxDN73ZzvRivUu1ZoYzPAZ3UvbObcFbT3Wc
i9wziaMNkkuCs0pMPWG8oMBSWHcM3ZsUd7mZ3pVAE2GQGKqZJgW0BA==
=/EP9
-----END PGP SIGNATURE-----
Index: subversion/libsvn_subr/dirent_uri.c
===================================================================
--- subversion/libsvn_subr/dirent_uri.c	(revision 887843)
+++ subversion/libsvn_subr/dirent_uri.c	(working copy)
@@ -1539,6 +1539,8 @@
   char *buffer;
   apr_status_t apr_err;
   const char *path_apr;
+  
+  SVN_ERR_ASSERT(!svn_path_is_url(relative));
 
   /* Merge the current working directory with the relative dirent. */
   SVN_ERR(svn_path_cstring_from_utf8(&path_apr, relative, pool));

Reply via email to