Typo in apache2-maintscript-helper causes MPM check to misfire

Bug #1782806 reported by Tim Bishop on 2018-07-20
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apache2 (Debian)
Fix Released
Unknown
apache2 (Ubuntu)
High
Andreas Hasenack
Bionic
Undecided
Unassigned

Bug Description

The following line appears to have a typo:

a2query -m "$mpm_$MPM" > /dev/null 2>&1 || a2query_ret=$?

It should read:

a2query -m "mpm_$MPM" > /dev/null 2>&1 || a2query_ret=$?

Since $mpm is not defined. Later on there are references to enabling and disabling "mpm_$MPM".

https://salsa.debian.org/apache-team/apache2/blob/master/debian/debhelper/apache2-maintscript-helper#L290

This appears to trip up the Puppet apache module since it creates a prefork module (rather than mpm_prefork), which results in the above query returning a positive response. This is what's happening in bug #1771934.

Fix is obvious and trivial so can hopefully be implemented soon. Appears only to affect bionic since xenial had different code.

Related branches

tags: added: bitesize
Changed in apache2 (Ubuntu):
status: New → Triaged
importance: Undecided → High
Changed in apache2 (Debian):
status: Unknown → New
Changed in apache2 (Debian):
status: New → Fix Released
Changed in apache2 (Ubuntu):
status: Triaged → In Progress
assignee: nobody → Andreas Hasenack (ahasenack)
Andreas Hasenack (ahasenack) wrote :

This is fixed in cosmic.

Changed in apache2 (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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