From: Stephan Hoffmann <sho@relinux.de>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] vsftpd: install sample config file to /etc
Date: Mon, 21 Jan 2013 19:14:05 +0100 [thread overview]
Message-ID: <50FD856D.3020002@relinux.de> (raw)
In-Reply-To: <50FA748B.903@mind.be>
Am 19.01.2013 11:25, schrieb Arnout Vandecappelle:
> On 16/01/13 10:24, Stephan Hoffmann wrote:
>> vsftpd comes with a sample config file that can serve as a starting
>> point for customization and enables a basic functionality. Since
>> this config refers to a directory "/usr/share/empty" this is also
>> created.
>
> I think the point of /usr/share/empty is that it's a directory that
> doesn't exist, so vsftpd will not allow any up/download unless you
> change the configuration file.
Arnout, Peter,
I checked this. Obviously, vsftpd uses this directory to chroot there.
If it isn't present it refuses to work:
> 500 OOPS: vsftpd: not found: directory given in
> 'secure_chroot_dir':/usr/share/empty
At least I admit that my commit message was misleading. So I'll send a
patch to add this;-)
> I think it's a bad idea to default giving access to some random
> location - even if it is read-only non-anonymous access to an empty
> directory.
That is not the case. Default configuration allows anonymous login, but
shows empty directory without write access.
Kind regards
Stephan
>
> But that's just my opinion.
>
> Regards,
> Arnout
>
>>
>> Signed-off-by: Stephan Hoffmann<sho@relinux.de>
>> ---
>> package/vsftpd/vsftpd.mk | 4 ++++
>> 1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/package/vsftpd/vsftpd.mk b/package/vsftpd/vsftpd.mk
>> index ca711bb..f694b87 100644
>> --- a/package/vsftpd/vsftpd.mk
>> +++ b/package/vsftpd/vsftpd.mk
>> @@ -37,6 +37,10 @@ define VSFTPD_INSTALL_TARGET_CMDS
>> test -f $(TARGET_DIR)/etc/init.d/S70vsftpd || \
>> $(INSTALL) -D -m 755 package/vsftpd/vsftpd-init \
>> $(TARGET_DIR)/etc/init.d/S70vsftpd
>> + test -f $(TARGET_DIR)/etc/vsftpd.conf || \
>> + $(INSTALL) -D -m 644 $(@D)/vsftpd.conf \
>> + $(TARGET_DIR)/etc/vsftpd.conf
>> + $(INSTALL) -d -m 755 $(TARGET_DIR)/usr/share/empty
>> endef
>>
>> define VSFTPD_UNINSTALL_TARGET_CMDS
>
>
--
reLinux - Stephan Hoffmann
Am Schmidtgrund 124 50765 K?ln
Tel. +49.221.95595-19 Fax: -64
www.reLinux.de sho at reLinux.de
next prev parent reply other threads:[~2013-01-21 18:14 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-16 9:24 [Buildroot] [PATCH] vsftpd: install sample config file to /etc Stephan Hoffmann
2013-01-19 10:25 ` Arnout Vandecappelle
2013-01-21 18:14 ` Stephan Hoffmann [this message]
2013-01-24 6:39 ` Arnout Vandecappelle
2013-01-20 21:17 ` Peter Korsgaard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50FD856D.3020002@relinux.de \
--to=sho@relinux.de \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.