客户潜在 XSS 未经适当消毒或验证

Question

我在代码上的检查马克思扫描中收到以下消息。获取文本元素的用户输入。然后，该元素的值在没有经过适当清理或验证的情况下流过代码，并最终在方法中显示给用户。下面是我的代码。可以帮助我如何清理以下场景吗？

脚本如下:

function setSummary(select,summary,lim, summaryHidden, limHidden) {
    if(select[select.selectedIndex].value == '-1') {
        summary.innerHTML = '';
        lim.innerHTML = '';
        summaryHidden.value = '';
        limHidden.value = '';
    } else {
        summary.innerHTML = 'Test1 - '+select[select.selectedIndex].text;
        summaryHidden.value = 'Test2 - '+select[select.selectedIndex].text;
        lim.innerHTML = 'Virtual - '+select[select.selectedIndex].text;
        limHidden.value = 'Virtual - '+select[select.selectedIndex].text;
    }
}

我在下面的行中得到checkmarx漏洞，原因是没有经过适当的清理或验证

summary.innerHTML = 'Test1 - '+select[select.selectedIndex].text;

请帮我一些，如何正确清洁上面的线路。

Answer 1

这是否是可利用的，实际上取决于如何调用该函数的上下文，对象< code>select来自哪里(以及它是如何填充的——这是外部可控制的吗？)

无论如何，这里最简单的解决方案是将值设置为innerText属性，而不是innerHTML，如下所示：

summary.innerText = 'Test1 - ' + select[select.selectedIndex].text;