somebody started to set a function returncode in the validation
stuff and everybody copies it, e.g.
myfunction()
{
	fire_command
	return $?
}
a function automatically returns with the last returncode,
so we can safely remove the command 'return $?'. reference:
http://tldp.org/LDP/abs/html/exit-status.html
"The last command executed in the function or script determines the exit status."
Signed-off-by: Bastian Bittorf <bittorf@bluebottle.com>
SVN-Revision: 42278
		
	
		
			
				
	
	
		
			41 lines
		
	
	
		
			770 B
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			41 lines
		
	
	
		
			770 B
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
#!/bin/sh /etc/rc.common
 | 
						|
# Copyright (C) 2011 OpenWrt.org
 | 
						|
 | 
						|
START=98
 | 
						|
 | 
						|
USE_PROCD=1
 | 
						|
PROG=/usr/sbin/ntpd
 | 
						|
 | 
						|
validate_ntp_section() {
 | 
						|
	uci_validate_section system timeserver "${1}" \
 | 
						|
		'server:list(host)' 'enabled:bool:1' 'enable_server:bool:0'
 | 
						|
}
 | 
						|
 | 
						|
start_service() {
 | 
						|
	local server enabled enable_server peer
 | 
						|
 | 
						|
	validate_ntp_section ntp || {
 | 
						|
		echo "validation failed"
 | 
						|
		return 1
 | 
						|
	}
 | 
						|
 | 
						|
	[ $enabled = 0 ] && return
 | 
						|
 | 
						|
	[ -z "$server" ] && return
 | 
						|
 | 
						|
	procd_open_instance
 | 
						|
	procd_set_param command "$PROG" -n
 | 
						|
	[ "$enable_server" = "1" ] && procd_append_param command -l
 | 
						|
	for peer in $server; do
 | 
						|
		procd_append_param command -p $peer
 | 
						|
	done
 | 
						|
	procd_set_param respawn
 | 
						|
	procd_close_instance
 | 
						|
}
 | 
						|
 | 
						|
service_triggers()
 | 
						|
{
 | 
						|
	procd_add_reload_trigger "system"
 | 
						|
	procd_add_validation validate_ntp_section
 | 
						|
}
 |