Skip to content

Edit findl4jvulnerabilities #14

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

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

Conversation

ChrisCalv
Copy link

Realized I had the '! Alert:' in one step too high, so during this I added a new message in the section where we are verifying the Jndilookup.class file for that jar,
The logic is if we verify that that file exists two parents up then this is a valid log4j vulnerability, in this case we will write a warning with the '! Alert:' precedent followed by a message that includes the jarfile name that we are evaluating.

The log will appear as
! Alert: The MD5 hash for $jarfile was found in the bad list and the jndilookup.class file was verified to exist, this file needs to be patched.
and $jarfile is equal to the literal path of the file in which we are about to extract.

s-nix and others added 24 commits December 21, 2021 15:02
…of-date

Bump version of log4shell from 1.3.0 to 1.6.1
…om-luna

Suppress Warning Messages from Luna (log4shell) and Improve Logging
…of-get-wmiobject

Replace Get-WMIObject with Get-CimInstance
Moved the log4shell and Yara scanners to Call operators and moved robocopy to same window of PowerShell so that a new console is not opened.
Move Luna and Yara to Call Operator + Robocopy -NoNewWindow Addition
Add '! Alert: ' to Write-log when md5 found in bad list
Move ! Alert to after verification of lookup class
Add the JarFileName to the message.
@s-nix
Copy link

s-nix commented Apr 10, 2023

@stotler Can you please close this PR. @ChrisCalv open it erroneously.

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.

2 participants