clamav-milter init script fails on pidfile

Bug #363796 reported by Matt LaPlante
6
Affects Status Importance Assigned to Milestone
clamav (Ubuntu)
Fix Released
High
Unassigned
Jaunty
Fix Released
High
Unassigned

Bug Description

Binary package hint: clamav-milter

The latest clamav-milter (clamav-milter_0.95.1+dfsg-1ubuntu1) fails to run from a clean install due to "--pidfile" in the init script:

root@prizm:/tmp# apt-get install clamav-milter
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following extra packages will be installed:
  clamav clamav-base clamav-daemon clamav-freshclam libclamav6
Suggested packages:
  clamav-docs libclamunrar6
The following NEW packages will be installed:
  clamav clamav-base clamav-daemon clamav-freshclam clamav-milter libclamav6
0 upgraded, 6 newly installed, 0 to remove and 33 not upgraded.
Need to get 0B/23.1MB of archives.
After this operation, 25.2MB of additional disk space will be used.
Do you want to continue [Y/n]?
Preconfiguring packages ...
Selecting previously deselected package libclamav6.
(Reading database ... 166036 files and directories currently installed.)
Unpacking libclamav6 (from .../libclamav6_0.95.1+dfsg-1ubuntu1_i386.deb) ...
.....
Setting up clamav-milter (0.95.1+dfsg-1ubuntu1) ...
 * Starting Sendmail milter plugin for ClamAV clamav-milter /usr/sbin/clamav-milter: unrecognized option `--pidfile'
ERROR: Unknown option passed
ERROR: Can't parse command line options
                                                                                                                                                                                 [fail]
.....

root@prizm:/tmp# /etc/init.d/clamav-milter start
 * Starting Sendmail milter plugin for ClamAV clamav-milter /usr/sbin/clamav-milter: unrecognized option `--pidfile'
ERROR: Unknown option passed
ERROR: Can't parse command line options
                                                                                                                                                                                 [fail]
root@prizm:/tmp#

TEST CASE:

sudo apt-get install clamav-milter

Watch it fail:

Setting up clamav-milter (0.95.1+dfsg-1ubuntu1) ...
 * Starting Sendmail milter plugin for ClamAV clamav-milter/usr/sbin/clamav-milter: unrecognized option `--pidfile'
ERROR: Unknown option passed
ERROR: Can't parse command line options
                                                                                                                       [fail]

Install the upgraded packages. Watch it not fail:

Preparing to replace clamav-milter 0.95.1+dfsg-1ubuntu1 (using .../clamav-milter_0.95.1+dfsg-1ubuntu1.1_i386.deb) ...
 * Stopping Sendmail milter plugin for ClamAV clamav-milter [ OK ]
Unpacking replacement clamav-milter ...
Processing triggers for man-db ...
Setting up clamav-milter (0.95.1+dfsg-1ubuntu.1) ...
Installing new version of config file /etc/init.d/clamav-milter ...
Disabling old logrotate script for clamav-milter
 * Starting Sendmail milter plugin for ClamAV clamav-milter [ OK ]

Revision history for this message
Scott Kitterman (kitterman) wrote : Re: [Bug 363796] [NEW] clamav-milter init script fails on pidfile

Thanks. We are planning an early post-relase update due to some other
issues. I'll see what I can do to get this fixed too.

Revision history for this message
Imre Gergely (cemc) wrote :

This one should be fixed in the package uploaded to the clamav PPA (version 0.95.1+dfsg-1ubuntu1+ppa2). Please test it if you can, and update this bugreport. This fix will probably appear in -updates after the Jaunty release.

https://launchpad.net/~ubuntu-clamav/+archive/ppa

And here's a guide on activating the PPA:

https://help.launchpad.net/Packaging/PPA#Installing%20software%20from%20a%20PPA

Thank you.

Revision history for this message
Matt LaPlante (cybrmatt) wrote : RE: [Bug 363796] Re: clamav-milter init script fails on pidfile

PPA version looks good to me for this bug.

Revision history for this message
Scott Kitterman (kitterman) wrote :

Excellent.

@cemc: I'm offline today. Would you please run your changes by sgran or mt.

I'll work on preparing the update tonight or in the morning.

Changed in clamav (Ubuntu):
importance: Undecided → High
milestone: none → jaunty-updates
status: New → In Progress
description: updated
Revision history for this message
Scott Kitterman (kitterman) wrote :

Uploaded for post-release SRU. Note to Ubuntu SRU: Changes were done in coordination with the Debian maintainers and are expected in their next upload too.

Changed in clamav (Ubuntu Jaunty):
status: In Progress → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted clamav into jaunty-proposed-proposed; please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Revision history for this message
Matt LaPlante (cybrmatt) wrote : RE: [Bug 363796] Re: clamav-milter init script fails on pidfile

Looks Good To Me

Revision history for this message
Imre Gergely (cemc) wrote :

Confirmed that the init script is working as expected, no error messages, clamav-milter can be started/stopped.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Matt LaPlante (cybrmatt) wrote :

I have found what I believe is one more bug in this version. For some
reason, after doing a clean install I wind up with / owned by the clamav
user. I don't know that this is a regression or an existing flaw, but
either way it's somewhat bad as it breaks ssh chroots (and maybe other
things...). I can open a new bug if necessary, but it would be nice to get
the fix into this release if possible.

Revision history for this message
Scott Kitterman (kitterman) wrote :

Please open a new bug. There aren't changes in this sru that relate that,
so I'm confident it's not an sru regression.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package clamav - 0.95.1+dfsg-1ubuntu1.1

---------------
clamav (0.95.1+dfsg-1ubuntu1.1) jaunty-proposed; urgency=low

  * Fixed clamd apparmor profile based on pre-release testing
    - added permissions for qpsmtpd, p3scan spools
  * clamav-milter postinst/logrotate fix from pkg-clamav git repo (LP: #363804)
  * clamav-milter init script fix (LP: #363796)

 -- Imre Gergely <email address hidden> Mon, 20 Apr 2009 17:34:28 +0300

Changed in clamav (Ubuntu Jaunty):
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

copied to karmic

Changed in clamav (Ubuntu):
milestone: jaunty-updates → none
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.