Skip to content

fix(security): bump pillow, mlflow, gitpython, cryptography for known… - #1450

Merged
FrankD412 merged 3 commits into
mainfrom
nmailhot/security-bump-cves-main
Oct 25, 2026
Merged

FrankD412 merged 3 commits into
mainfrom
nmailhot/security-bump-cves-main

Conversation

@nv-nmailhot

@nv-nmailhot nv-nmailhot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

… CVEs

  • pillow 12.2.0 -> 12.3.0: fixes 7 CVEs (up to CVSS 8.2, heap OOB writes, decompression-bomb-check bypasses). Base runtime dependency, already fixed on release/0.13.0 (fix(deps): bump pillow to ~=12.3.0 for CVE fixes #1440) -- this forward-ports that fix to main.
  • gitpython 3.1.59 -> 3.1.62: fixes CVE-2026-87817/87818/87819 (incl. an RCE via git directory impersonation, 7.5 ReDoS).
  • cryptography 49.0.0 -> 50.0.1: fixes CVE-2026-69247 (Bleichenbacher oracle in PKCS#7 EnvelopedData decryption). Required bumping mlflow 3.15.1 -> 3.16.1 to lift its cryptography<50 cap.

nltk 3.9.1 is NOT bumped here: lighteval 0.13.0 (latest release) hard-pins nltk==3.9.1 exactly, so nltk can't move without lighteval releasing a new version. Tracked as a follow-up, not blocking.

gitpython/cryptography/mlflow only reach the dependency graph via the accuracy optional extra (lighteval/mlflow chain) and are not installed in the shipped runtime container -- fixed opportunistically since not urgent, but pillow is the one with actual runtime exposure.

Summary by CodeRabbit

  • Chores
    • Updated the Pillow dependency to the 12.3 release series.

… CVEs

- pillow 12.2.0 -> 12.3.0: fixes 7 CVEs (up to CVSS 8.2, heap OOB writes,
  decompression-bomb-check bypasses). Base runtime dependency, already
  fixed on release/0.13.0 (#1440) -- this forward-ports that fix to main.
- gitpython 3.1.59 -> 3.1.62: fixes CVE-2026-87817/87818/87819 (incl. an
  RCE via git directory impersonation, 7.5 ReDoS).
- cryptography 49.0.0 -> 50.0.1: fixes CVE-2026-69247 (Bleichenbacher
  oracle in PKCS#7 EnvelopedData decryption). Required bumping mlflow
  3.15.1 -> 3.16.1 to lift its `cryptography<50` cap.

nltk 3.9.1 is NOT bumped here: lighteval 0.13.0 (latest release) hard-pins
`nltk==3.9.1` exactly, so nltk can't move without lighteval releasing a
new version. Tracked as a follow-up, not blocking.

gitpython/cryptography/mlflow only reach the dependency graph via the
`accuracy` optional extra (lighteval/mlflow chain) and are not installed
in the shipped runtime container -- fixed opportunistically since not
urgent, but pillow is the one with actual runtime exposure.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@nv-nmailhot
nv-nmailhot requested a review from a team as a code owner September 18, 2026 20:34
@github-actions

github-actions Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Try out this PR

Quick install:

pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@1b4baa7c66b057e804d122be17219497a514b438

Recommended with virtual environment (using uv):

uv venv --python 3.12 && source .venv/bin/activate
uv pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@1b4baa7c66b057e804d122be17219497a514b438

Last updated for commit: 1b4baa7 • Browse code

@github-actions github-actions Bot added the fix label Sep 18, 2026
@coderabbitai

coderabbitai Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

Files ignored due to path filters: 1
  • uv.lock is excluded by !**/*.lock
Run configuration
Commits

Reviewing files that changed from the base of the PR and between ecc80db and 1b4baa7.

Files ignored due to path filters: 1
  • uv.lock is excluded by !**/*.lock

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: ai-dynamo/aiperf/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: f0857113-0dab-4ef5-ac91-fb63879bbd07

📥 Commits

Reviewing files that changed from the base of the PR and between 23a63a4 and 60aada1.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • pyproject.toml

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.

Walkthrough

The Pillow dependency constraint in pyproject.toml now targets the 12.3.x release series instead of the 12.2.x series.

Changes

Dependency Update

Layer / File(s) Summary
Pillow version constraint
pyproject.toml
The Pillow requirement changes from ~=12.2.0 to ~=12.3.0.

Priority: High

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: Minimal · up to 60aad

No actionable merge-blocking risk is evident in this dependency-only update.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies a security-related dependency update and names the affected packages. It matches the pull request objectives, including the Pillow, MLflow, GitPython, and cryptography upg…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

A rabbit checks the version line
Pillow hops to three-oh-nine
One small change in the project burrow
Keeps the package path in order
Ears up for the next release
The build now follows the breeze

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Sep 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions Bot added the stale label Oct 22, 2026
@FrankD412

Copy link
Copy Markdown
Contributor

/ok to test 1b4baa7

@FrankD412
FrankD412 merged commit 2602253 into main Oct 25, 2026
44 checks passed
@FrankD412
FrankD412 deleted the nmailhot/security-bump-cves-main branch October 25, 2026 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants