Re: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Rich Felker
On Thu, Jun 26, 2014 at 04:47:04PM +0200, Denys Vlasenko wrote: > On Thu, Jun 26, 2014 at 1:37 PM, Morten Kvistgaard > wrote: > >>> ... > >>> execve("proc/self/exe", ["ftpd", "-l", "/"], [/* 9 vars */]) = -1 > >>> ENOENT (No such file or directory) ... > >> > >>This is strange. Any ideas why this

Re: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Denys Vlasenko
On Thu, Jun 26, 2014 at 1:37 PM, Morten Kvistgaard wrote: >>> ... >>> execve("proc/self/exe", ["ftpd", "-l", "/"], [/* 9 vars */]) = -1 >>> ENOENT (No such file or directory) ... >> >>This is strange. Any ideas why this fails on your machine? > > Yes, the fchdir(G.root_fd) is not enough to break t

Re: [PATCH 0/6] Add a unit-testing framework to Busybox

2014-06-26 Thread Bartosz Gołaszewski
2014-06-26 14:12 GMT+02:00 Rich Felker : > On Thu, Jun 26, 2014 at 12:35:53PM +0200, Denys Vlasenko wrote: >> Global data is initialized to zero at program load time. > > And even if you need a non-zero initial value, static initialization > is usually suitable; there's no need for code for this ki

Re: [Question] Questions about the BusyBox specification.

2014-06-26 Thread tito
On Thursday 26 June 2014 14:05:34 Frank Ihle wrote: > Dear BusyBox, > > due to we consider about using BusyBox on our Embedded System, I have some > questions left whose answers couldn't be found on the BusyBox homepage or > documentation. > > I hope somebody can give me a bit more information

RE: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Morten Kvistgaard
> I have no idea why soneone would use NOMMU on Ubuntu, but most likely busybox > is dynamically linked and therefor exec busybox fails because it doesn't find > the dynamic linker. Even if it would find the dynamic linker, the linker > wouldn't find the dynamic libraries. Busybox is not config

Re: [PATCH 0/6] Add a unit-testing framework to Busybox

2014-06-26 Thread Rich Felker
On Thu, Jun 26, 2014 at 12:35:53PM +0200, Denys Vlasenko wrote: > On Thu, Jun 26, 2014 at 12:00 PM, Bartosz Gołaszewski > wrote: > > 2014-06-22 16:31 GMT+02:00 Denys Vlasenko : > >> Applied all patches with some editing. > >> Thanks a lot! > > > > Hi Denys, > > > > I see, that you removed the INIT

Re: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Ralf Friedl
Denys Vlasenko wrote: On Thu, Jun 26, 2014 at 11:45 AM, Morten Kvistgaard wrote: I've attached my strace. I'm not sure that it tells me anything though. I've run: sudo strace -p -f -o ftpd.strace Is there a better way? Anyway, the current code will exit jail with the following code: ... if (

[Question] Questions about the BusyBox specification.

2014-06-26 Thread Frank Ihle
Dear BusyBox, due to we consider about using BusyBox on our Embedded System, I have some questions left whose answers couldn't be found on the BusyBox homepage or documentation. I hope somebody can give me a bit more information, probably these questions/answers could be put on the BusyBox ho

RE: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Morten Kvistgaard
>> ... >> execve("proc/self/exe", ["ftpd", "-l", "/"], [/* 9 vars */]) = -1 >> ENOENT (No such file or directory) ... > >This is strange. Any ideas why this fails on your machine? Yes, the fchdir(G.root_fd) is not enough to break the jail. (And it's not just my machine. It's all of our Ubuntu ve

Re: udhcpd leases file is not written

2014-06-26 Thread Denys Vlasenko
On Tue, Jun 24, 2014 at 6:04 PM, Vinothkumar Vivekanandan wrote: > I am using BusyBox based dhcp server, udhcpd. > > This is my configuration file. But no lease details is been written in > /var/lib/misc/udhcpd.leases. Need some help. I need more data: - busybox version - .config file - "strace -

Re: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Denys Vlasenko
On Thu, Jun 26, 2014 at 11:45 AM, Morten Kvistgaard wrote: > I've attached my strace. > > I'm not sure that it tells me anything though. > I've run: sudo strace -p -f -o ftpd.strace > Is there a better way? > > Anyway, the current code will exit jail with the following code: > > ... > if (fchdir(G

Re: [PATCH 0/6] Add a unit-testing framework to Busybox

2014-06-26 Thread Denys Vlasenko
On Thu, Jun 26, 2014 at 12:00 PM, Bartosz Gołaszewski wrote: > 2014-06-22 16:31 GMT+02:00 Denys Vlasenko : >> Applied all patches with some editing. >> Thanks a lot! > > Hi Denys, > > I see, that you removed the INIT_FIRST macro from my implementation. > Are you sure llist_t *tests will always be

Re: [PATCH 0/6] Add a unit-testing framework to Busybox

2014-06-26 Thread Bartosz Gołaszewski
2014-06-22 16:31 GMT+02:00 Denys Vlasenko : > Applied all patches with some editing. > Thanks a lot! Hi Denys, I see, that you removed the INIT_FIRST macro from my implementation. Are you sure llist_t *tests will always be initialized before calling the test registering functions? Bartosz Gołasz

Re: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Denys Vlasenko
On Thu, Jun 26, 2014 at 9:27 AM, Morten Kvistgaard wrote: > Here's how to reproduce the error: > > I'm using a i386 Ubuntu 10.04. > > - Enable the "Force NOMMU build" in menuconfig. (Also make sure the ftpd is > added.) > - Add the busybox ftpd to your inetd. Something like: > > ftp stream t

RE: ftpd authentication [PATCH] ftpd: NOMMU/chroot fix

2014-06-26 Thread Morten Kvistgaard
Hello Denys, Here's how to reproduce the error: I'm using a i386 Ubuntu 10.04. - Enable the "Force NOMMU build" in menuconfig. (Also make sure the ftpd is added.) - Add the busybox ftpd to your inetd. Something like: ftp stream tcp nowait root/home/mk/workspace/busybox/busybox