Skip to content

dbus-broker: fix CVE-2026-16730#545948

Open
geri1701 wants to merge 1 commit into
NixOS:staging-nixosfrom
geri1701:dbus-broker-cve-2026-16730
Open

dbus-broker: fix CVE-2026-16730#545948
geri1701 wants to merge 1 commit into
NixOS:staging-nixosfrom
geri1701:dbus-broker-cve-2026-16730

Conversation

@geri1701

Copy link
Copy Markdown
Member

Backport both accepted fixes from bus1/dbus-broker#439 for CVE-2026-16730. The first commit accounts peer pidfds to the connecting peer, and the second raises the broker's soft RLIMIT_NOFILE to its hard limit. Both are required to prevent connection pressure from terminating the session bus.

Upstream issue: bus1/dbus-broker#435

Addresses #545679

Assisted-by: pi coding agent / Mika (OpenAI gpt-5.6-sol)

Things done

nixpkgs-branch-check[bot]

This comment was marked as outdated.

@nixpkgs-ci
nixpkgs-ci Bot requested review from peterhoeg and r-vdp July 26, 2026 13:30
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-nixos-tests This PR causes rebuilds for all NixOS tests and should normally target the staging branches. llm-assisted Contributions that include code generation with LLMs labels Jul 26, 2026
@geri1701 geri1701 added 1.severity: security Issues which raise a security issue, or PRs that fix one backport release-26.05 Backport PR automatically labels Jul 26, 2026

@r-vdp r-vdp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we just fetch the two commits separately using the commit hashes instead of using a range like this? That would make it a lot easier to see at a glance which commits we are including exactly.

@geri1701
geri1701 force-pushed the dbus-broker-cve-2026-16730 branch from 9424f4f to 5451e4e Compare July 26, 2026 13:48
@geri1701
geri1701 changed the base branch from master to staging-nixos July 26, 2026 13:48
@nixpkgs-ci nixpkgs-ci Bot closed this Jul 26, 2026
@nixpkgs-ci nixpkgs-ci Bot reopened this Jul 26, 2026
@geri1701
geri1701 force-pushed the dbus-broker-cve-2026-16730 branch from 5451e4e to df01f7b Compare July 26, 2026 13:53
Backport both accepted upstream fixes to account peer pidfds and raise the broker soft RLIMIT_NOFILE to its hard limit.

bus1/dbus-broker#439

Assisted-by: pi coding agent / Mika (OpenAI gpt-5.6-sol)
@nixpkgs-branch-check
nixpkgs-branch-check Bot dismissed their stale review July 26, 2026 14:01

Review dismissed automatically

@geri1701
geri1701 force-pushed the dbus-broker-cve-2026-16730 branch from df01f7b to e64d5b6 Compare July 26, 2026 14:03
@geri1701

Copy link
Copy Markdown
Member Author

Can we just fetch the two commits separately using the commit hashes instead of using a range like this? That would make it a lot easier to see at a glance which commits we are including exactly.

Done, I’ve changed this to fetch each commit separately.

@geri1701 geri1701 removed the backport release-26.05 Backport PR automatically label Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.severity: security Issues which raise a security issue, or PRs that fix one 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-nixos-tests This PR causes rebuilds for all NixOS tests and should normally target the staging branches. llm-assisted Contributions that include code generation with LLMs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants