On 7/11/07, DAIKI MATSUDA <[EMAIL PROTECTED]> wrote:
Hello, all.

I found a little bug in crm_mon source code (crm/admin/crm_mon.c). It
can create HTML file with --as-html option. But it includes not <br>
tag but <br/> for carriage return code. I think <br> may be correct
and attach the patch. Could you confirm?

no.  those tags should be closed in correctly formed HTML
it just like xml really...


Best Regards
MATSUDA, Daiki

--- crm_mon.c.orig      2007-07-11 15:13:39.000000000 +0900
+++ crm_mon.c   2007-07-11 15:13:56.000000000 +0900
@@ -645,18 +645,18 @@ print_html_status(crm_data_t *cib, const
                time_t now = time(NULL);
                now_str = ctime(&now);
                now_str[24] = EOS; /* replace the newline */
-               fprintf(stream, "Last updated: <b>%s</b><br/>\n", now_str);
+               fprintf(stream, "Last updated: <b>%s</b><br>\n", now_str);
        }

        if(dc == NULL) {
-               fprintf(stream, "Current DC: <font
color=\"red\"><b>NONE</b></font><br/>");
+               fprintf(stream, "Current DC: <font
color=\"red\"><b>NONE</b></font><br>");
        } else {
-               fprintf(stream, "Current DC: %s (%s)<br/>",
+               fprintf(stream, "Current DC: %s (%s)<br>",
                        dc->details->uname, dc->details->id);
        }
-       fprintf(stream, "%d Nodes configured.<br/>",
+       fprintf(stream, "%d Nodes configured.<br>",
                  g_list_length(data_set.nodes));
-       fprintf(stream, "%d Resources configured.<br/>",
+       fprintf(stream, "%d Resources configured.<br>",
                  g_list_length(data_set.resources));

        /*** CONFIG ***/
_______________________________________________________
Linux-HA-Dev: [email protected]
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

_______________________________________________________
Linux-HA-Dev: [email protected]
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to