RE: ftpd authentication

2014-08-05 Thread Morten Kvistgaard
Hullo Denys, Can you help us again. I’ve posted the patch for ftpd authentication, but it seems that the trunk getpwnam (et al.) have gone haywire. (I could override the getpwnam as Tanguy says, but it seems a little flawed.) Regards, Morten Kvistgaard From: Tanguy Pruvot

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
Yeah, mayhaps... Thanks for the prompt reply. I tried to debug the code, but the busybox code here is a bit messy heavily abusing macros in C and all that. It ain't easy I must confess! For instance, see this section: http://git.busybox.net/busybox/tree/libpwdgrp/pwd_grp.c#n227 The _source_ is

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
Here is my tested fix without being to debug the busybox code, so only code reading and understanding were my friends: commit 9610650b6ce2a4c1904f78a2dcdb47cad3d2e3d1 Author: Laszlo Papp lp...@kde.org Date: Tue Aug 5 11:42:24 2014 +0100 Allow 256 bytes long usernames as per Unix standards

Re: adduser/passwd: too long username

2014-08-05 Thread tito
On Tuesday 05 August 2014 12:46:08 you wrote: Here is my tested fix without being to debug the busybox code, so only code reading and understanding were my friends: commit 9610650b6ce2a4c1904f78a2dcdb47cad3d2e3d1 Author: Laszlo Papp lp...@kde.org Date: Tue Aug 5 11:42:24 2014 +0100

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
I disagree. There is nothing to reject here. It fixes _my issue_ at hand. If you want to fix other bugs in your system that you are facing in reality, by all means, fix them in a separate change. I definitely do not agree with dynamically allocated buffer for the simple reasons of: 1) Slow. 2)

Re: adduser/passwd: too long username

2014-08-05 Thread tito
On Tuesday 05 August 2014 14:27:57 you wrote: I disagree. There is nothing to reject here. It fixes _my issue_ at hand. If you want to fix other bugs in your system that you are facing in reality, by all means, fix them in a separate change. I definitely do not agree with dynamically

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
On Tue, Aug 5, 2014 at 1:42 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 14:27:57 you wrote: I disagree. There is nothing to reject here. It fixes _my issue_ at hand. If you want to fix other bugs in your system that you are facing in reality, by all means, fix them in a

Re: adduser/passwd: too long username

2014-08-05 Thread tito
On Tuesday 05 August 2014 14:47:53 you wrote: On Tue, Aug 5, 2014 at 1:42 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 14:27:57 you wrote: I disagree. There is nothing to reject here. It fixes _my issue_ at hand. If you want to fix other bugs in your system that you

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
On Tue, Aug 5, 2014 at 2:06 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 14:47:53 you wrote: On Tue, Aug 5, 2014 at 1:42 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 14:27:57 you wrote: I disagree. There is nothing to reject here. It fixes _my

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
On Tue, Aug 5, 2014 at 2:14 PM, Laszlo Papp lp...@kde.org wrote: On Tue, Aug 5, 2014 at 2:06 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 14:47:53 you wrote: On Tue, Aug 5, 2014 at 1:42 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 14:27:57 you

Re: adduser/passwd: too long username

2014-08-05 Thread tito
On Tuesday 05 August 2014 15:21:49 you wrote: On Tue, Aug 5, 2014 at 2:14 PM, Laszlo Papp lp...@kde.org wrote: On Tue, Aug 5, 2014 at 2:06 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 14:47:53 you wrote: On Tue, Aug 5, 2014 at 1:42 PM, tito farmat...@tiscali.it

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
On Tue, Aug 5, 2014 at 3:06 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 15:21:49 you wrote: On Tue, Aug 5, 2014 at 2:14 PM, Laszlo Papp lp...@kde.org wrote: On Tue, Aug 5, 2014 at 2:06 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
On Tue, Aug 5, 2014 at 3:13 PM, Laszlo Papp lp...@kde.org wrote: On Tue, Aug 5, 2014 at 3:06 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 15:21:49 you wrote: On Tue, Aug 5, 2014 at 2:14 PM, Laszlo Papp lp...@kde.org wrote: On Tue, Aug 5, 2014 at 2:06 PM,

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
commit 980965767ef3ace983746ee25e92665b87d16755 Author: Laszlo Papp lp...@kde.org Date: Tue Aug 5 11:42:24 2014 +0100 Allow 256 bytes long usernames as per Unix standards (usually) diff --git a/libpwdgrp/pwd_grp.c b/libpwdgrp/pwd_grp.c index 2060d78..9e4424f 100644 ---

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
To challenge this a bit further for completeness: the other option would be to limit the username length to 32 or 48 bytes rather than allowing the whole 128-byte desktop-range. 256 does look like a bit excessive that getconf returns: getconf LOGIN_NAME_MAX 256 I find it hard to achieve in even

Re: adduser/passwd: too long username

2014-08-05 Thread Denys Vlasenko
On Mon, Aug 4, 2014 at 7:06 PM, Laszlo Papp lp...@kde.org wrote: sudo busybox adduser f passwd: unknown user

Re: adduser/passwd: too long username

2014-08-05 Thread tito
On Tuesday 05 August 2014 18:09:08 you wrote: To challenge this a bit further for completeness: the other option would be to limit the username length to 32 or 48 bytes rather than allowing the whole 128-byte desktop-range. 256 does look like a bit excessive that getconf returns: getconf

Re: adduser/passwd: too long username

2014-08-05 Thread tito
On Tuesday 05 August 2014 17:05:11 you wrote: On Tue, Aug 5, 2014 at 3:13 PM, Laszlo Papp lp...@kde.org wrote: On Tue, Aug 5, 2014 at 3:06 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 15:21:49 you wrote: On Tue, Aug 5, 2014 at 2:14 PM, Laszlo Papp lp...@kde.org

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
it will be / only in case the home dir does not exist at login time as the login program does chdir to the home That is what the linked page also says, yes. dir else you would be locked out of a broken system (damaged partition mounted to /home, /root folder corrupted on main file

Re: adduser/passwd: too long username

2014-08-05 Thread Laszlo Papp
On Tue, Aug 5, 2014 at 8:24 PM, tito farmat...@tiscali.it wrote: On Tuesday 05 August 2014 18:09:08 you wrote: To challenge this a bit further for completeness: the other option would be to limit the username length to 32 or 48 bytes rather than allowing the whole 128-byte desktop-range.

Re: ftpd authentication

2014-08-05 Thread Denys Vlasenko
On Mon, Aug 4, 2014 at 12:38 PM, Morten Kvistgaard m...@pch-engineering.dk wrote: I've attached a patch for adding basic authentication to the ftpd. This used to work with version 1.21.1. And walter harms tested it with 1.22.1. And it worked with trunk 3 months ago. It doesn't seem to work

Re: How do I (unconditionally) enable unicode support in busybox?

2014-08-05 Thread James Bowlin
On Mon, Aug 04, 2014 at 09:10 PM, Rich Felker said: I suspect it's an issue with some option to run busybox commands in the shell process itself rather than forking and execing. Seeing a full config file would be useful. I've attached a config I've been using for a while as plain text. If it

Re: How do I (unconditionally) enable unicode support in busybox?

2014-08-05 Thread James Bowlin
On Mon, Aug 04, 2014 at 03:00 PM, Michael Conrad said: On 8/4/2014 2:48 PM, James Bowlin wrote: [ $LANG = en_US.UTF-8 ] || LANG=en_US.UTF-8 exec $0 $@ I think you should focus on this bug. Which busybox version? Which libc version? Because my busybox runs it just fine. (as should any