Skip to content

feat: check pipewire socket in flatpak scripts#2076

Draft
Commenter25 wants to merge 3 commits intosecureblue:livefrom
Commenter25:pipewire-panic
Draft

feat: check pipewire socket in flatpak scripts#2076
Commenter25 wants to merge 3 commits intosecureblue:livefrom
Commenter25:pipewire-panic

Conversation

@Commenter25
Copy link
Copy Markdown
Contributor

@Commenter25 Commenter25 commented Mar 25, 2026

OUTDATED

i'll update this when stuff i need gets merged dont mind me for now :)

As far as I understand, access to xdg-run/pipewire-0 carries the same risks as the PulseAudio socket. Anything with access can listen to all audio and any microphones. As such, I think it makes sense to check for it in the audit script, and restrict it in flatpak-permissions-lockdown, just like PulseAudio.

@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 1, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@Commenter25 Commenter25 marked this pull request as draft April 1, 2026 18:30
@Commenter25
Copy link
Copy Markdown
Contributor Author

Commenter25 commented Apr 1, 2026

Decided to split this into separate PRs for easier review. Marked as draft for now to make that clear.

@Commenter25
Copy link
Copy Markdown
Contributor Author

Actually, the more I think about it, perhaps PipeWire socket access should be marked as a fail? Or perhaps we need a new state between warning and fail? Unconditional access to video streams feels rather extreme, considering apps should really be using the camera or screencast portals. It's not quite X11 bad, but it's pretty bad. PulseAudio is obviously flawed too, but less access is probably better. Plus, I'm still researching it, but it seems like we might be able to restrict the ability for certain apps using PulseAudio to make record streams, which I'm not sure is possible with direct PipeWire access...

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.

1 participant