Skip to content

Commit 2e2f1dc

Browse files
author
Marius Burkard
committed
- fixed XSS vulnerability in select2 usage
1 parent 1caeea6 commit 2e2f1dc

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

interface/web/themes/default/assets/javascripts/ispconfig.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -103,13 +103,13 @@ var ISPConfig = {
103103
width: 'element',
104104
selectOnBlur: true,
105105
allowClear: true,
106-
formatResult: function(o) {
107-
if(o.id && $(o.element).parent().hasClass('flags')) return '<span class="flags flag-' + o.id.toLowerCase() + '">' + o.text + '</span>';
108-
else return o.text;
106+
formatResult: function(o, cont, qry, escapeMarkup) {
107+
if(o.id && $(o.element).parent().hasClass('flags')) return '<span class="flags flag-' + o.id.toLowerCase() + '">' + escapeMarkup(o.text) + '</span>';
108+
else return escapeMarkup(o.text);
109109
},
110-
formatSelection: function(o) {
111-
if(o.id && $(o.element).parent().hasClass('flags')) return '<span class="flags flag-' + o.id.toLowerCase() + '">' + o.text + '</span>';
112-
else return o.text;
110+
formatSelection: function(o, cont, escapeMarkup) {
111+
if(o.id && $(o.element).parent().hasClass('flags')) return '<span class="flags flag-' + o.id.toLowerCase() + '">' + escapeMarkup(o.text) + '</span>';
112+
else return escapeMarkup(o.text);
113113
}
114114
}).on('change', function(e) {
115115
if ($("#pageForm .table #Filter").length > 0) {

0 commit comments

Comments
 (0)