Typo in etc/init/vsftpd.conf
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
vsftpd (Ubuntu) |
Fix Released
|
Low
|
Luke Faraone | ||
Lucid |
Fix Released
|
Low
|
Unassigned |
Bug Description
Binary package hint: vsftpd
[ -d /var/run/
Obviously "emply" is not the same as "empty".
IMPACT: very low. vsftpd needs an empty directory for chroot
environment, according to comments in vsftpd.conf:
# This option should be the name of a directory which is empty. Also, the
# directory should not be writable by the ftp user. This directory is used
# as a secure chroot() jail at times vsftpd does not require filesystem
# access.
secure_
The upstart script in /etc/init/
and it created it with the proper permissions IF there is no such
directory. As the script in Lucid has a typo in the checking part,
the condition is never true and the directory gets created no matter
what.
This is fixed in Maverick. Patch is attached for Lucid.
TEST CASE:
There is no visible problem to reproduce. See IMPACT above.
Without the patch the directory creation happens every time, with the
patch it happens ONLY if the directory does not exist.
REGRESSION POTENTIAL: low/none.
NOTE: as this is not a serious bug, it should be SRU'd together with
#648202 which also corrects a bug in init/vsftpd.conf.
Related branches
- Dave Walker (community): Approve
-
Diff: 72 lines (+24/-8)3 files modifieddebian/changelog (+13/-0)
debian/vsftpd.postinst (+8/-5)
debian/vsftpd.upstart (+3/-3)
Changed in vsftpd (Ubuntu): | |
status: | New → Confirmed |
tags: | added: patch |
Changed in vsftpd (Ubuntu): | |
status: | Confirmed → Triaged |
importance: | Undecided → Medium |
importance: | Medium → Low |
Changed in vsftpd (Ubuntu): | |
assignee: | nobody → Luke Faraone (lfaraone) |
status: | Triaged → In Progress |
description: | updated |
description: | updated |
Changed in vsftpd (Ubuntu Lucid): | |
status: | New → Confirmed |
Changed in vsftpd (Ubuntu Lucid): | |
assignee: | nobody → Andres Rodriguez (andreserl) |
status: | Confirmed → In Progress |
Changed in vsftpd (Ubuntu Lucid): | |
importance: | Undecided → Low |
Changed in vsftpd (Ubuntu Lucid): | |
status: | In Progress → Confirmed |
assignee: | Andres Rodriguez (andreserl) → nobody |
importance: | Low → Medium |
importance: | Medium → Low |
tags: |
added: verification-done removed: verification-needed |
tags: | added: testcase |
This debdiff fix typo error and closes this patch.
Thanks!