[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] Fix #1209 (fwd)

From: Peter N. Lundblad <lundblad_at_linux.nu>
Date: 2004-04-07 16:10:08 CEST

[Sorry if this comes through many times; I seem to have some problems with
my e-mail account]

Hi,

I think this fixes issue 1209.

Regards
//Peter

Fix #1209: dav_svn_deliver() is not doing any HTML or (real) XML
path escaping.

* subversion/mod_dav_svn/repos.c
(dav_svn_deliver): Escape filenames with entities when generating HTML.

Index: subversion/mod_dav_svn/repos.c
===================================================================
--- subversion/mod_dav_svn/repos.c (revision 9287)
+++ subversion/mod_dav_svn/repos.c (working copy)
@@ -2090,7 +2090,11 @@
         if (gen_html)
           name = href;

+ /* We quote special characters in both XML and HTML. */
+ name = apr_xml_quote_string(entry_pool, name, !gen_html);
+
         href = ap_escape_uri(entry_pool, href);
+ href = apr_xml_quote_string(entry_pool, href, 1);

         if (gen_html)
           {
@@ -2101,8 +2105,6 @@
         else
           {
             const char *const tag = (is_dir ? "dir" : "file");
- name = apr_xml_quote_string(entry_pool, name, 1);
- href = apr_xml_quote_string(entry_pool, href, 1);

             /* ### This is where the we could search for props */

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Apr 7 16:03:00 2004

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.