wsdd2: fix stopping service
authorRafał Miłecki <rafal@milecki.pl>
Thu, 11 May 2023 11:27:32 +0000 (13:27 +0200)
committerRosen Penev <rosenp@gmail.com>
Thu, 11 May 2023 15:21:42 +0000 (18:21 +0300)
Function start_service() is called whenever service may need reloading.
If SMB server is not running it could be simply because it has been
stopped. Reloading service in such case is not an error so:
1. Don't log error as it isn't one
2. Don't exit with error code as it was confusing procd

This change fixes scenario like:
/etc/init.d/ksmbd stop
/etc/init.d/wsdd2 reload
(previously above wasn't stopping wsdd2)

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
net/wsdd2/Makefile
net/wsdd2/files/wsdd2.init

index 1c5c1bd062984e92c169a4bb46585e9fae6de1cb..a10a68cac7d93b7df0ea5889762679dfe14c1e38 100644 (file)
@@ -1,7 +1,7 @@
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=wsdd2
-PKG_RELEASE:=2
+PKG_RELEASE:=3
 
 PKG_SOURCE_PROTO:=git
 PKG_SOURCE_URL:=https://github.com/Netgear/wsdd2.git
index e3df25e2d167e6bdac4c6f381f0ef00fc189e8b3..7b8930e06cb2aa6170106dd59ac638da528c6831 100644 (file)
@@ -29,8 +29,8 @@ start_service() {
        }
 
        [ -z "$SMB_CONF" ] && {
-               logger -p daemon.error -t 'wsdd2' "samba36/4 or ksmbd is not running, can't start wsdd2!"
-               exit 1
+               logger -p daemon.info -t 'wsdd2' "samba36/4 or ksmbd is not running"
+               return
        }
        
        # cleanup comments