samba: remove pid at killproc in initscript

Message ID 20201101170608.5259-1-arne_f@ipfire.org
State Accepted
Commit 58b374c255df2b59ecda3814d41cb4bad0d01fa6
Headers
Series samba: remove pid at killproc in initscript |

Commit Message

Arne Fitzenreiter Nov. 1, 2020, 5:06 p.m. UTC
  sometime a stale nmbd or smbd process prevent start of samba.
this change should kill all processes.

Signed-off-by: Arne Fitzenreiter <arne_f@ipfire.org>
---
 lfs/samba                      | 2 +-
 src/initscripts/packages/samba | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)
  

Comments

Michael Tremer Nov. 2, 2020, 10:29 a.m. UTC | #1
This looks okay to me.

Reviewed-by: Michael Tremer <michael.tremer@ipfire.org>

> On 1 Nov 2020, at 17:06, Arne Fitzenreiter <arne_f@ipfire.org> wrote:
> 
> sometime a stale nmbd or smbd process prevent start of samba.
> this change should kill all processes.
> 
> Signed-off-by: Arne Fitzenreiter <arne_f@ipfire.org>
> ---
> lfs/samba                      | 2 +-
> src/initscripts/packages/samba | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/lfs/samba b/lfs/samba
> index 92a6b1121..29dfec2e9 100644
> --- a/lfs/samba
> +++ b/lfs/samba
> @@ -32,7 +32,7 @@ DL_FROM    = $(URL_IPFIRE)
> DIR_APP    = $(DIR_SRC)/$(THISAPP)
> TARGET     = $(DIR_INFO)/$(THISAPP)
> PROG       = samba
> -PAK_VER    = 73
> +PAK_VER    = 74
> 
> DEPS       = avahi cups libtirpc krb5 perl-Parse-Yapp
> 
> diff --git a/src/initscripts/packages/samba b/src/initscripts/packages/samba
> index 2aab39977..fbab2dabe 100644
> --- a/src/initscripts/packages/samba
> +++ b/src/initscripts/packages/samba
> @@ -31,13 +31,13 @@ case "$1" in
> 
> 	stop)
> 		boot_mesg "Stopping smbd..."
> -		killproc -p /var/run/smbd.pid /usr/sbin/smbd
> +		killproc /usr/sbin/smbd
> 
> 		boot_mesg "Stopping nmbd..."
> -		killproc -p /var/run/nmbd.pid /usr/sbin/nmbd
> +		killproc /usr/sbin/nmbd
> 
> 		boot_mesg "Stopping winbind..."
> -		killproc -p /var/run/winbindd.pid /usr/sbin/winbindd
> +		killproc /usr/sbin/winbindd
>                 ;;
> 
> 	reload)
> -- 
> 2.17.1
>
  

Patch

diff --git a/lfs/samba b/lfs/samba
index 92a6b1121..29dfec2e9 100644
--- a/lfs/samba
+++ b/lfs/samba
@@ -32,7 +32,7 @@  DL_FROM    = $(URL_IPFIRE)
 DIR_APP    = $(DIR_SRC)/$(THISAPP)
 TARGET     = $(DIR_INFO)/$(THISAPP)
 PROG       = samba
-PAK_VER    = 73
+PAK_VER    = 74
 
 DEPS       = avahi cups libtirpc krb5 perl-Parse-Yapp
 
diff --git a/src/initscripts/packages/samba b/src/initscripts/packages/samba
index 2aab39977..fbab2dabe 100644
--- a/src/initscripts/packages/samba
+++ b/src/initscripts/packages/samba
@@ -31,13 +31,13 @@  case "$1" in
 
 	stop)
 		boot_mesg "Stopping smbd..."
-		killproc -p /var/run/smbd.pid /usr/sbin/smbd
+		killproc /usr/sbin/smbd
 
 		boot_mesg "Stopping nmbd..."
-		killproc -p /var/run/nmbd.pid /usr/sbin/nmbd
+		killproc /usr/sbin/nmbd
 
 		boot_mesg "Stopping winbind..."
-		killproc -p /var/run/winbindd.pid /usr/sbin/winbindd
+		killproc /usr/sbin/winbindd
                 ;;
 
 	reload)