Skip to content

[AutoPR- Security] Patch curl for CVE-2026-3784, CVE-2026-3783, CVE-2026-1965 [MEDIUM]#16207

Merged
kgodara912 merged 3 commits into
microsoft:mainfrom
azurelinux-security:azure-autosec/curl/2.0/1071438
Apr 27, 2026
Merged

[AutoPR- Security] Patch curl for CVE-2026-3784, CVE-2026-3783, CVE-2026-1965 [MEDIUM]#16207
kgodara912 merged 3 commits into
microsoft:mainfrom
azurelinux-security:azure-autosec/curl/2.0/1071438

Conversation

@azurelinux-security
Copy link
Copy Markdown

@azurelinux-security azurelinux-security commented Mar 17, 2026

Auto Patch curl for CVE-2026-3784, CVE-2026-3783, CVE-2026-1965.

Autosec pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=1071438&view=results

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

  • The toolchain has been rebuilt successfully (or no changes were made to it)
  • The toolchain/worker package manifests are up-to-date
  • Any updated packages successfully build (or no packages were changed)
  • Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
  • Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
  • All package sources are available
  • cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
  • LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
  • All source files have up-to-date hashes in the *.signatures.json files
  • sudo make go-tidy-all and sudo make go-test-coverage pass
  • Documentation has been updated to match any changes to the build system
  • Ready to merge

Summary

What does the PR accomplish, why was it needed?

Change Log
Does this affect the toolchain?

YES/NO

Associated issues
  • N/A
Links to CVEs
Test Methodology

@akhila-guruju
Copy link
Copy Markdown

akhila-guruju commented Mar 18, 2026

For CVE-2026-1965:
Backported patch has been taken from ubuntu, CVE-2026-1965-1.patch and CVE-2026-1965-2.patch
Since there is something wrong on the website, the links are not functioning. We can see the diff from here, https://launchpadlibrarian.net/851032327/curl_8.5.0-2ubuntu10.4_8.5.0-2ubuntu10.8.diff.gz
Note: search for CVE-2026-1965-1.patch or url: fix reuse of connections using HTTP Negotiate to navigate to exact diff.

For CVE-2026-3783:
Refactored the test suite according to v8.11.1
crlf="headers" in tests/data/test2006 is a v8.19.0-only test framework feature and will fail for v8.11.1 as the value "headers" is not defined in this version. It has been introduced in v8.17.0
curl-8_19_0/docs/tests/FILEFORMAT.md

We have curl of version 8.11.1, According to the curl test suite file format 8_11_1/tests/FILEFORMAT.md, the crlf only takes one value that is "yes". So, refactored according to v8.11.1

For CVE-2026-3784:
Patch matches with the upstream patch. only one extra comment added by AI.

@akhila-guruju
Copy link
Copy Markdown

resolved merge conflicts

@Kanishk-Bansal Kanishk-Bansal marked this pull request as ready for review April 8, 2026 14:00
@Kanishk-Bansal Kanishk-Bansal requested a review from a team as a code owner April 8, 2026 14:00
@Kanishk-Bansal
Copy link
Copy Markdown

Buddy Build

@akhila-guruju
Copy link
Copy Markdown

Buddy Build has passed.
image

Copy link
Copy Markdown

@suresh-thelkar suresh-thelkar left a comment

Choose a reason for hiding this comment

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

Code changes look good to me. I sign off.

I have also run the full build given below. Please make sure that it runs successfully.
https://dev.azure.com/mariner-org/mariner/_build/results?buildId=1094302&view=results

@Kanishk-Bansal Kanishk-Bansal added the ready-for-stable-review PR has passed initial review and is now ready for a second-level stable maintainer review label Apr 14, 2026
@akhila-guruju
Copy link
Copy Markdown

Full build was not successful. Checking the reason of failure.

@akhila-guruju
Copy link
Copy Markdown

akhila-guruju commented Apr 17, 2026

libnvidia-container is failing with Segmentation fault.
image

Copy link
Copy Markdown

@kgodara912 kgodara912 left a comment

Choose a reason for hiding this comment

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

All the patches almost match with respective upstream references in the changes except, 1965 -> there was a minor follow-up fix compared to launchpad one which is incorporated and mentioned as origin in patch. Full build is successful for arm64 and the partial success state due to push error for amd64 image building didn't allow other stage runs but till package building, it is fine. Buddy build is successful. Currently tests are not running likely due to restriction in pipeline environment to access to/from the upstream repos. LGTM.

@kgodara912 kgodara912 merged commit 82d63a2 into microsoft:main Apr 27, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AutoPR-Security main PR Destined for main Packaging ready-for-stable-review PR has passed initial review and is now ready for a second-level stable maintainer review security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants