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 CVE-2022-0543 Detector #286

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

Conversation

shpei1963
Copy link

PRP request: #285

Verified on vulnerable redis: docker run --network="host" --rm -p 6379:6379 --name vuln -d vulhub/redis:5.0.7
Redis not vulnerable to this CVE: docker run --network="host" --rm -p 6379:6379 --name vuln -d redis

@shpei1963
Copy link
Author

Any updates for this PR?

@nttran8 nttran8 self-requested a review May 4, 2023 09:16
@nttran8 nttran8 self-assigned this May 4, 2023
@nttran8
Copy link
Collaborator

nttran8 commented May 4, 2023

Hi @shpei1963! Please run https://github.com/google/google-java-format against your Java files before starting the code review, this would greatly reduce review overhead due to linter errors. Thanks! 🙂

@nttran8 nttran8 removed their request for review January 8, 2024 12:41
@nttran8 nttran8 removed their assignment Jan 8, 2024
@tooryx tooryx linked an issue Feb 1, 2024 that may be closed by this pull request
@tooryx tooryx added PRP:Inactive Contributor main The main issue a contributor is working on (top of the contribution queue). labels Feb 1, 2024
@tooryx
Copy link
Member

tooryx commented Feb 2, 2024

Hi @shpei1963,

I see that there was no activity on this PR for a while. Are you still willing to contribute?
Otherwise I will close this PR.

~tooryx

@am0o0
Copy link
Contributor

am0o0 commented Aug 6, 2024

@tooryx I'd like to work on this. this is an important CVE IMO.

@tooryx tooryx self-requested a review August 8, 2024 07:34
@tooryx tooryx reopened this Aug 8, 2024
@tooryx
Copy link
Member

tooryx commented Aug 8, 2024

Hi @am0o0,

The PR is already written so it does not really make sense to attribute it to someone else.
I will modify it myself and get it merged.

~tooryx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contributor main The main issue a contributor is working on (top of the contribution queue). PRP:Inactive
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PRP: Redis CVE-2022-0543 RCE
4 participants