[prev in list] [next in list] [prev in thread] [next in thread]
List: busybox
Subject: Re: adduser/passwd: too long username
From: Laszlo Papp <lpapp () kde ! org>
Date: 2014-08-21 23:45:05
Message-ID: CAOMwXhPP2e=Dogik3Tme2VtUPge4A=RB8_qLpwyBos17Hq25pg () mail ! gmail ! com
[Download RAW message or body]
On Thu, Aug 21, 2014 at 10:10 PM, Natanael Copa <n@tanael.org> wrote:
> Den 21. aug. 2014 kl. 08:42 skrev Laszlo Papp <lpapp@kde.org>:
>
> > On Thu, Aug 21, 2014 at 7:37 AM, Natanael Copa <ncopa@alpinelinux.org> wrote:
> > > On Mon, 18 Aug 2014 20:37:00 +0200
> > > Natanael Copa <ncopa@alpinelinux.org> wrote:
> > >
> > > > I never sent those patches upstream because I don't think this is the
> > > > correct fix. For Alpine Linux the "correct" fix was to use the libc
> > > > implementation and a libc that handles this properly (musl libc)
> > > > together with the patch I sent to this mailing list.
> > > >
> > > > http://lists.busybox.net/pipermail/busybox/2014-April/080809.html
> > > >
> > > > Commit message didn't say it but that patch is also needed to fix the
> > > > username/groupsize issue - with glibc too I believe.
> > >
> > > Denys, any chance that the above mentioned patch gets applied?
> > >
> > > Should I send a v2 with better commit message?
> >
> > No, please check out tito's work. That seems to be the right to do.
>
> we need both.
>
> as i understand, titos patch solves the problem for when you enable busybox's \
> implementation with CONFIG_USE_BB_PWD_GRP. you actually have the same problem with \
> it disabled, even if system implementation can handle it correctly. that is what my \
> patch solves.
That seem to be an irrelevant discussion in this thread, then. Should
have been sent to a separate thread. This thread is about issues when
CONFIG_USE_BB_PWD_GRP is enabled.
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic