Bug 76587
| Summary: | changes to system-auth pam config not processed by vsftpd | ||
|---|---|---|---|
| Product: | [Retired] Red Hat Linux | Reporter: | Jason Corley <jason.corley> |
| Component: | vsftpd | Assignee: | Radek Vokál <rvokal> |
| Status: | CLOSED RAWHIDE | QA Contact: | Mike McLean <mikem> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | 7.3 | CC: | chris.ricker |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2004-08-23 07:58:47 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Jason Corley
2002-10-23 18:56:34 UTC
This *may* be fixed with 1.1.3-1, FWIW.
No, sadly it doesn't fix the pam problem. I rebuilt the srpm from rawhide,
version 1.1.3-2. I found two errors in the 1.1.3-2 spec and the init script.
The first is that /etc/rc.d/init.d/vsftpd is mode 644. The second is a missing
bracket calling an array in the init script. Here are patches to fix both in
case you haven't already done so:
--- vsftpd.init Fri Dec 13 19:28:27 2002
+++ vsftpd.init.new Mon Dec 16 10:43:55 2002
@@ -34,7 +34,7 @@
index=0
while [ "${index}" -lt "${site_count}" ] ; do
- site=`basename $sites[${index}]} .conf`
+ site=`basename ${sites[${index}]} .conf`
echo -n $"Starting $prog for $site: "
( /usr/sbin/vsftpd ${sites[${index}]} &)
daemon true
--- vsftpd.spec Fri Dec 13 19:41:14 2002
+++ vsftpd.spec.new Mon Dec 16 10:17:28 2002
@@ -46,7 +46,7 @@
install -m 644 %{SOURCE2} $RPM_BUILD_ROOT/etc/pam.d/vsftpd
install -m 600 %{SOURCE3} $RPM_BUILD_ROOT/etc/vsftpd.ftpusers
install -m 600 %{SOURCE4} $RPM_BUILD_ROOT/etc/vsftpd.user_list
-install -m 644 %{SOURCE5} $RPM_BUILD_ROOT/etc/init.d/vsftpd
+install -m 755 %{SOURCE5} $RPM_BUILD_ROOT/etc/init.d/vsftpd
%clean
[ "$RPM_BUILD_ROOT" != "/" ] && rm -rf $RPM_BUILD_ROOT
Since the init script looks for *.conf in /etc/vsftpd, would there be any reason to not move vsftpd.user_list and vsftpd.ftpusers into /etc/vsftpd? Just seems more System V-ish IMHO. Jason Since I was adding patches to this anyway, here's one to put all the config
files into /etc/vsftpd (also includes the mode fix for /etc/rc.d/init.d/vsftpd).
--- vsftpd.spec Fri Dec 13 19:41:14 2002
+++ vsftpd.spec.new Mon Dec 16 11:03:59 2002
@@ -44,9 +44,9 @@
install -m 644 vsftpd.8 $RPM_BUILD_ROOT/%{_mandir}/man8/
install -m 644 RedHat/vsftpd.log $RPM_BUILD_ROOT/etc/logrotate.d/vsftpd.log
install -m 644 %{SOURCE2} $RPM_BUILD_ROOT/etc/pam.d/vsftpd
-install -m 600 %{SOURCE3} $RPM_BUILD_ROOT/etc/vsftpd.ftpusers
-install -m 600 %{SOURCE4} $RPM_BUILD_ROOT/etc/vsftpd.user_list
-install -m 644 %{SOURCE5} $RPM_BUILD_ROOT/etc/init.d/vsftpd
+install -m 600 %{SOURCE3} $RPM_BUILD_ROOT/etc/vsftpd/vsftpd.ftpusers
+install -m 600 %{SOURCE4} $RPM_BUILD_ROOT/etc/vsftpd/vsftpd.user_list
+install -m 755 %{SOURCE5} $RPM_BUILD_ROOT/etc/init.d/vsftpd
%clean
[ "$RPM_BUILD_ROOT" != "/" ] && rm -rf $RPM_BUILD_ROOT
@@ -65,7 +65,6 @@
%defattr(-,root,root)
/usr/sbin/vsftpd
/etc/init.d/vsftpd
-%config(noreplace) /etc/vsftpd.*
%dir /etc/vsftpd
%config(noreplace) /etc/vsftpd/*
%config(noreplace) /etc/pam.d/vsftpd
Initial issues are fixed now. *** Bug 85613 has been marked as a duplicate of this bug. *** Should be fixed with vsftpd-2.0.1 |