From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624

Description of problem:
The vsftpd startup script /etc/rc.d/init.d/vsftpd has a couple of
bogus lines in the start() function:
    daemon true
    RETVAL=$?
The first line seems to be there simply to get the '[ OK ]' message
on the screen.  If so, it's better to use the success() function.
The second line is misleading because $? is the return value of the
daemon() function, which will always be 0 here.

The script also uses needlessly complicated indexing to iterate
over the config files where a simple for-loop suffices.

Another minor problem is the message that gets displayed
by these lines:
    site=`basename ${sites[${index}]} .conf`
    echo -n $"Starting $prog for $site: "
Since typically there is only one config file vsftpd.conf, this
results in the slightly baffling message
    Starting vsftpd for vsftpd


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

How reproducible:
Always

Steps to Reproduce:
look at the code   

Additional info:

Attached is a patch to fix all these problems.  I return 0 from
start() because I cannot think of a reliable way to see if
vsftpd started up successfully.