[Buildroot] [PATCH] host-mkpasswd: fix crash on Fedora 28 build host
giometriq at gmail.com
Thu May 10 16:02:10 UTC 2018
Arnout, Yann, All,
I just noticed there are actually two places host-mkpasswd is being used:
root account (skeleton-init-common.mk) and users table. I think,
if we're going to replace mkpasswd with a script, this all should go through
one single point somehow (something like support/scripts/crypt_wrapper).
On Thu, May 10, 2018 at 12:52 AM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> I don't like that we reinvent such tools... Especially when dealing with
> security-related tools...
>> dd if=/dev/urandom bs=1 count=12 | python -c "
> This is not optimal. Better to do a single read instead of 12: bs=12
>> import sys, crypt, base64
>> salt = sys.stdin.read()
> Why such convoluted code to read 12 bytes of random data? Why not use
> Python's builtins for that?
> import secrets
> blob = secrets.token_bytes(12)
I actually like Arnout way of using /dev/urandom. The more we make use of
system resources (and less responsibility for python) the better.
> Sorry, I am not too convinced that writing our own tool is good...
> Yann E. MORIN.
To be honest, like all of you, I'm also very uncomfortable replacing
a python solution. There're three reasons that made me convinced this
is the right thing to do.
First, it's cryptographically sound. We're not going to roll our own
It’s simply making a LIBC crypt(3) function call. Like mkpasswd, this is just a
system/libc call wrapper.
Second, host-mkpasswd (which is carved out of another package to be a
buildroot’s own bundle) apparently needs periodic attention (as can be
seen in the
mailing list archive throughout the years). In my case it actually
built the shadow
with empty password for the root (SHA functions are ignored for musl)
any errors. I know these blatant failures are easily caught by the
developer before any kind
of release but they all reveal how rapidly this package is going
And finally, this (python or perl script) seems to be the only
that actually works without adding hundreds of lines of code (or a
full-blown whois package)
to the build infrastructure with their own specific libc compatibility
patches and version bumps.
And it works on Alpine!
I personally settle for a code that is aesthetically wanting or
if the alternative is to take more of the maintainers’ attention and time.
An absolute rare commodity we can ill afford.
Mike Z. Vand
More information about the buildroot