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

[Michael Tautschnig]

> 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?
> 

I just downloaded a fresh .deb from the Debian servers and its definitely not
screwed up in there. The bug reporter did not provide a Version: pseudo-header,
bug juding from reportbug's info he's using testing aka lenny, and that is
0.94.dfsg.2-1, which I just checked.

We do have the code without the above line in etch(-security), but not in any of
the later versions...

Best,
Michael

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 194 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/pkg-clamav-devel/attachments/20090113/b7e182e3/attachment.pgp