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

Add additional port 6835/tcp to Apache ActiveMQ OpenWire Transport detection #11237

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ret2src
Copy link

@ret2src ret2src commented Nov 22, 2024

PR Information

This PR adds port 6835/tcp to the detection of Apache ActiveMQ OpenWire Transport services.
Additionally, it improves the regex to use an arbitrary number of whitespace characters between ProviderVersion and the version number. (Update: I've reverted this optimization, connecting to an actual ActiveMQ port showed that we indeed have to match three bytes \t\x00\x06).

Typically, the service runs on port 61616/tcp. However, during three independent penetration tests I've seen the service run on 6835/tcp, over which I was able to successfully exploit CVE-2023-46604.

This PR assumes that PR #4401 works as intended.

Template Validation

I've validated this template locally?

  • YES
  • NO

Using a mock service (since I currently don't have access to an ActiveMQ instance), the detection seems to work as intended:

Verification using a mock service

Update: I've verified this template with ActiveMQ 5.15.6:

Verification with ActiveMQ 5.15.6

Additional Details

I've observed the following patterns during my penetration tests:

$ telnet 192.0.2.3 6835
Trying 192.0.2.3...
Connected to 192.0.2.3.
Escape character is '^]'.
;ActiveMQ
         )
          TcpNoDelayEnabledSizePrefixDisabled	CacheSize
                                                         ProviderName	ActiveMQStackTraceEnabledPlatformDetails	Java
                                                                                                                    CacheEnabledTightEncodingEnabled
                               MaxFrameSize����MaxInactivityDurationu0 MaxInactivityDurationInitalDelay'ProviderVersion	5.15.6

@DhiyaneshGeek DhiyaneshGeek added the Done Ready to merge label Nov 24, 2024
@DhiyaneshGeek DhiyaneshGeek added the good first issue Good for newcomers label Nov 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done Ready to merge good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants