Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fresh solr 2.9.2 oss #766

Open
wants to merge 58 commits into
base: develop-2.9
Choose a base branch
from

Conversation

jagguli
Copy link

@jagguli jagguli commented Apr 22, 2020

Pull request with yokozuna updated to solr7

Please also see previous PR for discussion #765

llelf and others added 28 commits July 9, 2018 21:56
This reverts commit 0fb1d04d1cbd410d6ec7cb6373fdbcb51f4a96f2.
this is from a bug-hunt (which was in a entirely different place)
XML External Entity attack

1. if HTTP API is exposed:

  - read any file on the system — via /search/extract, the error message leaks
    file content;

  - send HTTP «GET /» request to any host — by PUT/POSTing text/xml document,
    or via /search/extract.  This is also likely riak DoS if the host is
    attacker-controlled.

2. if PB API is exposed

  - send HTTP «GET /» request to any host — by PUT/POST, see above.

Example request:

<?xml version="1.0"?>
<!DOCTYPE meow [
  <!ENTITY xxe2 SYSTEM "/etc/passwd">
  <!ENTITY xxe1 SYSTEM "http://host/ping-me">
]>
<meow>&xxe1;</meow>
It doesn't check user-provided path in any way.
This allows overriding any file on the system with riak permissions.
Security fixes

1. if HTTP API is exposed:

  - read any file on the system — via /search/extract, the error message leaks
    file content;

  - send HTTP «GET /» request to any host — by PUT/POSTing text/xml document,
    or via /search/extract.  This is also likely riak DoS if the host is
    attacker-controlled.

2. if PB API is exposed:

  - send HTTP «GET /» request to any host — by PUT/POST, see above.

3. if HTTP API is exposed:

  - override (with garbage) any file on the system with riak permissions.
@jagguli jagguli mentioned this pull request Apr 22, 2020
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.

None yet

2 participants