[SRU] init script pid parsing has failure cases
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
nginx (Debian) |
Fix Released
|
Unknown
|
|||
nginx (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Trusty |
Fix Released
|
Medium
|
Thomas Ward |
Bug Description
[Impact]
* The init script fails to start correctly due to the parsing used by the init file. The init file here now will scan all of nginx.conf and find references to 'pid, rather than the actual pidfile line.
* This means that other directives can cause this to fail. Removing /dev/null redirections from the init script, the failure error is wildly apparent. Using the below test case, the 'failure' shows as this when /dev/null redirections in the init script are removed"
teward@
nginx: invalid option: "~*(crawl|
[Test Case]
There are multiple ways to trigger this, however we'll use the provided test case here.
(TEST CASE 1)
Add this section of code to anywhere in the http { } block of /etc/nginx/
map $http_user_agent $is_bot {
default 0;
~*(
}
With the currently existing package, run `sudo service nginx start` - it should return a Failure state. Or it will silently fail. `pidof nginx` will show no output if it fails.
With the modified code/package, do the same, it will not error out, and will correctly start nginx. `pidof nginx` will show multiple PIDs.
[Regression Potential]
The changes applied originate from Debian, and are in all subsequent package releases currently in supported Ubuntu releases.
Regression potential is likely near-zero.
[Original Description]
After upgrading a server from 12.04 to 14.04 I found that the nginx web server wasn't starting automatically. I could start the server without error from the command line. Running "sudo /etc/init.d/nginx start" would produce no visible error, but the service would not be started. Running "sudo /etc/init.d/nginx restart" produces a visible "fail" error.
The nginx startup script (/etc/init.d/nginx) includes this line:
PID=$(awk -F'[ \t;]+' '/[^#]pid/ {print $2}' /etc/nginx/
This is intended to find this line (or similar) in the nginx.conf file:
pid /var/run/nginx.pid;
My nginx.conf file includes this map directive:
map $http_user_agent $is_bot {
}
The issue is that awk returns the line that includes "spider", which creates an incorrect value for the location of the pid file, and the startup script fails. The version of /etc/init.d/nginx provided with 12.04 did not inlcude this awk command and the startup script worked without error.
$ lsb_release -rd
Description: Ubuntu 14.04 LTS
Release: 14.04
$ sudo nginx -v
nginx version: nginx/1.4.6 (Ubuntu)
$
ProblemType: Bug
DistroRelease: Ubuntu 14.04
Package: nginx 1.4.6-1ubuntu3
ProcVersionSign
Uname: Linux 3.13.0-24-generic x86_64
ApportVersion: 2.14.1-0ubuntu3
Architecture: amd64
Date: Wed Apr 30 13:30:35 2014
Dependencies:
InstallationDate: Installed on 2014-01-30 (90 days ago)
InstallationMedia: Ubuntu-Server 12.04.3 LTS "Precise Pangolin" - Release amd64 (20130820.2)
PackageArchitec
ProcEnviron:
SHELL=/bin/bash
TERM=xterm
PATH=(custom, no user)
LANG=en_US.UTF-8
XDG_RUNTIME_
SourcePackage: nginx
UpgradeStatus: Upgraded to trusty on 2014-04-25 (4 days ago)
Related branches
summary: |
- awk in startup script finds wrong pid + init script pid parsing has failure cases |
tags: | added: needs-upstream-report |
Changed in nginx (Debian): | |
status: | Unknown → New |
Changed in nginx (Debian): | |
status: | New → Fix Committed |
Changed in nginx (Debian): | |
status: | Fix Committed → Fix Released |
description: | updated |
Changed in nginx (Ubuntu Trusty): | |
status: | New → In Progress |
importance: | Undecided → Medium |
assignee: | nobody → Thomas Ward (teward) |
summary: |
- init script pid parsing has failure cases + [SRU] init script pid parsing has failure cases |
Thank you for taking the time to report this bug and helping to make Ubuntu better.
Confirmed in Trusty. This needs checking on Debian, and perhaps submission to the Debian BTS.
Marking Importance: Medium since there's an easy workaround available (modify /etc/init.d/nginx).