Merge branch 'ent-7657-Vulnerabilidad-xss-en-nombre-de-modulos' into 'develop'

fixed xss vulnerability

See merge request artica/pandorafms!4196
This commit is contained in:
Daniel Rodriguez 2021-08-09 08:59:44 +00:00
commit 46b17e5156
3 changed files with 26 additions and 17 deletions

View File

@ -2368,7 +2368,7 @@ switch ($tab) {
var aget_id_os = '<?php echo agents_get_os(modules_get_agentmodule_agent(get_parameter('id_agent_module'))); ?>'; var aget_id_os = '<?php echo agents_get_os(modules_get_agentmodule_agent(get_parameter('id_agent_module'))); ?>';
if('<?php echo html_entity_decode(modules_get_agentmodule_name(get_parameter('id_agent_module'))); ?>' != $('#text-name').val() && if('<?php echo modules_get_agentmodule_name(get_parameter('id_agent_module')); ?>' != $('#text-name').val() &&
'<?php echo agents_get_os(modules_get_agentmodule_agent(get_parameter('id_agent_module'))); ?>' == 19){ '<?php echo agents_get_os(modules_get_agentmodule_agent(get_parameter('id_agent_module'))); ?>' == 19){
event.preventDefault(); event.preventDefault();
@ -2402,7 +2402,7 @@ switch ($tab) {
var module_type_snmp = '<?php echo modules_get_agentmodule_type(get_parameter('id_agent_module')); ?>'; var module_type_snmp = '<?php echo modules_get_agentmodule_type(get_parameter('id_agent_module')); ?>';
if('<?php echo html_entity_decode(modules_get_agentmodule_name(get_parameter('id_agent_module'))); ?>' != $('#text-name').val() && ( if('<?php echo modules_get_agentmodule_name(get_parameter('id_agent_module')); ?>' != $('#text-name').val() && (
module_type_snmp == 15 || module_type_snmp == 16 || module_type_snmp == 17 || module_type_snmp == 18)){ module_type_snmp == 15 || module_type_snmp == 16 || module_type_snmp == 17 || module_type_snmp == 18)){
event.preventDefault(); event.preventDefault();

View File

@ -215,7 +215,7 @@ $table_simple->colspan[3][1] = 3;
$table_simple->data[0][0] = __('Name'); $table_simple->data[0][0] = __('Name');
$table_simple->data[0][1] = html_print_input_text_extended( $table_simple->data[0][1] = html_print_input_text_extended(
'name', 'name',
io_safe_input(html_entity_decode($name, ENT_QUOTES, 'UTF-8')), $name,
'text-name', 'text-name',
'', '',
45, 45,

View File

@ -281,11 +281,11 @@ function agent_changed_by_multiple_agents(event, id_agent, selected) {
$.each(data, function(i, val) { $.each(data, function(i, val) {
var s = js_html_entity_decode(val); var s = js_html_entity_decode(val);
s = s.replace(/"/g, "&quot;").replace(/'/g, "&apos;");
i = i.replace(/"/g, "&quot;").replace(/'/g, "&apos;");
$("#module").append( $("#module").append(
$("<option></option>") $('<option value="' + i + '" title="' + s + '"></option>').text(val)
.html(s)
.attr("value", i)
.attr("title", s)
); );
all_common_modules.push(i); all_common_modules.push(i);
@ -385,12 +385,11 @@ function agent_changed_by_multiple_agents_with_alerts(
} }
} }
jQuery.each(data, function(i, val) { jQuery.each(data, function(i, val) {
s = js_html_entity_decode(val); var s = js_html_entity_decode(val);
$("#module").append(
$("<option></option>") s = s.replace(/"/g, "&quot;").replace(/'/g, "&apos;");
.html(s)
.attr("value", val) $("#module").append($('<option value="' + s + '"></option>').text(val));
);
$("#module").fadeIn("normal"); $("#module").fadeIn("normal");
}); });
if (selected != undefined) $("#module").attr("value", selected); if (selected != undefined) $("#module").attr("value", selected);
@ -481,12 +480,22 @@ function alert_templates_changed_by_multiple_agents_with_alerts(
} }
} }
jQuery.each(data, function(i, val) { jQuery.each(data, function(i, val) {
s = js_html_entity_decode(val); var decoded_val = js_html_entity_decode(val);
decoded_val = decoded_val
.replace(/"/g, "&quot;")
.replace(/'/g, "&apos;");
$("#module").append( $("#module").append(
$("<option></option>") $(
.html(s) '<option value="' +
.attr("value", val) decoded_val +
'" title="' +
decoded_val +
'"></option>'
).text(val)
); );
$("#module").fadeIn("normal"); $("#module").fadeIn("normal");
}); });
if (selected != undefined) $("#module").attr("value", selected); if (selected != undefined) $("#module").attr("value", selected);