[PATCH] resent: fix namespace handling in mod_dav_fs

[Joe Orton <jo...@manyfish.co.uk>]

(If further persuasion is needed, there's a regression test for this bug
in litmus 0.9, http://www.webdav.org/neon/litmus/)

Namespace mappings are being dropped in mod_dav_fs; if any property
values are set which define namespaces these come out mangled in the
PROPFIND response.

PR: 11637
Submitted by: Amit Athavale <am...@persistent.co.in>

--- modules/dav/fs/dbm.c	3 Feb 2003 17:52:57 -0000	1.28
+++ modules/dav/fs/dbm.c	24 Apr 2003 09:51:21 -0000
@@ -599,7 +599,7 @@
     ** we don't know the namespace yet, then add it to the map and to our
     ** table of known namespaces.
     */
-    pmap = apr_palloc(db->pool, namespaces->nelts * sizeof(*pmap));
+    m->ns_map = pmap = apr_palloc(db->pool, namespaces->nelts * sizeof(*pmap));
     for (i = namespaces->nelts, puri = (const char **)namespaces->elts;
          i-- > 0;
          ++puri, ++pmap) {
@@ -627,7 +627,6 @@
         }
     }
 
-    m->ns_map = pmap;
     *mapping = m;
     return NULL;
 }

Re: [PATCH] resent: fix namespace handling in mod_dav_fs

[Jeff Trawick <tr...@attglobal.net>]

Joe Orton wrote:
> (If further persuasion is needed, there's a regression test for this bug
> in litmus 0.9, http://www.webdav.org/neon/litmus/)
> 
> Namespace mappings are being dropped in mod_dav_fs; if any property
> values are set which define namespaces these come out mangled in the
> PROPFIND response.

committed, thanks!