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

Bug when exporting Reports related to CVE-2021-4104 #191

Open
markus8899 opened this issue Dec 21, 2021 · 7 comments
Open

Bug when exporting Reports related to CVE-2021-4104 #191

markus8899 opened this issue Dec 21, 2021 · 7 comments
Assignees
Labels
bug Something isn't working patch released

Comments

@markus8899
Copy link

hello,

I just noticed that .jar files that are vulnerable to CVE-2021-4104 seem to have a problem when exporting out to .csv or .json.

I run log4j-scan.exe with the parameter --fix --Report-csv --report-json. The console shows that there are findings and that they have been fixed. This is also written to the log file.

If I then run the scan again to check whether everything is OK, the console shows that the files were found but have already been mitigated. However, a different status is written in the report, namely "Potentially vulnerable". This only happens with files that are affected by CVE-2021-4104. All other files are correctly written in the log with "Mitigated".

I can reproduce this on any computer with this CVE.

Thanks for all your work!
Regards, Markus

@xeraph xeraph added the bug Something isn't working label Dec 21, 2021
@xeraph xeraph self-assigned this Dec 21, 2021
@xeraph
Copy link
Contributor

xeraph commented Dec 21, 2021

@markus8899 Would you test new v2.5.3 release?

@markus8899
Copy link
Author

It now works perfectly. Thank you very much!

@xeraph
Copy link
Contributor

xeraph commented Dec 21, 2021

@markus8899 Happy to hear that. :D

@markus8899
Copy link
Author

Sorry, I just noticed that CVE-2021-42550 still behaves as described above. Could you please check / fix this?

@markus8899 markus8899 reopened this Dec 22, 2021
@xeraph
Copy link
Contributor

xeraph commented Dec 22, 2021

@markus8899 Scanner does not fix logback. I cannot ensure that it is safe to remove ch/qos/logback/classic/util/JNDIUtil.class. If you do it manually, you would see mitigated report.

@markus8899
Copy link
Author

oh, i see. thanks

@markus8899
Copy link
Author

Hello, thank you for your great work.
i think the bug appears again in version 3.0.1. Anyway, when we check the exported .logs, the classes from CVE-2021-4104 are again marked as vulnerable although the classes have already been deleted from the archives via Logpresso. maybe this can be fixed again.
Thank you!

@markus8899 markus8899 reopened this Mar 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working patch released
Projects
None yet
Development

No branches or pull requests

2 participants