/etc/init.d/samhain: invalid arguments

Bug #1330424 reported by Alex Muntada
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
samhain (Debian)
Fix Released
Unknown
samhain (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

When "/etc/logrotate.d/samhain" runs "/etc/init.d/samhain reload" it raises the following error:

/etc/cron.daily/logrotate:
/etc/init.d/samhain: invalid arguments
error: error running shared postrotate script for '/var/log/samhain/*.log '
run-parts: /etc/cron.daily/logrotate exited with return code 1

The problem is that /etc/init.d/samhain is missing an argument in the call to pidofproc. Just adding "$DAEMON" fixes that:

--- /etc/init.d/samhain 2014-03-29 08:39:04.000000000 +0100
+++ samhain 2014-06-16 11:43:31.979809768 +0200
@@ -35,7 +35,7 @@
      pid=`cat $PIDFILE`
 # No pid, probably no daemon present
      if [ -n "$pid" ] ; then
- pidofproc -p $PIDFILE
+ pidofproc -p $PIDFILE $DAEMON
       return $?
      fi

Also reported upstream.

Changed in samhain (Debian):
status: Unknown → New
Changed in samhain (Debian):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package samhain - 3.1.0-6

---------------
samhain (3.1.0-6) unstable; urgency=medium

  * debian/rules: Add an option to disable the ASM code when building
    on amd64 targets. Work around to fix the FTBFS on amd64 and kfreebsd
    (Closes: #735530)
    Patch provided by Adam Conrad.
  * debian/control: Change build-deps to add autotools-dev. Build on new
    architectures will get config.{sub,guess} automatically without you needing
    to do a fresh upload to update the diff.
    Patch provided by Adam Conrad.
  * debian/samhain.init,debian/postinst: Initialise the integrity database if
    it does not exist the first time the init.d script is run, do not do it in
    postinst. This prevents package installation from sending a mail to root
    which causes problems in automated testing tools (such as piuparts)
    (Closes: #749602)
  * debian/samhain.init: Fix call to pidofproc, provide it with the required
    arguments (Closes: #751747) (LP: #1330424)

 -- Javier Fernández-Sanguino Peña <email address hidden> Wed, 18 Jun 2014 21:39:56 +0200

Changed in samhain (Ubuntu):
status: New → 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.