JanusGraph / JanusGraph/janusgraph

Security Vulnerability in Dom4j (CVE-2018-1000632)

Open
#1,342 3 comments 0 reactions 0 assignees View on GitHub
kind/security
Dominant language
Java
Stars
5.8k
Forks
1.2k
Avg merge
13h 53m
Merged PRs (30d)
6

Description

janusgraph-core, janusgraph-cql and janusgraph-solr indirectly depend on dom4j.

Which the dependency reflections uses.

http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2018-1000632

**Decription**
dom4j version prior to version 2.1.1 contains a CWE-91: XML Injection vulnerability in Class: Element. Methods: addElement, addAttribute that can result in an attacker tampering with XML documents through XML injection. This attack appear to be exploitable via an attacker specifying attributes or elements in the XML document. This vulnerability appears to have been fixed in 2.1.1 or later.

```
[info] dom4j:dom4j:1.6.1
[info] +-org.reflections:reflections:0.9.9-RC1
[info] +-org.janusgraph:janusgraph-core:0.3.1
[info] +-org.janusgraph:janusgraph-cql:0.3.1
[info] +-org.janusgraph:janusgraph-solr:0.3.1
```

Contributor guide

Open the contributing guide

Research direction

Start by running the dependency report for janusgraph-core, janusgraph-cql, and janusgraph-solr to confirm how reflections brings in dom4j 1.6.1. Check the relevant build configuration and verify that the resolved dom4j version is at least 2.1.1 without breaking dependency resolution; rerun the dependency or security checks to confirm the vulnerable version is gone.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
databases, security
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.