Checkmarx显示代码有二阶注入的风险

Question

Checkmark扫描了我们的代码，显示这些代码有二阶注入的风险

@SuppressWarnings("unchecked")
public List<Map<String, Object>> findBySQL(String sql, List<ScalarType> types, List<Object> values, Info info) throws ApplicationException {
    try {
        SQLQuery query = currentSession().createSQLQuery(sql);
        if (types != null) {
            for (ScalarType scalar : types) {
                query.addScalar(scalar.getColumn(), scalar.getType());
            }
        }

        if (values != null) {
            for (int i = 0; i < values.size(); i++) {
                query.setParameter(i, values.get(i));
            }
        }
        query.setResultTransformer(Transformers.ALIAS_TO_ENTITY_MAP);
        return query.list();
    } catch (Exception e) {
        throw new ApplicationException(e, info);
    }
}

我们的代码使用preparedStatement来执行sql。但是为什么这些代码仍然存在风险，我该如何修复它呢？

Answer 1

为了解决Checkmarx中的问题，需要验证该sql字符串

Answer 2

Checkmarx会抛出错误，因为您为查询参数设置的值没有针对其类型进行验证。

例如，让我们假设使用准备语句形成的查询如下所示，并且要传递给查询参数的值是'Test'

Select * from XYZ where COL1 = ?

如果您的代码被泄露，并且如果入侵者通过了查询参数中的'Test'或1=1，那么该条件将始终为真，并将返回表中的所有记录。

因此，在执行查询之前，您应该验证所有输入。

希望这有帮助