Bug#729704: audit: Init script should depend on $remote_fs for awk

[Laurent Bigonville]

Hello Tyler,

We want auditd to be started as early as possible during the boot,
adding a dependency against $remote_fs is delaying it.

On my machine,

Currently:
bigon@fornost:/etc/rc2.d$ ls -la|grep audit
lrwxrwxrwx.   1 root root16 fév 18  2012 S01auditd -> ../init.d/auditd

With the new dependency against $remote_fs:
bigon@fornost:/etc/rc2.d$ ls -la|grep audit
lrwxrwxrwx.   1 root root16 déc 30 12:20 S17auditd -> ../init.d/auditd

Maybe {g,m}awk should be moved out of /usr instead?

Cheers,

Laurent Bigonville


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Bug#729704: audit: Init script should depend on $remote_fs for awk

[Tyler Hicks]

Package: audit
Version: 1:2.3.2-2
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu trusty ubuntu-patch

Dear Maintainer,

While merging the Debian package into Ubuntu, I was looking into the new
augenrules feature. I noticed that it didn't work when USE_AUGENRULES
was set to "yes" because the init script's PATH does not include
/usr/bin/ but augenrules calls /usr/bin/awk.

In Ubuntu, the attached patch was applied to achieve the following:


  * debian/auditd.init: The start command now requires $remote_fs to be
started because it may call /bin/augenrules, which depends on
/usr/bin/awk. $PATH must also be updated so that augenrules can find awk.


Thanks for considering the patch.


-- System Information:
Debian Release: wheezy/sid
  APT prefers saucy-updates
  APT policy: (500, 'saucy-updates'), (500, 'saucy-security'), (500, 'saucy')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.11.0-12-generic (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
diff -Nru audit-2.3.2/debian/auditd.init audit-2.3.2/debian/auditd.init
--- audit-2.3.2/debian/auditd.init	2013-08-29 01:38:31.0 -0700
+++ audit-2.3.2/debian/auditd.init	2013-11-15 13:52:14.0 -0800
@@ -1,7 +1,7 @@
 #! /bin/sh
 ### BEGIN INIT INFO
 # Provides:  auditd
-# Required-Start:$local_fs
+# Required-Start:$remote_fs
 # Required-Stop: $local_fs
 # Default-Start: 2 3 4 5
 # Default-Stop:  0 1 6
@@ -12,8 +12,7 @@
 # Author: Philipp Matthias Hahn 
 # Based on Debians /etc/init.d/skeleton and Auditds init.d/auditd.init
 
-# PATH should only include /usr/* if it runs after the mountnfs.sh script
-PATH=/sbin:/bin
+PATH=/sbin:/usr/sbin:/bin:/usr/bin
 DESC="audit daemon"
 NAME=auditd
 DAEMON=/sbin/auditd