🐤 Sync 2026-03-22 23:39:52

luci-app-adguardhome/root/etc/init.d/adguardhome

@@ -8,30 +8,6 @@ STOP=89
 PROG=/usr/bin/AdGuardHome
 USE_PROCD=1
 
-config_cb() {
-	[ $# -eq 0 ] && return
-
-	option_cb() {
-		local option="$1"
-		local value="$2"
-
-		case $option in
-
-		# Support old option names
-		config)
-			option='config_file'
-			;;
-
-		workdir)
-			option='work_dir'
-			;;
-
-		esac
-
-		eval $option="$value"
-	}
-}
-
 boot() {
 	ADGUARDHOME_BOOT=1
 	start "$@"
@@ -146,25 +122,38 @@ start_service() {
 		return 0
 	fi
 
-	local config_file='/etc/adguardhome/adguardhome.yaml'
-	local gc=0
-	local group='adguardhome'
-	local maxprocs=0
-	local memlimit=0
-	local user='adguardhome'
-	local verbose=0
-	local work_dir='/var/lib/adguardhome'
-	local redirect='none'
-	local enabled=0
-
 	config_load 'adguardhome'
 
+	local config_name='config'
+	local config_file config group user verbose work_dir workdir
+	local gc maxprocs memlimit redirect enabled
+
+	uci_validate_section 'adguardhome' 'adguardhome' "$config_name" \
+		'enabled:bool:0' \
+		'redirect:string:none' \
+		'gc:uinteger:0' \
+		'group:string:adguardhome' \
+		'config:string' \
+		'config_file:string:/etc/adguardhome/adguardhome.yaml' \
+		'jail_mount:list(string)' \
+		'jail_mount_rw:list(string)' \
+		'maxprocs:uinteger:0' \
+		'memlimit:uinteger:0' \
+		'user:string:adguardhome' \
+		'verbose:bool:0' \
+		'workdir:string' \
+		'work_dir:string:/var/lib/adguardhome'
+
+	# Compatibility with older configs
+	[ -n "$config" ] && config_file="$config"
+	[ -n "$workdir" ] && work_dir="$workdir"
+
     [ "$enabled" -eq 0 ] && return 1
 
 	local config_dir
 	config_dir=$(dirname "$config_file")
 	if [ "$config_dir" = '/etc' ]; then
-		echo "AdGuard Home config must be stored in its own directory, and not in /etc" >&2
+		echo 'AdGuard Home config must be stored in its own directory, and not in /etc' >&2
 		return 1
 	fi
 	mkdir -m 0700 -p "$config_dir"
@@ -191,24 +180,25 @@ start_service() {
             ;;
     esac
 
-    procd_open_instance
+    procd_open_instance adguardhome
 
 	procd_set_param command "$PROG"
+
 	[ "$gc" -le 0 ] || procd_append_param env GOGC="$gc"
 	[ "$maxprocs" -le 0 ] || procd_append_param env GOMAXPROCS="$maxprocs"
 	[ "$memlimit" -le 0 ] || procd_append_param env GOMEMLIMIT="$memlimit"
+
 	procd_append_param command --config "$config_file"
 	procd_append_param command --logfile syslog
 	procd_append_param command --no-check-update
-	[ "$verbose" = 1 ] && \
-		procd_append_param command --verbose
+	[ "$verbose" = 1 ] && procd_append_param command --verbose
 	procd_append_param command --work-dir "$work_dir"
 
 	procd_set_param stdout 1
 	procd_set_param stderr 1
 	procd_set_param user "$user"
 	procd_set_param group "$group"
-	procd_set_param capabilities /etc/capabilities/adguardhome.json
+	procd_set_param capabilities '/etc/capabilities/adguardhome.json'
 	procd_set_param no_new_privs 1
 	procd_set_param respawn
 
@@ -220,10 +210,10 @@ start_service() {
 	procd_add_jail_mount_rw "$config_dir"
 	procd_add_jail_mount_rw "$work_dir"
 
-	procd_add_jail_mount /etc/hosts
-	procd_add_jail_mount /etc/ssl/certs
-	config_list_foreach config jail_mount procd_add_jail_mount
-	config_list_foreach config jail_mount_rw procd_add_jail_mount_rw
+	procd_add_jail_mount '/etc/hosts'
+	procd_add_jail_mount '/etc/ssl/certs'
+	config_list_foreach "$config_name" jail_mount procd_add_jail_mount
+	config_list_foreach "$config_name" jail_mount_rw procd_add_jail_mount_rw
 
 	procd_close_instance
 }
@@ -257,12 +247,12 @@ reload_service() {
 
 # Service triggers
 service_triggers() {
-    procd_add_reload_trigger "adguardhome"
+    procd_add_reload_trigger adguardhome
     
     # Add interface trigger during boot
     if [ -n "$ADGUARDHOME_BOOT" ]; then
         # Wait for interfaces to be up before starting AdGuard Home
         # Prevents issues like https://github.com/openwrt/packages/issues/21868
-        procd_add_raw_trigger "interface.*.up" 5000 /etc/init.d/adguardhome restart
+        procd_add_raw_trigger 'interface.*.up' 5000 /etc/init.d/adguardhome restart
     fi
 }