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

[GHSA-x9vc-6hfv-hg8c] Npgsql vulnerable to SQL Injection via Protocol Message Size Overflow #4379

Conversation

NinoFloris
Copy link

Updates

  • Affected products
  • Description
  • References

Comments
I'm a maintainer for Npgsql. It was unclear to us the security vulnerability author provided the template for the advisory, and that we had to edit it. The suggested changes trim down the summary, remove the POCs links and provide the correct affected and patched versions.

@github
Copy link
Collaborator

github commented May 9, 2024

Hi there @paul-gerste-sonarsource and @NinoFloris! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@NinoFloris
Copy link
Author

@advisory-database could you please take a look? Users are opening quite some issues and PRs (7 so far over here) related to this. Our apologies for the mistake, we were unfamiliar with the GitHub process.

@advisory-database advisory-database bot merged commit be0c372 into NinoFloris/advisory-improvement-4379 May 10, 2024
2 checks passed
@advisory-database advisory-database bot deleted the NinoFloris-GHSA-x9vc-6hfv-hg8c branch May 10, 2024 16:46
@advisory-database
Copy link
Contributor

Hi @NinoFloris! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

@Kritner
Copy link

Kritner commented May 10, 2024

any idea how long the change needs to be in place before it'll stop being flagged as a NU1903? I would have thought it'd get checked "each time" but maybe there's some level of caching involved. (I realize the answer is probably not known from this repo's maintainers but...)

image

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

3 participants