[Pkg-clamav-devel] Bug#511688: Bug#511688: "/etc/init.d/clamav-milter stop" doesn't stop the clamav-milter daemon

[Stephen Gran]

This one time, at band camp, Bruno Tréguier said:
> Stephen Gran wrote :
> 
> >sh -x /etc/init.d/clamav-milter stop
> 
> Sure ! Here you are:

Aha!  If you compare your output:

> root at kayak:~# sh -x /etc/init.d/clamav-milter stop
> + OPTIND=1
> + log_daemon_msg 'Stopping Sendmail milter plugin for ClamAV' clamav-milter
> + '[' -z 'Stopping Sendmail milter plugin for ClamAV' ']'
> + '[' -z clamav-milter ']'
> + echo -n 'Stopping Sendmail milter plugin for ClamAV: clamav-milter'
> Stopping Sendmail milter plugin for ClamAV: clamav-milter+ '[' -n -22983 ']'
> + kill -15 --22983

With the script in our repo:

  stop)
  OPTIND=1
  log_daemon_msg "Stopping $DESC" "$BASENAME"
  if [ -n "$PID" ]; then
    PID=`echo $PID | sed 's/[^0-9]//g'`
    kill -15 -"$PID" 2>/dev/null || true

You're missing the line that does the fixup:
    PID=`echo $PID | sed 's/[^0-9]//g'`

I wonder how that came to be?  Did you not accept a change to the init
script on upgrade at some point?  Is it screwed up in the .deb?  Am I
losing my mind?

Cheers,
-- 
 -----------------------------------------------------------------
|   ,''`.                                            Stephen Gran |
|  : :' :                                        sgran at debian.org |
|  `. `'                        Debian user, admin, and developer |
|    `-                                     http://www.debian.org |
 -----------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-clamav-devel/attachments/20090113/1c7b8a2b/attachment.pgp