clawdie/colibri
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 44

fix(rc.d): use child pidfile + unique procname for colibri-daemon (Sam & Hermes) #17

Merged
clawdie merged 1 commit from fix/rc.d-procname-pidfile into main 2026-06-04 10:38:33 +02:00
Conversation 0 Commits 1 Files changed 1 +10 -8
clawdie commented 2026-06-04 10:38:13 +02:00
Owner
Copy link

Fix three bugs identified in live USB diagnostics (COLIBRI-XFCE-HANDOFF-04.JUN.2026):

  1. procname collision: 'colibri-daemon' instead of '/usr/sbin/daemon'
    so service status finds OUR process, not tailscaled or any other
    daemon(8)-managed service.

  2. pidfile flag: -p (child pidfile) instead of -P (supervisor pidfile)
    so the pidfile holds the colibri-daemon PID, which rc.subr can
    match against the unique procname.

  3. Cascading bypass: fixing procname prevents rc.subr from skipping
    daemon(8) entirely. Process tree should now be:
    rc.d → daemon(8) → colibri-daemon (with crash restart)

Fix three bugs identified in live USB diagnostics (COLIBRI-XFCE-HANDOFF-04.JUN.2026): 1. procname collision: 'colibri-daemon' instead of '/usr/sbin/daemon' so service status finds OUR process, not tailscaled or any other daemon(8)-managed service. 2. pidfile flag: -p (child pidfile) instead of -P (supervisor pidfile) so the pidfile holds the colibri-daemon PID, which rc.subr can match against the unique procname. 3. Cascading bypass: fixing procname prevents rc.subr from skipping daemon(8) entirely. Process tree should now be: rc.d → daemon(8) → colibri-daemon (with crash restart)
clawdie added 1 commit 2026-06-04 10:38:16 +02:00 
Fix three bugs identified in live USB diagnostics (COLIBRI-XFCE-HANDOFF-04.JUN.2026):

1. procname collision: 'colibri-daemon' instead of '/usr/sbin/daemon'
   so service status finds OUR process, not tailscaled or any other
   daemon(8)-managed service.

2. pidfile flag: -p (child pidfile) instead of -P (supervisor pidfile)
   so the pidfile holds the colibri-daemon PID, which rc.subr can
   match against the unique procname.

3. Cascading bypass: fixing procname prevents rc.subr from skipping
   daemon(8) entirely. Process tree should now be:
   rc.d → daemon(8) → colibri-daemon (with crash restart)
clawdie merged commit f3bec17b19 into main 2026-06-04 10:38:33 +02:00
clawdie deleted branch fix/rc.d-procname-pidfile 2026-06-04 10:38:33 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
first-proof blocker

gates the first proven end-to-end

  
hardening

security/robustness follow-up, not a first-proof gate

No labels
first-proof blocker
hardening
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: clawdie/colibri#17
No description provided.
Delete branch "fix/rc.d-procname-pidfile"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?