Skip to content

Document File Manager D-Bus shim security feature#360

Closed
assisted-by-ai wants to merge 1 commit intoKicksecure:masterfrom
assisted-by-ai:claude/build-fm-shim-backend-SMfJx
Closed

Document File Manager D-Bus shim security feature#360
assisted-by-ai wants to merge 1 commit intoKicksecure:masterfrom
assisted-by-ai:claude/build-fm-shim-backend-SMfJx

Conversation

@assisted-by-ai
Copy link
Copy Markdown

Summary

This change adds documentation for the File Manager D-Bus shim, a new security feature that intercepts and controls D-Bus access to the file manager.

Changes

  • Added comprehensive documentation section explaining the File Manager D-Bus shim functionality
  • Documented the two-component architecture:
    • C backend that intercepts org.freedesktop.FileManager1 D-Bus method calls with hardening flags (_FORTIFY_SOURCE=3, PIE, RELRO, CFI)
    • Python/PyQt5 frontend that validates URIs, resolves symlinks, and presents user confirmation dialogs
  • Documented security features including URI validation (local file:// only), rejection of Unicode/control characters, path existence verification, and symlink resolution
  • Listed relevant source files and configuration locations for reference

Details

The documentation explains how the shim prevents applications from silently opening directories via D-Bus by requiring explicit user confirmation. It addresses PCManFM-Qt D-Bus handling issues while providing an additional control layer for users. The backend runs as a systemd user service that automatically starts on login.

https://claude.ai/code/session_01RHBTARXyvjzrgxyCX6ntnA

The fm-shim backend and frontend were not yet documented. Add a new
section describing the D-Bus shim that intercepts FileManager1 calls
and prompts the user before opening directories.

https://claude.ai/code/session_01RHBTARXyvjzrgxyCX6ntnA
Copy link
Copy Markdown
Contributor

@ArrayBolt3 ArrayBolt3 left a comment

Choose a reason for hiding this comment

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

Redone in ArrayBolt3@972f4c9.

by the user to function properly, see notes in
`/etc/security-misc/emerg-shutdown/30_security_misc.conf`.

## File Manager D-Bus shim
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.

Idea accepted, but I'm going to rewrite this, it's both too detailed in places that don't matter as much and not detailed enough in places that matter more.

@adrelanos adrelanos closed this Apr 10, 2026
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.

4 participants