求助：修复Checkmarx检测到的Client DOM Stored XSS漏洞

问题场景 #

Ajax请求成功后处理后端返回数据的流程中，被Checkmarx扫描出Client DOM Stored XSS漏洞，尝试将generateGrid函数中的$替换为jQuery后问题仍未解决。

相关代码片段 #

前端Ajax请求函数 #

function getGridData(sURL, sData, sErrorMsg){
    showElement($('#loadingBG')); 
    $.ajax({
        type: "POST",
        url: sURL,
        data:sData, //Data sent to controller//
        dataType:"json",             
        cache: false,
        async:false,
        loadtext: "Loading...", 
        on403: function(t) {
            location.href= contextPath+'/sequencedWP';
        },
        error: function(data, err){
            //alert("AJAX Error: "+err);
            if(data.status==403){
                location.href= contextPath+'/sequencedWP';
            }else{
                alert("AJAX Error: "+JSON.stringify(data));
            }
        },            
        success: function(data){
            isLocked = false;
            if(sURL.indexOf("exportToExcel")== -1){
                $("#sequencedWPSummaryTbl").jqGrid('GridUnload'); //Unload Old Grid//
                $("#sequencedWPTbl").jqGrid('GridUnload'); //Unload Old Grid//
                $("#sequencedWPSummaryTbl1").jqGrid('GridUnload');
                //  $(".ui-pg-input").val(data.filterBean.currentPage);
                sortOrder = data.filterBean.orderByDirection;
                searchParam = data.filterBean.filterParam;
                sortColumnName = data.filterBean.columnName;
                successMsg = data.successMsg;
                searchJsonData = data.filterBean.searchData;
                idsOfSelectedRows = new Array();
                containerList= data.containerList;
                shipWithId = new Array();
                pickWaveId = new Array();
                pickWaveIdAll = new Array();
                wpTypes = new Array();
                statusIds = new Array();
                pickSeqArray = new Array();
                filterData='';
                if (typeof searchJsonData != 'undefined' && ''!=searchJsonData)
                {
                    filterData =  JSON.parse(searchJsonData);
                }
            }
        }
    });
}

涉及漏洞的generateGrid函数 #

function generateGrid(workPackage){   
    var fscList = '{';
    $(containerList).each(function() {
        fscList += this.containerId + ':"' + this.containerTypeCode + '",';
    });
    fscList += '}';

后端返回数据代码 #

model.addAttribute("jsonData",gridData.getJSONString());

response.getWriter().write(gridData.getJSONString());   

Checkmarx报错信息 #

The method generateGrid embeds untrusted data in generated output with $, at line 3 of sequencedWorkPackage.js file. This untrusted data is embedded into the output without proper sanitization or encoding, enabling an attacker to inject malicious code into the generated web-page.

修复方案 #

漏洞根源 #

generateGrid函数中手动拼接JSON字符串，直接将后端返回的containerId和containerTypeCode（不可信数据）拼接到fscList中，未做任何 sanitization 或编码处理，导致攻击者可通过构造恶意数据注入脚本。替换$为jQuery无法解决问题，因为漏洞核心是数据拼接方式而非jQuery别名。

具体修复步骤 #

1. 优先：使用JSON.stringify安全生成JSON字符串
   避免手动拼接，利用原生JSON序列化自动处理特殊字符：

function generateGrid(workPackage){   
    var fscObj = {};
    $(containerList).each(function() {
        fscObj[this.containerId] = this.containerTypeCode;
    });
    var fscList = JSON.stringify(fscObj);
}

2. 备选：字段级手动转义（不推荐）
   若必须手动拼接，需对每个字段进行HTML/JS转义：

// 定义通用转义函数
function escapeJsString(str) {
    return str.replace(/["\\]/g, function(match) {
        return match === '"' ? '\\"' : '\\\\';
    });
}

function generateGrid(workPackage){   
    var fscList = '{';
    $(containerList).each(function() {
        var escapedId = escapeJsString(this.containerId);
        var escapedCode = escapeJsString(this.containerTypeCode);
        fscList += escapedId + ':"' + escapedCode + '",';
    });
    // 移除末尾多余逗号，避免JSON格式错误
    fscList = fscList.slice(0, -1) + '}';
}

3. 后端辅助防护
   确保后端使用可靠的JSON序列化工具（如Jackson、Gson）生成gridData.getJSONString()，避免手动拼接JSON，从源头减少不可信数据的风险。

内容的提问来源于stack exchange，提问作者Chanky