Skip to content

Update VEX file with false positives#173

Open
janhoy wants to merge 3 commits into
mainfrom
Update-vex
Open

Update VEX file with false positives#173
janhoy wants to merge 3 commits into
mainfrom
Update-vex

Conversation

@janhoy

@janhoy janhoy commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

LLM generated, please review.

I tasked Claude Code with running docker scout and analyzing each CVE for exploitability in Solr 10.0. For each CVE that the LLM is fairly certain is a false positive, I told it to update the VEX file. This PR is the result.

Comment thread vex-input.json
"analysis": {
"state": "not_affected",
"justification": "requires_configuration",
"detail": "All five CVEs require non-default Log4j layout or appender configurations that Solr does not use. CVE-2026-34480 affects XmlLayout (Solr uses PatternLayout). CVE-2026-34478 affects Rfc5424Layout with TCP/TLS syslog framing (Solr does not configure a SyslogAppender with TCP framing). CVE-2026-34477 is an incomplete fix for SSL hostname verification in SMTP/Socket/Syslog appenders — Solr does not configure these appenders with TLS. CVE-2026-34479 affects Log4j1XmlLayout in the 1.x bridge (Solr does not use Log4j 1.x XML layout). CVE-2026-34481 affects JsonTemplateLayout when logging MapMessage with attacker-controlled floating-point values — Solr does not use JsonTemplateLayout. Solr's default log configuration uses PatternLayout and does not include any of the affected appender/layout types."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed: these vulnerabilities impact a very small number of users and Solr is not one of them.

I will run the VEX Generator for the remaining ones in the weekend, but the explanations look plausible.

@epugh

epugh commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

@janhoy I update the justification to match out CycloneDX choices. @ppkarwasz i think you vlaidated the change, so I think this could be merged?

@epugh

epugh commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

should this be run for Solr 9.8 or 9.10?

@ppkarwasz

Copy link
Copy Markdown
Member

I only double-checked the Log4j Core vulnerabilities in an IDE, but the rest also looks plausible.

@epugh

epugh commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

This PR is now pretty out of date with the other changes... @janhoy if you think the vex entries that turned up make sense, I'd volunteer to at least get those entries from vex-input.json --> solr/vex/markdown.md files so we can get this to done?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants