ddns-scripts: fix duplication of processes when reloading

When a "service ddns reload" is issued, the ddns processes are being
duplicated.
It seems the culprit is the -1 in the 'killall' command, and removing it
fixes the issue.
The -1 means to use SIGHUP, instead of the default SIGTERM.
In this case, the idea is to kill all other processes, so SIGTERM is
appropriate. In case the processes hang, SIGKILL may be another option.

Signed-off-by: Miguel Angel Mulero Martinez <migmul@gmail.com>
This commit is contained in:
Miguel Angel Mulero Martinez 2024-04-16 15:53:29 +02:00
parent e44aae02d0
commit 525182892b
2 changed files with 2 additions and 2 deletions

View File

@ -8,7 +8,7 @@ include $(TOPDIR)/rules.mk
PKG_NAME:=ddns-scripts
PKG_VERSION:=2.8.2
PKG_RELEASE:=43
PKG_RELEASE:=44
PKG_LICENSE:=GPL-2.0

View File

@ -89,7 +89,7 @@ case "$1" in
exit 1
;;
reload)
killall -1 dynamic_dns_updater.sh 2>/dev/null
killall dynamic_dns_updater.sh 2>/dev/null
exit $?
;;
*) usage_err "unknown command - $1";;