add htmlEnode as a renderer to fields which lets you
input arbitrary strings, to prevent html tags

Signed-off-by: Dominik Csapak <d.csa...@proxmox.com>
---
 www/manager/dc/AuthView.js          | 1 +
 www/manager/dc/GroupView.js         | 1 +
 www/manager/dc/PoolView.js          | 1 +
 www/manager/dc/SecurityGroups.js    | 2 +-
 www/manager/dc/UserView.js          | 1 +
 www/manager/grid/FirewallAliases.js | 2 +-
 www/manager/ha/Groups.js            | 3 ++-
 www/manager/ha/Resources.js         | 3 ++-
 www/manager/lxc/SnapshotTree.js     | 2 +-
 www/manager/panel/IPSet.js          | 2 +-
 www/manager/pool/StatusView.js      | 1 +
 www/manager/qemu/Options.js         | 1 +
 www/manager/qemu/SnapshotTree.js    | 2 +-
 13 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/www/manager/dc/AuthView.js b/www/manager/dc/AuthView.js
index 83e79c6..2a2961c 100644
--- a/www/manager/dc/AuthView.js
+++ b/www/manager/dc/AuthView.js
@@ -132,6 +132,7 @@ Ext.define('PVE.dc.AuthView', {
                    header: gettext('Comment'),
                    sortable: false,
                    dataIndex: 'comment',
+                   renderer: Ext.String.htmlEncode,
                    flex: 1
                }
            ],
