From 8c33debb528919abdb11abd7561538abb57b875b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Rafa=C5=82=20Mi=C5=82ecki?= Date: Mon, 30 Dec 2019 07:54:53 +0100 Subject: [PATCH] samba36: log error if getting device info failed MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Signed-off-by: Rafał Miłecki --- package/network/services/samba36/Makefile | 2 +- package/network/services/samba36/files/samba.hotplug | 12 +++++++++--- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/package/network/services/samba36/Makefile b/package/network/services/samba36/Makefile index a4392f62c5a..161f6b45aa9 100644 --- a/package/network/services/samba36/Makefile +++ b/package/network/services/samba36/Makefile @@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=samba PKG_VERSION:=3.6.25 -PKG_RELEASE:=14 +PKG_RELEASE:=15 PKG_SOURCE_URL:=https://download.samba.org/pub/samba \ https://download.samba.org/pub/samba/stable diff --git a/package/network/services/samba36/files/samba.hotplug b/package/network/services/samba36/files/samba.hotplug index e1ae64e655c..abb4346af00 100644 --- a/package/network/services/samba36/files/samba.hotplug +++ b/package/network/services/samba36/files/samba.hotplug @@ -6,7 +6,7 @@ device_get_vars() { json_init json_load "$(ubus call block info)" - json_select devices + json_select devices || return 1 json_get_keys keys for key in $keys @@ -18,13 +18,16 @@ device_get_vars() { shift json_get_vars $@ json_select .. - break + json_select .. + return 0 } json_select .. done json_select .. + + return 2 } [ -f /var/run/config/samba ] || { @@ -32,7 +35,10 @@ device_get_vars() { } [ "$ACTION" = "add" ] && { - device_get_vars $DEVICE label mount + device_get_vars $DEVICE label mount || { + logger -t samba-hotplug "Failed to get $DEVICE info" + exit 1 + } [ -n "$mount" ] && { uci -c /var/run/config batch <<-EOF set samba.$DEVICE="sambashare" -- 2.30.2