字符串输入参数的SSRF CheckMarx漏洞

Question

我正在为我的一个项目运行CheckMarx扫描，它带有一个SSRF漏洞，用于方法的一个输入字符串参数。我的方法如下所示，参数param1抛出了SSRF漏洞。

public String method1(@WebParam(name = "param1") final String param1) {
    LOG.info("Inside method1...")
    if (StringUtils.isBlank(param1) || !StringUtils.isAlphanumeric(param1)) {
        throw new DataManipulationException();
    }
    // Call 3rd party here (method in line 87 below)
}

在该方法中，我使用HttpClient GetMethod调用一个第三方URL，并将param1作为查询字符串参数传递。

CheckMarx的SSRF是：

The application sends a request to a remote server, for some resource, using @DestinationElement in \src\com\test\Test.java:87. However, an attacker can control the target of the request, by sending a URL or other data in param1 at \src\com\test\Test.java:55.

在55号线，我有

public String method1(@WebParam(name = "param1") final String param1) {

在第87行我有

private String processRequest(final GetMethod method) throws IOException {

如果能帮助解决此SSRF漏洞，我们将不胜感激。谢谢你。

Answer 1

要确认SSRF漏洞，还需要processRequest方法的代码片段。据我所知，CheckMarx报告了这个问题，因为“param1”应该包含第三方URL，应用程序正在使用该URL。缓解SSRF风险可能有以下几种可能性：

• 是否需要第三方库来执行html" target="_blank">函数？如果否，那么您可以安全地删除该代码。
• 如果您需要第三方，并且URL保持不变，那么您可以对该URL进行硬编码。
• 如果你不能硬编码，并且它一直在变化，那么你能列出可能的第三方网址吗？所有可能的第三方URL必须手动审查，并执行白名单，以允许所需的URL。或者所有可能的URL都应该插入到后端的文件/数据库中，用户可以传递该特定URL的id。
• 如果URL每次都在变化，并且没有办法白名单，你应该防止这种方法不会进一步伤害应用程序。在这种情况下，需要方法的代码片段，并且应该正确地检查方法。