mirror of
https://github.com/Cacti/cacti.git
synced 2026-11-08 21:16:17 +09:00
0001794: html tags on management lists are escaped
git-svn-id: svn+ssh://repo.cacti.net/var/data/svnroot/cacti/cacti/branches/0.8.7@6050 860744bd-22fc-0310-8c96-e9fe5004b5ca
This commit is contained in:
1 parent
d9aea0098f
commit
3c8bd4255c
3 files changed
+9
-6
No files matched your search
+5
-4
@@ -1310,16 +1310,17 @@ function ds() {
|
||||
$i = 0;
|
||||
if (sizeof($data_sources) > 0) {
|
||||
foreach ($data_sources as $data_source) {
|
||||
$data_template_name = ((empty($data_source["data_template_name"])) ? "<em>None</em>" : $data_source["data_template_name"]);
|
||||
$data_input_name = ((empty($data_source["data_input_name"])) ? "<em>External</em>" : $data_source["data_input_name"]);
|
||||
/* we're escaping strings here, so no need to escape them on form_selectable_cell */
|
||||
$data_template_name = ((empty($data_source["data_template_name"])) ? "<em>None</em>" : htmlspecialchars($data_source["data_template_name"]));
|
||||
$data_input_name = ((empty($data_source["data_input_name"])) ? "<em>External</em>" : htmlspecialchars($data_source["data_input_name"]));
|
||||
$poller_interval = ((isset($poller_intervals[$data_source["local_data_id"]])) ? $poller_intervals[$data_source["local_data_id"]] : 0);
|
||||
form_alternate_row_color($colors["alternate"], $colors["light"], $i, 'line' . $data_source["local_data_id"]); $i++;
|
||||
form_selectable_cell("<a class='linkEditMain' href='" . htmlspecialchars("data_sources.php?action=ds_edit&id=" . $data_source["local_data_id"]) . "' title='" . $data_source["name_cache"] . "'>" . ((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", title_trim(htmlspecialchars($data_source["name_cache"]), read_config_option("max_title_data_source"))) : title_trim(htmlspecialchars($data_source["name_cache"]), read_config_option("max_title_data_source"))) . "</a>", $data_source["local_data_id"]);
|
||||
form_selectable_cell($data_source['local_data_id'], $data_source['local_data_id']);
|
||||
form_selectable_cell(((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", htmlspecialchars($data_input_name)) : htmlspecialchars($data_input_name)), $data_source["local_data_id"]);
|
||||
form_selectable_cell(((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", $data_input_name) : $data_input_name), $data_source["local_data_id"]);
|
||||
form_selectable_cell(get_poller_interval($poller_interval), $data_source["local_data_id"]);
|
||||
form_selectable_cell(($data_source['active'] == "on" ? "Yes" : "No"), $data_source["local_data_id"]);
|
||||
form_selectable_cell(((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", htmlspecialchars($data_source['data_template_name'])) : htmlspecialchars($data_source['data_template_name'])), $data_source["local_data_id"]);
|
||||
form_selectable_cell(((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", $data_template_name) : $data_template_name), $data_source["local_data_id"]);
|
||||
form_checkbox_cell($data_source["name_cache"], $data_source["local_data_id"]);
|
||||
form_end_row();
|
||||
}
|
||||
|
||||
+3
-2
@@ -1317,11 +1317,12 @@ function graph() {
|
||||
$i = 0;
|
||||
if (sizeof($graph_list) > 0) {
|
||||
foreach ($graph_list as $graph) {
|
||||
$template_name = ((empty($graph["name"])) ? "<em>None</em>" : $graph["name"]);
|
||||
/* we're escaping strings here, so no need to escape them on form_selectable_cell */
|
||||
$template_name = ((empty($graph["name"])) ? "<em>None</em>" : htmlspecialchars($graph["name"]));
|
||||
form_alternate_row_color($colors["alternate"], $colors["light"], $i, 'line' . $graph["local_graph_id"]); $i++;
|
||||
form_selectable_cell("<a class='linkEditMain' href='" . htmlspecialchars("graphs.php?action=graph_edit&id=" . $graph["local_graph_id"]) . "' title='" . htmlspecialchars($graph["title_cache"]) . "'>" . ((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", title_trim(htmlspecialchars($graph["title_cache"]), read_config_option("max_title_graph"))) : title_trim(htmlspecialchars($graph["title_cache"]), read_config_option("max_title_graph"))) . "</a>", $graph["local_graph_id"]);
|
||||
form_selectable_cell($graph["local_graph_id"], $graph["local_graph_id"]);
|
||||
form_selectable_cell(((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", htmlspecialchars($template_name)) : htmlspecialchars($template_name)), $graph["local_graph_id"]);
|
||||
form_selectable_cell(((get_request_var_request("filter") != "") ? eregi_replace("(" . preg_quote(get_request_var_request("filter")) . ")", "<span style='background-color: #F8D93D;'>\\1</span>", $template_name) : $template_name), $graph["local_graph_id"]);
|
||||
form_selectable_cell($graph["height"] . "x" . $graph["width"], $graph["local_graph_id"]);
|
||||
form_checkbox_cell($graph["title_cache"], $graph["local_graph_id"]);
|
||||
form_end_row();
|
||||
|
||||
@@ -620,6 +620,7 @@ function update_data_query_sort_cache($host_id, $data_query_id) {
|
||||
}
|
||||
|
||||
/* update the cache */
|
||||
/* TODO: if both $sort field and $title_format are empty, this yields funny results */
|
||||
db_execute("update host_snmp_query set sort_field = '$sort_field', title_format = '$title_format' where host_id = '$host_id' and snmp_query_id = '$data_query_id'");
|
||||
}
|
||||
|
||||
|
||||
Reference in new issue
Block a user