Bug 232404

Summary: 'error reading non-root config file' when no user configfile is available
Product: Red Hat Enterprise Linux 3 Reporter: Martin (calim) <mh>
Component: vsftpdAssignee: Maros Barabas <mbarabas>
Status: CLOSED ERRATA QA Contact:
Severity: high Docs Contact:
Priority: medium    
Version: 3.8Keywords: Regression
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: RHBA-2007-0425 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-06-11 18:44:40 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 Martin (calim) 2007-03-15 09:15:31 UTC
Description of problem:


Version-Release number of selected component (if applicable):

vsftpd-1.2.1-3E.6
This bug seems to exists on centos 4.3 too.
http://bugs.centos.org/view.php?id=1393

How reproducible:

If i set user_config_dir in vsftpd.conf each user has to have a config file
othervise the connection get's terminated.
This behaviour is definitly wrong and the error message is misleading.

The reason for this behaviour is a bug in file 'twoprocess.c' around line 422

see additional information.


Additional info:

/* Security - ignore unless owned by root */
if (!vsf_sysutil_retval_is_error(retval) &&
    vsf_sysutil_statbuf_get_uid(p_statbuf) == VSFTP_ROOT_UID)
{
  vsf_parseconf_load_file(str_getbuf(&filename_str), 1);
} else {
  die('error reading non-root config file');
}

THE BUG:
The 'else' part (which throws the error) is not only entered if the config file
is not owned by root, it is also entered if the user config file does not exist.

Comment 1 Martin (calim) 2007-03-19 15:49:33 UTC
Hi there !

any news ?

Comment 3 RHEL Program Management 2007-03-21 07:04:33 UTC
This bugzilla has Keywords: Regression.  

Since no regressions are allowed between releases, 
it is also being proposed as a blocker for this release.  

Please resolve ASAP.

Comment 4 Martin (calim) 2007-03-21 15:27:33 UTC
Notice: Just to remind the bug exists in the centos 4 package too. (see the
centos bug report above).

Comment 5 Maros Barabas 2007-03-28 11:27:10 UTC
Hi,

     please try vsftpd-1.2.1-3E.9.

Comment 6 Martin (calim) 2007-03-28 13:30:21 UTC
from where do I get the package ?

Comment 7 Martin (calim) 2007-03-29 13:29:09 UTC
i can test this package if you give me a download link :) I would be happy to
know what changes has been made too ^^

Comment 8 Maros Barabas 2007-04-02 07:41:53 UTC
Hi,
  sorry for delay, here is the link:
    http://people.redhat.com/rbiba/maros/vsftpd-1.2.1-3E.9.src.rpm

Comment 9 Martin (calim) 2007-04-02 08:44:21 UTC
Thank you : Seems ok for the moment. I suggest to wait 24h for eventual side effect.

Comment 10 Martin (calim) 2007-04-02 08:50:01 UTC
Ah, as the problem also exists on centos 4, do you plan to build a RPM for this
release too ?

Comment 11 Maros Barabas 2007-04-02 10:56:33 UTC
Red Hat doesn't support CentOS at all. If you are asking about this bug on rhel
4 I hope that soon.

Comment 12 Martin (calim) 2007-04-02 11:19:32 UTC
Yes sorry, I spoke about rhel4. If you need I can open the same bug report for
rhel 4 (to keep in mind).

Comment 18 Red Hat Bugzilla 2007-06-11 18:44:40 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on the solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2007-0425.html


Comment 19 Martin (calim) 2007-06-13 09:16:42 UTC
Thank you. Is it fixed for RHEL4 too ?