Make daemon socket/PID paths environment-independent; fix rc service HOME

1 files changed, 34 insertions, 4 deletions

diff --git a/src/config.rs b/src/config.rs
index 67cb366..f20fb3a 100644
--- a/src/config.rs
+++ b/src/config.rs
@@ -303,9 +303,39 @@ impl Config {
     }
 
     pub fn get_socket_path() -> PathBuf {
-        let runtime_dir = std::env::var("XDG_RUNTIME_DIR")
-            .map(PathBuf::from)
-            .unwrap_or_else(|_| PathBuf::from("/tmp"));
-        runtime_dir.join("rsplayer").join("daemon.sock")
+        runtime_dir().join("daemon.sock")
     }
 }
+
+/// Directory holding the daemon socket and PID file.
+///
+/// Uses the user's XDG state directory (~/.local/state/rsplayer) so the path
+/// is identical no matter how the daemon was started (rc.d service, login
+/// shell, ...). The previous XDG_RUNTIME_DIR-based path depended on the
+/// environment and broke the service/CLI handshake on FreeBSD, where
+/// XDG_RUNTIME_DIR is often unset (falling back to the world-writable
+/// /tmp/rsplayer).
+pub fn runtime_dir() -> PathBuf {
+    if let Some(base) = directories::BaseDirs::new()
+        && let Some(state) = base.state_dir()
+    {
+        return state.join("rsplayer");
+    }
+    // Fallback when no home/state dir is resolvable
+    std::env::var("XDG_RUNTIME_DIR")
+        .map(PathBuf::from)
+        .unwrap_or_else(|_| PathBuf::from("/tmp"))
+        .join("rsplayer")
+}
+
+/// Create the runtime directory (private, 0700) and return its path.
+pub fn ensure_runtime_dir() -> std::io::Result<PathBuf> {
+    let dir = runtime_dir();
+    std::fs::create_dir_all(&dir)?;
+    #[cfg(unix)]
+    {
+        use std::os::unix::fs::PermissionsExt;
+        std::fs::set_permissions(&dir, std::fs::Permissions::from_mode(0o700))?;
+    }
+    Ok(dir)
+}