diff --git a/www/manager/dc/GroupView.js b/www/manager/dc/GroupView.js
index 6950a46..9612721 100644
--- a/www/manager/dc/GroupView.js
+++ b/www/manager/dc/GroupView.js
@@ -95,6 +95,7 @@ Ext.define('PVE.dc.GroupView', {
                {
                    header: gettext('Comment'),
                    sortable: false,
+                   renderer: Ext.String.htmlEncode,
                    dataIndex: 'comment',
                    flex: 1
                }
diff --git a/www/manager/dc/PoolView.js b/www/manager/dc/PoolView.js
index 4ae99e2..0d552ec 100644
--- a/www/manager/dc/PoolView.js
+++ b/www/manager/dc/PoolView.js
@@ -96,6 +96,7 @@ Ext.define('PVE.dc.PoolView', {
                    header: gettext('Comment'),
                    sortable: false,
                    dataIndex: 'comment',
+                   renderer: Ext.String.htmlEncode,
                    flex: 1
                }
            ],
diff --git a/www/manager/dc/SecurityGroups.js b/www/manager/dc/SecurityGroups.js
index 0e31295..d8562e7 100644
--- a/www/manager/dc/SecurityGroups.js
+++ b/www/manager/dc/SecurityGroups.js
@@ -167,7 +167,7 @@ Ext.define('PVE.SecurityGroupList', {
            selModel: sm,
            columns: [
                { header: gettext('Group'), dataIndex: 'group', width: 100 },
-               { header: gettext('Comment'), dataIndex: 'comment', flex: 1 }
+               { header: gettext('Comment'), dataIndex: 'comment', renderer: 
Ext.String.htmlEncode, flex: 1 }
            ],
            listeners: {
                itemdblclick: run_editor,
diff --git a/www/manager/dc/UserView.js b/www/manager/dc/UserView.js
index c4f8a8b..d4ad713 100644
--- a/www/manager/dc/UserView.js
+++ b/www/manager/dc/UserView.js
@@ -220,6 +220,7 @@ Ext.define('PVE.dc.UserView', {
                    id: 'comment',
                    header: gettext('Comment'),
                    sortable: false,
+                   renderer: Ext.String.htmlEncode,
                    dataIndex: 'comment',
                    flex: 1
                }
diff --git a/www/manager/grid/FirewallAliases.js 
b/www/manager/grid/FirewallAliases.js
index 353b97c..2f38393 100644
--- a/www/manager/grid/FirewallAliases.js
+++ b/www/manager/grid/FirewallAliases.js
@@ -171,7 +171,7 @@ Ext.define('PVE.FirewallAliases', {
            columns: [
                { header: gettext('Name'), dataIndex: 'name', width: 100 },
                { header:  gettext('IP/CIDR'), dataIndex: 'cidr', width: 100 },
-               { header: gettext('Comment'), dataIndex: 'comment', flex: 1 }
+               { header: gettext('Comment'), dataIndex: 'comment', renderer: 
Ext.String.htmlEncode, flex: 1 }
            ],
            listeners: {
                itemdblclick: run_editor
diff --git a/www/manager/ha/Groups.js b/www/manager/ha/Groups.js
index e596c00..28be5f7 100644
--- a/www/manager/ha/Groups.js
+++ b/www/manager/ha/Groups.js
@@ -108,7 +108,8 @@ Ext.define('PVE.ha.GroupsView', {
                {
                    header: gettext('Comment'),
                    flex: 1,
-                   dataIndex: 'comment'
+                   dataIndex: 'comment',
+                   renderer: Ext.String.htmlEncode
                }
            ],
            listeners: {
diff --git a/www/manager/ha/Resources.js b/www/manager/ha/Resources.js
index 495f039..87a9723 100644
--- a/www/manager/ha/Resources.js
+++ b/www/manager/ha/Resources.js
@@ -133,7 +133,8 @@ Ext.define('PVE.ha.ResourcesView', {
                {
                    header: gettext('Description'),
                    flex: 1,
-                   dataIndex: 'comment'
+                   dataIndex: 'comment',
+                   renderer: Ext.String.htmlEncode
                }
            ],
            listeners: {
diff --git a/www/manager/lxc/SnapshotTree.js b/www/manager/lxc/SnapshotTree.js
index f13e64f..cbe7043 100644
--- a/www/manager/lxc/SnapshotTree.js
+++ b/www/manager/lxc/SnapshotTree.js
@@ -279,7 +279,7 @@ Ext.define('PVE.lxc.SnapshotTree', {
                        if (record.data.name === 'current') {
                            return gettext("You are here!");
                        } else {
-                           return value;
+                           return Ext.String.htmlEncode(value);
                        }
                    }
                }
diff --git a/www/manager/panel/IPSet.js b/www/manager/panel/IPSet.js
index 2aae7df..821f467 100644
--- a/www/manager/panel/IPSet.js
+++ b/www/manager/panel/IPSet.js
@@ -148,7 +148,7 @@ Ext.define('PVE.IPSetList', {
            selModel: sm,
            columns: [
                { header: 'IPSet', dataIndex: 'name', width: 100 },
-               { header: gettext('Comment'), dataIndex: 'comment', flex: 1 }
+               { header: gettext('Comment'), dataIndex: 'comment', renderer: 
Ext.String.htmlEncode, flex: 1 }
            ],
            listeners: {
                itemdblclick: run_editor,
diff --git a/www/manager/pool/StatusView.js b/www/manager/pool/StatusView.js
index 8049364..a376e09 100644
--- a/www/manager/pool/StatusView.js
+++ b/www/manager/pool/StatusView.js
@@ -13,6 +13,7 @@ Ext.define('PVE.pool.StatusView', {
        var rows = {
            comment: {
                header: gettext('Comment'), 
+               renderer: Ext.String.htmlEncode,
                required: true
            }
        };
diff --git a/www/manager/qemu/Options.js b/www/manager/qemu/Options.js
index c31af13..9ba913c 100644
--- a/www/manager/qemu/Options.js
+++ b/www/manager/qemu/Options.js
@@ -242,6 +242,7 @@ Ext.define('PVE.qemu.Options', {
            smbios1: {
                header: gettext('SMBIOS settings (type1)'),
                defaultValue: '',
+               renderer: Ext.String.htmlEncode,
                editor: caps.vms['VM.Config.HWType'] ? 'PVE.qemu.Smbios1Edit' : 
undefined
            },
            agent: {
diff --git a/www/manager/qemu/SnapshotTree.js b/www/manager/qemu/SnapshotTree.js
index 6f377a2..6efa944 100644
--- a/www/manager/qemu/SnapshotTree.js
+++ b/www/manager/qemu/SnapshotTree.js
@@ -279,7 +279,7 @@ Ext.define('PVE.qemu.SnapshotTree', {
                        if (record.data.name === 'current') {
                            return gettext("You are here!");
                        } else {
-                           return value;
+                           return Ext.String.htmlEncode(value);
                        }
                    }
                }
-- 
2.1.4


_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to