gandalf/secubox-openwrt
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(secubox-core): Prevent duplicate avahi-publish processes in mesh discovery

Browse Source 
Track avahi-publish PID in /tmp/secubox-avahi-mesh.pid and check if process
is still running before spawning a new one. Prevents process accumulation
when discover_peers() is called repeatedly.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
CyberMind-FR 2026-02-06 17:21:07 +01:00
parent 0a3b1dfc6e
commit 301dccec33
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
package/secubox/secubox-core/root/usr/lib/secubox/p2p-mesh.sh
View File

@ -284,9 +284,20 @@ discover_peers() {
echo "$msg" | socat - UDP-DATAGRAM:255.255.255.255:$DISCOVERY_PORT,broadcast
fi
# Also try mDNS if available
# Also try mDNS if available (only start if not already running)
local avahi_pid_file="/tmp/secubox-avahi-mesh.pid"
if command -v avahi-publish >/dev/null; then
# Check if avahi-publish is already running for this service
if [ -f "$avahi_pid_file" ]; then
local old_pid=$(cat "$avahi_pid_file" 2>/dev/null)
if [ -n "$old_pid" ] && kill -0 "$old_pid" 2>/dev/null; then
# Already running, skip
return 0
fi
fi
# Start new avahi-publish and save PID
avahi-publish -s "secubox-mesh-$node_id" _secubox._tcp $MESH_PORT &
echo $! > "$avahi_pid_file"
fi
}