aboutsummaryrefslogtreecommitdiffstats
path: root/package/system/ubox
diff options
context:
space:
mode:
authorJohn Crispin <blogic@openwrt.org>2013-11-20 16:56:41 +0000
committerJohn Crispin <blogic@openwrt.org>2013-11-20 16:56:41 +0000
commit50577494080e865ce4758c004d1e9249ab1343bd (patch)
tree561e008fe6d1576328aabd7c178ba9babd44ef30 /package/system/ubox
parent3b9b5d38fa3877a769caa4bcf1ac1e521ebb0023 (diff)
downloadmaster-187ad058-50577494080e865ce4758c004d1e9249ab1343bd.tar.gz
master-187ad058-50577494080e865ce4758c004d1e9249ab1343bd.tar.bz2
master-187ad058-50577494080e865ce4758c004d1e9249ab1343bd.zip
procd: add validation to the log init script
Signed-off-by: John Crispin <blogic@openwrt.org> git-svn-id: svn://svn.openwrt.org/openwrt/trunk@38879 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'package/system/ubox')
-rw-r--r--package/system/ubox/files/log.init33
1 files changed, 23 insertions, 10 deletions
diff --git a/package/system/ubox/files/log.init b/package/system/ubox/files/log.init
index 7676ec379b..a1e910f519 100644
--- a/package/system/ubox/files/log.init
+++ b/package/system/ubox/files/log.init
@@ -8,14 +8,27 @@ STOP=89
USE_PROCD=1
PROG=/sbin/logread
+validate_log_section()
+{
+ uci_validate_section system system "${1}" \
+ 'Password:string' \
+ 'log_size:uinteger' \
+ 'log_ip:ipaddr' \
+ 'log_port:port:514' \
+ 'log_proto:or("tcp", "udp"):udp' \
+ 'log_prefix:string'
+ return $?
+}
+
start_service_file()
{
local pid_file="/var/run/logread.${PIDCOUNT}.pid"
local log_file log_size
- config_get log_file $1 log_file
- config_get log_size $1 log_size
-
+ validate_log_section "${1}" || {
+ echo "validation failed"
+ return 1
+ }
[ -z "${log_file}" ] && return
procd_open_instance
@@ -27,17 +40,16 @@ start_service_file()
start_service_remote()
{
local pid_file="/var/run/logread.${PIDCOUNT}.pid"
- local log_ip log_port log_proto
-
- config_get log_ip $1 log_ip
- config_get log_port $1 log_port
- config_get log_proto $1 log_proto udp
- config_get log_prefix $1 log_prefix
+ local log_ip log_port log_proto log_prefix
+ validate_log_section "${1}" || {
+ echo "validation failed"
+ return 1
+ }
[ -z "${log_ip}" ] && return
procd_open_instance
- procd_set_param command "$PROG" -f -r "$log_ip" "${log_port:-514}" -p "$pid_file"
+ procd_set_param command "$PROG" -f -r "$log_ip" "${log_port}" -p "$pid_file"
[ "${log_proto}" != "udp" ] || procd_append_param command -u
[ -z "${log_prefix}" ] || procd_append_param command -P ${log_prefix}
procd_close_instance
@@ -46,6 +58,7 @@ start_service_remote()
service_triggers()
{
procd_add_config_trigger "system" "/etc/init.d/log" "start"
+ procd_add_validation validate_log_section
}
start_service()