From patchwork Tue Jan 19 08:02:27 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: amine ahd X-Patchwork-Id: 569876 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from arrakis.dune.hu (arrakis.dune.hu [78.24.191.176]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id B097E140B8C for ; Tue, 19 Jan 2016 19:02:55 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b=zIEL5DIa; dkim-atps=neutral Received: from arrakis.dune.hu (localhost [127.0.0.1]) by arrakis.dune.hu (Postfix) with ESMTP id 4B6F028C786; Tue, 19 Jan 2016 09:02:00 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on arrakis.dune.hu X-Spam-Level: X-Spam-Status: No, score=-1.5 required=5.0 tests=BAYES_00,FREEMAIL_FROM, T_DKIM_INVALID autolearn=unavailable version=3.3.2 Received: from arrakis.dune.hu (localhost [127.0.0.1]) by arrakis.dune.hu (Postfix) with ESMTP id 9E85628C77D for ; Tue, 19 Jan 2016 09:01:54 +0100 (CET) X-policyd-weight: using cached result; rate: -8.5 Received: from mail-wm0-f43.google.com (mail-wm0-f43.google.com [74.125.82.43]) by arrakis.dune.hu (Postfix) with ESMTPS for ; Tue, 19 Jan 2016 09:01:54 +0100 (CET) Received: by mail-wm0-f43.google.com with SMTP id b14so157160368wmb.1 for ; Tue, 19 Jan 2016 00:02:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=ATM/kQ6JLOrga0aZOES/FlmUoYLCKTQPJhReVnqy5ME=; b=zIEL5DIam9PE++L03/JNmruBVsC8FSKDF4osKxpwxHQrnWGchrKD1oBDKCSW1aKUX6 aY4J8eXYzrG6fSiiBtGYXrcqlBd2T4xjxadb81d6eBKcrFyswN9z7yn1unql4zyrDj+b 3CV4hYvvwnHkWwYBffHyzYEEv9/g1HMyHMY6rGNltsW29XLXNiS7J5AEM93DUIuMiqQu gkwjxtWfQOdhXPre5PAi5/Vyh0NP482Mys/23B90NDpb34pRa3AMC6bKGSyKHx+40mS2 L3eZk0VQiAZFLsOTNHUwdXnyoC/vsmR09GGgw4uMUS+jd2vJzWm4CoPC5NOgPCKSe6TF qbUw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=ATM/kQ6JLOrga0aZOES/FlmUoYLCKTQPJhReVnqy5ME=; b=KCYcNSYDe/P0KjnCsooH/iZ0ctHLpHHGVCJHsU1IRQzA1+VIpGtdMlATKsMpqZ6/80 zXBY8thlkOeZTtIlw7jf2x3F7lPjxa6VNqJCayZlFbfXb5kK+tjIdBNjXMoygg3Og5kc OGX4p1zfn8j/7GacfvI/ILpnVn5dJ6CqYmR2+YeL/o2Mh/yTphn1bwAkpC8TpUIk72TA ivY4dr1Hnsud5Y/oWQt9FzoyKQP65TItFlrF2STt5E0lUzjzNFo/QaLNaeDtLSehfXm/ Xl0Z/iVhkBbbyVCf1HG8z1LIeaSvTMcRQWNsolj9YrdIdcDC8qnJJSnrW4rFMBsFJZTd wQnw== X-Gm-Message-State: AG10YOSY4ZeQkPYdL8fs470VtU6EStNPKh/9KjlOAk7m3EZbE03n50LIkNkW7WELzqxe8A== X-Received: by 10.28.186.87 with SMTP id k84mr17112256wmf.13.1453190553178; Tue, 19 Jan 2016 00:02:33 -0800 (PST) Received: from localhost.localdomain ([5.158.136.57]) by smtp.gmail.com with ESMTPSA id t199sm19218905wmt.23.2016.01.19.00.02.31 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 19 Jan 2016 00:02:31 -0800 (PST) From: amine ahd To: openwrt-devel@lists.openwrt.org Date: Tue, 19 Jan 2016 09:02:27 +0100 Message-Id: <1453190547-5031-1-git-send-email-amine.ahd@gmail.com> X-Mailer: git-send-email 2.5.0 In-Reply-To: <1450800035-15465-1-git-send-email-amine.ahd@gmail.com> References: <1450800035-15465-1-git-send-email-amine.ahd@gmail.com> Subject: [OpenWrt-Devel] [PATCH] use NTP server received via DHCP X-BeenThere: openwrt-devel@lists.openwrt.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: OpenWrt Development List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: openwrt-devel-bounces@lists.openwrt.org Sender: "openwrt-devel" --- package/utils/busybox/Makefile | 3 ++ package/utils/busybox/files/sysntpd | 33 +++++++++++++++--- package/utils/busybox/files/sysntpd.hotplug | 54 +++++++++++++++++++++++++++++ 3 files changed, 86 insertions(+), 4 deletions(-) create mode 100644 package/utils/busybox/files/sysntpd.hotplug diff --git a/package/utils/busybox/Makefile b/package/utils/busybox/Makefile index 5ca4363..3066a85 100644 --- a/package/utils/busybox/Makefile +++ b/package/utils/busybox/Makefile @@ -111,6 +111,9 @@ define Package/busybox/install $(CP) $(PKG_INSTALL_DIR)/* $(1)/ $(INSTALL_BIN) ./files/cron $(1)/etc/init.d/cron $(INSTALL_BIN) ./files/sysntpd $(1)/etc/init.d/sysntpd + $(INSTALL_DIR) $(1)/etc/hotplug.d + $(INSTALL_DIR) $(1)/etc/hotplug.d/iface + $(INSTALL_BIN) ./files/sysntpd.hotplug $(1)/etc/hotplug.d/iface/30-sysntpd $(INSTALL_BIN) ./files/ntpd-hotplug $(1)/usr/sbin/ntpd-hotplug -rm -rf $(1)/lib64 endef diff --git a/package/utils/busybox/files/sysntpd b/package/utils/busybox/files/sysntpd index f73bb83..118c3e3 100755 --- a/package/utils/busybox/files/sysntpd +++ b/package/utils/busybox/files/sysntpd @@ -1,6 +1,9 @@ #!/bin/sh /etc/rc.common # Copyright (C) 2011 OpenWrt.org +. /lib/functions.sh +. /usr/share/libubox/jshn.sh + START=98 USE_PROCD=1 @@ -13,21 +16,43 @@ validate_ntp_section() { } start_service() { - local server enabled enable_server peer - + local server enabled enable_server peer ntpservers iface status ntpserver dump + local dhcp_ifaces="$(uci -q get system.ntp.dhcp_ifaces)" + + config_load system + config_get_bool "use_dhcp" "ntp" "use_dhcp" validate_ntp_section ntp || { echo "validation failed" return 1 } [ $enabled = 0 ] && return - - [ -z "$server" ] && return + [ -z "$server" ] && [ "$use_dhcp" = 0 ] && return procd_open_instance procd_set_param command "$PROG" -n [ "$enable_server" = "1" ] && procd_append_param command -l [ -x "$HOTPLUG_SCRIPT" ] && procd_append_param command -S "$HOTPLUG_SCRIPT" + + [ "$use_dhcp" = 1 ] && { + if [ -z "$dhcp_ifaces" ]; then + dump="$(ubus call network.interface dump)" + ntpservers=$(jsonfilter -s "$dump" -e '$["interface"][*]["data"]["ntpserver"]') + else + for iface in $dhcp_ifaces; do + status="$(ubus call network.interface.$iface status)" + ntpserver=$(jsonfilter -s "$status" -e '$["data"]["ntpserver"]') + [ -n "$ntpserver" ] && \ + ntpservers="$ntpservers $ntpserver" + done + fi + # add this data so we can use it in the sysntpd hotplug script. + procd_set_param data ntp_servers="$ntpservers $server" + for ntpserver in $ntpservers; do + procd_append_param command -p $ntpserver + done + } + for peer in $server; do procd_append_param command -p $peer done diff --git a/package/utils/busybox/files/sysntpd.hotplug b/package/utils/busybox/files/sysntpd.hotplug new file mode 100644 index 0000000..34a2f7a --- /dev/null +++ b/package/utils/busybox/files/sysntpd.hotplug @@ -0,0 +1,54 @@ +#!/bin/sh + +. /lib/functions.sh +. /usr/share/libubox/jshn.sh + +is_valid_interface() { + local list="$(uci get system.ntp.dhcp_ifaces)" + [ -z "$list" ] && return 0 + + case " $list " in + *" $INTERFACE "*) + return 0 + ;; + *) + return 1 + ;; + esac +} + +config_load system +local proto="$(uci get network.$INTERFACE.proto)" +config_get_bool "use_dhcp" "ntp" "use_dhcp" +[ "$use_dhcp" = 1 ] && [ "$ACTION" = ifup ] && is_valid_interface && [ "$proto" = dhcp -o "$proto" = dhcp6 ] || exit 0 + +handle_default_ntp_servers() { + local server="$1" + new_ntp_servers="$new_ntp_servers $server" +} + +local dhcp_ntp_servers iface status ntpserver dump +local dhcp_ifaces="$(uci -q get system.ntp.dhcp_ifaces)" +if [ -z "$dhcp_ifaces" ]; then + dump="$(ubus call network.interface dump)" + dhcp_ntp_servers=$(jsonfilter -s "$dump" -e '$["interface"][*]["data"]["ntpserver"]') +else + for iface in $dhcp_ifaces; do + status="$(ubus call network.interface.$iface status)" + ntpserver=$(jsonfilter -s "$status" -e '$["data"]["ntpserver"]') + [ -n "$ntpserver" ] && \ + dhcp_ntp_servers="$dhcp_ntp_servers $ntpserver" + done +fi + +new_ntp_servers="$dhcp_ntp_servers" +#get the default list of ntp servers from the config file and append it to the new list +config_list_foreach "ntp" "server" handle_default_ntp_servers + +#get the current list of ntp servers in the running instance +local current_ntp_servers=$(ubus call service list '{"name":"sysntpd", "verbose":true}' | jsonfilter -e '$["sysntpd"]["instances"][*]["data"]["ntp_servers"]') +#if its an up action, the iface uses DHCP and the new list of ntp servers is different from the old, restart sysntpd +[ "$current_ntp_servers" != "$new_ntp_servers" ] || exit 0 + +logger -t sysntpd "Reloading sysntpd due to $ACTION of interface $INTERFACE and a change of NTP servers" +/etc/init.d/sysntpd enabled && /etc/init.d/sysntpd reload \ No newline at end of file