Skip to content

Fix PrinterSpooler D-Bus signal spoofing - #1691

Merged
michaelrsweet merged 1 commit into
OpenPrinting:masterfrom
oubayashi:master
Sep 2, 2026
Merged

Fix PrinterSpooler D-Bus signal spoofing#1691
michaelrsweet merged 1 commit into
OpenPrinting:masterfrom
oubayashi:master

Conversation

@oubayashi

Copy link
Copy Markdown
Contributor

I was able to reproduce this locally with a regular, non-root user.
Even though cups.conf says that only root can send the message, the
following command succeeds and is visible to listeners:

dbus-send --system --type=signal /com/redhat/PrinterSpooler com.redhat.PrinterSpooler.PrinterAdded string:fake

It looks like the current rule only allows root, but does not explicitly
deny anyone else. Since the system bus allows broadcast signals by
default, regular users can spoof these notifications too.

The existing policy grants root permission to send messages on the
com.redhat.PrinterSpooler interface, but does not deny unprivileged
clients. Since the system bus permits broadcast signals by default, an
ordinary user can emit a spoofed PrinterSpooler signal despite the
comment claiming that only root can send it.

Deny signals on this interface in the default policy and allow them
again for root. This preserves reception for all connections and limits
the change to signal messages, avoiding an unintended restriction on
other message types.

@michaelrsweet michaelrsweet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@michaelrsweet michaelrsweet self-assigned this Sep 2, 2026
@michaelrsweet michaelrsweet added bug Something isn't working priority-low security Security issue labels Sep 2, 2026
@michaelrsweet michaelrsweet added this to the v2.4.x milestone Sep 2, 2026
@michaelrsweet
michaelrsweet merged commit 16c4f6b into OpenPrinting:master Sep 2, 2026
michaelrsweet added a commit that referenced this pull request Sep 2, 2026
@michaelrsweet

Copy link
Copy Markdown
Member

[2.4.x 606bd5f] Fix D-Bus notifiction policy (Issue #1691)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority-low security Security issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants