Skip to content

[GHSA-qrr6-mg7r-m243] PHPUnit has Argument injection via newline in PHP INI values that are forwarded to child processes#7430

Merged
advisory-database[bot] merged 1 commit intosebastianbergmann/advisory-improvement-7430from
sebastianbergmann-GHSA-qrr6-mg7r-m243
Apr 20, 2026
Merged

[GHSA-qrr6-mg7r-m243] PHPUnit has Argument injection via newline in PHP INI values that are forwarded to child processes#7430
advisory-database[bot] merged 1 commit intosebastianbergmann/advisory-improvement-7430from
sebastianbergmann-GHSA-qrr6-mg7r-m243

Conversation

@sebastianbergmann
Copy link
Copy Markdown

Updates

  • Affected products

Comments
As I configured in GHSA-qrr6-mg7r-m243, only the two versions 12.5.21 and 13.1.5 are affected. Not the version ranges (<= 12.5.21, >= 13.0.0, <= 13.1.5) you published here.

@github
Copy link
Copy Markdown
Collaborator

github commented Apr 18, 2026

Hi there @sebastianbergmann! 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

Copilot AI review requested due to automatic review settings April 18, 2026 08:09
@github-actions github-actions bot changed the base branch from main to sebastianbergmann/advisory-improvement-7430 April 18, 2026 08:10
@sebastianbergmann
Copy link
Copy Markdown
Author

Hi there @sebastianbergmann! A community member has suggested an improvement to your security advisory.

The community member who suggested an improvement is me.

On GHSA-qrr6-mg7r-m243, I see the correct information:

grafik

On GHSA-qrr6-mg7r-m243, I see incorrect information:

grafik

Please correct the information and, if possible, explain how it was possible to publish incorrect information.

Thank you.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the OSV advisory for GHSA-qrr6-mg7r-m243 to reflect that only two discrete PHPUnit versions are affected (12.5.21 and 13.1.5), aligning with the upstream advisory clarification.

Changes:

  • Narrowed the affected ranges to only include 12.5.21 (fixed in 12.5.22) and 13.1.5 (fixed in 13.1.6)
  • Removed database_specific.last_known_affected_version_range entries
  • Bumped the modified timestamp

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@advisory-database
Copy link
Copy Markdown
Contributor

Hi @sebastianbergmann! 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!

@advisory-database advisory-database bot deleted the sebastianbergmann-GHSA-qrr6-mg7r-m243 branch April 20, 2026 18:58
@taladrane
Copy link
Copy Markdown
Collaborator

👋 @sebastianbergmann thank you for letting us know about this! I've gone ahead and corrected the advisory.

if possible, explain how it was possible to publish incorrect information.

this is a result of our curation efforts > when an operator is not included in the vulnerable version range of a repository GitHub Security Advisory (i.e. 12.5.21 as opposed to = 12.5.21), we typically infer it to mean "this version and all prior versions are affected" since that's typically what's meant by many repo GHSA authors when they do not include an operator for a vulnerable version range. the simplest way to prevent this from happening again is to include an explicit operator in your vulnerable version ranges - our documentation covers this quite well! this inference can of course be incorrect at times, and we do our best to verify that prior to publishing, but situations like this are exactly why this feedback repository exists, so thank you for your contribution!

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.

4 participants