From: Jim Jagielski <jim@apache.org>
Date: Fri, 28 Aug 2009 14:37:37 +0000 (+0000)
Subject: Merge r803704 from trunk:
X-Git-Tag: 2.2.14~91
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=34e74be24544da8b91b51eb0f4a8e227bb8a5e6f;p=thirdparty%2Fapache%2Fhttpd.git

Merge r803704 from trunk:

* Correctly create an empty cell if the description for a file is missing.

PR: 47682
Submitted by: Peter Poeml <poeml suse.de>
Reviewed by: rpluem

Submitted by: rpluem
Reviewed/backported by: jim


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

diff --git a/CHANGES b/CHANGES
index b1a8f75a0f0..2a89d2139ff 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,7 +1,8 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache 2.2.14
 
-
+  *) mod_autoindex: Correctly create an empty cell if the description
+     for a file is missing. PR 47682 [Peter Poeml <poeml suse.de>]
 
 Changes with Apache 2.2.13
 
diff --git a/STATUS b/STATUS
index d22e17fad0b..e10640ff53d 100644
--- a/STATUS
+++ b/STATUS
@@ -89,12 +89,6 @@ RELEASE SHOWSTOPPERS:
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
- * mod_autoindex: Correctly create an empty cell if the description for a
-                  file is missing (PR 47682).
-   Trunk Patch: http://cvs.apache.org/viewvc?view=rev&revision=803704
-   2.2.x Patch: Trunk version of patch works (offset 24 lines)
-   +1: fuankg, poirier, rpluem, jim
-
  * mod_mem_cache: fix seg fault under load due to pool concurrency problem
    PR: 47672
    Trunk Patch: n/a, mod_mem_cache is not in trunk
diff --git a/modules/generators/mod_autoindex.c b/modules/generators/mod_autoindex.c
index 1c38cf28c9b..ef17f66618f 100644
--- a/modules/generators/mod_autoindex.c
+++ b/modules/generators/mod_autoindex.c
@@ -1762,9 +1762,9 @@ static void output_directories(struct ent **ar, int n,
                                                         desc_width), NULL);
                     }
                 }
-            }
-            else {
-                ap_rputs("</td><td>&nbsp;", r);
+                else {
+                    ap_rputs("</td><td>&nbsp;", r);
+                }
             }
             ap_rputs("</td></tr>\n", r);
         }