diff options
| author | Jo-Philipp Wich <jo@mein.io> | 2021-12-23 17:06:09 +0100 |
|---|---|---|
| committer | Jo-Philipp Wich <jo@mein.io> | 2021-12-23 17:08:21 +0100 |
| commit | 993151504e8e810c083d3257555bdcdc2f00673a (patch) | |
| tree | 011ed8da414f36857a689cc8d95b479d24b6864c | |
| parent | cac0349d26445dc023af69fa8788ca1d2d8f70d4 (diff) | |
luci-base: form.js: do not execute embedded script code in stripTags()
Instead of relying on .innerHTML which executes embedded script code to
parse a given HTML fragment, use dom.parse() which utilizies DOMParser()
internally in order to extract textContent in a safe manner.
Fixes: FS#4199
Ref: https://bugs.openwrt.org/index.php?do=details&task_id=4199
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
| -rw-r--r-- | modules/luci-base/htdocs/luci-static/resources/form.js | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/modules/luci-base/htdocs/luci-static/resources/form.js b/modules/luci-base/htdocs/luci-static/resources/form.js index 312d83605d..23cc0b1cb5 100644 --- a/modules/luci-base/htdocs/luci-static/resources/form.js +++ b/modules/luci-base/htdocs/luci-static/resources/form.js @@ -287,7 +287,8 @@ var CBIAbstractElement = baseclass.extend(/** @lends LuCI.form.AbstractElement.p if (typeof(s) == 'string' && !s.match(/[<>]/)) return s; - var x = E('div', {}, s); + var x = dom.parse('<div>' + s + '</div>'); + return x.textContent || x.innerText || ''; }, |