Fix a storage leak in mod_mime_magic. The strdup() call when processing

a request should have been an ap_pstrdup() call.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/1.3.x@90467 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/CHANGES b/src/CHANGES
index bd7adbbd969a696b1f4bc2eadc20b5bfa2631d32..6ca851a1683ccc23f1572e4027e1f8fb14f4b6d3 100644 (file)
--- a/src/CHANGES
+++ b/src/CHANGES
@@ -1,5 +1,8 @@
 Changes with Apache 1.3.21
 
+  *) Fix a storage leak (a strdup() call) in mod_mime_magic.  
+     [Jeff Trawick]
+
   *) We have always used the obsolete/deprecated Netscape syntax
      for our tracking cookies; now the CookieStyle directive
      allows the Webmaster to choose the Netscape, RFC2109, or

diff --git a/src/modules/standard/mod_mime_magic.c b/src/modules/standard/mod_mime_magic.c
index 3f98176ff3579089c4505aa23742a86e10d63df8..e5440f58bc87570fa39576e3bd736c4dbcc18508 100644 (file)
--- a/src/modules/standard/mod_mime_magic.c
+++ b/src/modules/standard/mod_mime_magic.c
@@ -623,7 +623,7 @@ static int magic_rsl_printf(request_rec *r, char *str,...)
     va_end(ap);
 
     /* add the buffer to the list */
-    return magic_rsl_add(r, strdup(buf));
+    return magic_rsl_add(r, ap_pstrdup(r->pool, buf));
 }
 
 /* RSL hook for putchar-type functions */