From: Jim Jagielski <jim@apache.org>
Date: Wed, 27 Feb 2013 22:16:34 +0000 (+0000)
Subject: this is broken... pull out
X-Git-Tag: 2.5.0-alpha~5730
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=441cc2c358d70a66a00d50efc61073688c291423;p=apache

this is broken... pull out


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1450998 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/generators/mod_status.c b/modules/generators/mod_status.c
index 15e96b9574..fe832b32d0 100644
--- a/modules/generators/mod_status.c
+++ b/modules/generators/mod_status.c
@@ -621,7 +621,6 @@ static int status_handler(request_rec *r)
     }
 
     if (ap_extended_status && !short_report) {
-        apr_table_t *vhosts = apr_table_make(r->pool, 10);
         if (no_table_report)
             ap_rputs("<hr /><h2>Server Details</h2>\n\n", r);
         else
@@ -638,10 +637,6 @@ static int status_handler(request_rec *r)
 
         for (i = 0; i < server_limit; ++i) {
             for (j = 0; j < thread_limit; ++j) {
-                char *escvhost;
-                long last_used;
-                const char *vlast;
-
                 ws_record = ap_get_scoreboard_worker_from_indexes(i, j);
 
                 if (ws_record->access_count == 0 &&
@@ -661,22 +656,6 @@ static int status_handler(request_rec *r)
                 if (req_time < 0L)
                     req_time = 0L;
 
-                escvhost = ap_escape_html(r->pool, ws_record->vhost);
-                last_used = (long)apr_time_sec(nowtime - ws_record->last_used);
-                if (escvhost && *escvhost) {
-                    if ((vlast = apr_table_get(vhosts, escvhost)) != NULL) {
-                        long temp = atol(vlast);
-                        if (last_used < temp) {
-                            apr_table_setn(vhosts, apr_pstrdup(r->pool, escvhost),
-                                           apr_psprintf(r->pool, "%ld", last_used));
-                        }
-                    }
-                    else {
-                        apr_table_setn(vhosts, apr_pstrdup(r->pool, escvhost),
-                                           apr_psprintf(r->pool, "%ld", last_used));
-                    }
-                }
-
                 lres = ws_record->access_count;
                 my_lres = ws_record->my_access_count;
                 conn_lres = ws_record->conn_count;
@@ -756,7 +735,8 @@ static int status_handler(request_rec *r)
                                ws_record->times.tms_cutime / tick,
                                ws_record->times.tms_cstime / tick,
 #endif
-                               last_used,
+                               (long)apr_time_sec(nowtime -
+                                                  ws_record->last_used),
                                (long) req_time);
 
                     format_byte_out(r, conn_bytes);
@@ -772,7 +752,8 @@ static int status_handler(request_rec *r)
                                ap_escape_html(r->pool,
                                               ap_escape_logitem(r->pool,
                                                                 ws_record->request)),
-                               escvhost);
+                               ap_escape_html(r->pool,
+                                              ws_record->vhost));
                 }
                 else { /* !no_table_report */
                     if (ws_record->status == SERVER_DEAD)
@@ -841,7 +822,8 @@ static int status_handler(request_rec *r)
                                 ws_record->times.tms_cutime +
                                 ws_record->times.tms_cstime) / tick,
 #endif
-                               last_used,
+                               (long)apr_time_sec(nowtime -
+                                                  ws_record->last_used),
                                (long)req_time);
 
                     ap_rprintf(r, "</td><td>%-1.1f</td><td>%-2.2f</td><td>%-2.2f\n",
@@ -852,7 +834,8 @@ static int status_handler(request_rec *r)
                                   "<td nowrap>%s</td></tr>\n\n",
                                ap_escape_html(r->pool,
                                               ws_record->client),
-                               escvhost,
+                               ap_escape_html(r->pool,
+                                              ws_record->vhost),
                                ap_escape_html(r->pool,
                                               ap_escape_logitem(r->pool,
                                                       ws_record->request)));
@@ -861,10 +844,6 @@ static int status_handler(request_rec *r)
         } /* for (i...) */
 
         if (!no_table_report) {
-            int i;
-            const apr_array_header_t *arr = apr_table_elts(vhosts);
-            const apr_table_entry_t *elts = (const apr_table_entry_t *)arr->elts;
-
             ap_rputs("</table>\n \
 <hr /> \
 <table>\n \
@@ -883,13 +862,6 @@ static int status_handler(request_rec *r)
 <tr><th>Child</th><td>Megabytes transferred this child</td></tr>\n \
 <tr><th>Slot</th><td>Total megabytes transferred this slot</td></tr>\n \
 </table>\n", r);
-            ap_rputs("<hr />\n<table>\n\
-<tr><th>Vhost</th><th>Seconds since last used</th></tr>\n", r);
-            for (i = 0; i < arr->nelts; i++) {
-                ap_rprintf(r, "<tr><td><pre>%s</pre></td><td><pre>%27s</pre></td></tr>\n",
-                           elts[i].key, elts[i].val);
-            }
-            ap_rputs("</table>\n", r);
         }
     } /* if (ap_extended_status && !short_report) */
     else {