From patchwork Sat Dec 29 00:07:32 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [pull,request,v4] Pull request for branch yem-root-passwd Date: Fri, 28 Dec 2012 14:07:32 -0000 From: "Yann E. MORIN" X-Patchwork-Id: 208606 Message-Id: To: buildroot@busybox.net Cc: "Yann E. MORIN" Hello All! This is iteration #4 for setting the root password from the configuration menu: - first patch adds the basic functionality: - plain text password in menuconfig - MD5-encrypted in /etc/shadow - second patch adds additional encryption methods There has been previous review of this series by Arnout, who suggested dropping patch #2, and only use an MD5-encrypted password. Since MD5 is now considered to be a weak hash, stronger alternatives may be usefull for the security-conscious lurking among us. That's why I kept (resurrected) that second patch. Arnout also objected to having the root password in clear in the .config, and recommended that the user enters the already-encrypted password. That has, IMHO, a few drawbacks, in that it requires the user actually reads the help text, switch to an alternate terminal, generates a password, and copy-pastes it back in the initial terminal with the menuconfig. OTOH, if the user forgets his/her password, he/she can recover it by looking at the .config file. That's why I still advocates for entering a clear-text password in the menuconfig. Any more comments are welcome! Changes v3 -> v4: - fix the test for dependencies (Thomas) The following changes since commit 14989d0be59762a354e7c5c15a2eeb4826d8040b: package/tvheadend: new package (2012-12-28 23:40:26 +0100) are available in the git repository at: git://gitorious.org/buildroot/buildroot.git yem-root-passwd Yann E. MORIN (2): target: add option to set the root password target: add different methods to encode the root password support/dependencies/dependencies.sh | 9 +++++ system/Config.in | 67 ++++++++++++++++++++++++++++++++++ system/system.mk | 15 ++++++++ 3 files changed, 91 insertions(+), 0 deletions(-) Regards, Yann E. MORIN