Bug 5344 - Split up IMAGE_FEATURES - debug-tweaks into logical components
Summary: Split up IMAGE_FEATURES - debug-tweaks into logical components
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: configuration (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: Medium enhancement
Target Milestone: 1.8
Assignee: Saul Wold
QA Contact:
URL:
Whiteboard: 12 February 2015: Docs updated
Depends on:
Blocks:
 
Reported: 2013-10-14 09:46 UTC by Koen Kooi
Modified: 2015-02-12 17:34 UTC (History)
5 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Done (doc changes complete)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Koen Kooi 2013-10-14 09:46:12 UTC
Currently PAM and dropbear allow logins but openssh doesn't (patch sent for that). It would be nice to have a global knob to change this behaviour for all affected recipes.
Comment 1 Chong Lu 2014-03-25 01:44:22 UTC
There is following configuration in /etc/pam.d/common-auth file on target:

auth	[success=1 default=ignore]	pam_unix.so nullok

About nullok argument:

nullok
           The default action of this module is to not permit the user access to a service if their official password is blank. The nullok argument overrides this default
           and allows any user with a blank password to access the service.

So, we can use this argument to decide whether allow empty password login.
Comment 2 Chong Lu 2014-04-02 01:31:47 UTC
Hi Koen,

Can you describe this defect in detail?
Comment 3 Stephen K Jolley 2014-06-18 17:26:03 UTC
Assigned to Koen Kooi since that is who should supply the NEEDINFO per RP.
Comment 4 Mark Hatle 2014-07-17 15:19:16 UTC
The ability to do an empty password login is provided by debug-tweaks, but it also happens to do other things that may be less then desired.

I'm updating this to be an enhancement request that breaks up the debug-tweaks functionality into logical units so they can be individually used.

Change summary from:

Add knob to (dis)allow empty password login for PAM/dropbear/openssh/etc

to:

Split up IMAGE_FEATURES - debug-tweaks into logical components


Suggested enhancement:

Split debug-tweaks into it's logical components:

empty-root-password - run zap_empty_root_password

allow-empty-password - run ssh_allow_empty_password

post-install-logging - run postinst_enable_logging


Map "debug-tweaks" to enabled those three items.
Comment 5 Saul Wold 2015-02-09 15:51:32 UTC
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=d42963a3feda6c4cf074cbdacc7cf904734d70cf

ScottR, docs will need to add definitions for the split values of debug-tweaks 
as enumerated by Mark in the above comment.
Comment 6 Scott Rifenbark 2015-02-12 17:34:00 UTC
Hi, 

The documentation changes for this fix are here:

http://www.yoctoproject.org/docs/1.8/ref-manual/ref-manual.html#var-EXTRA_IMAGE_FEATURES.

http://www.yoctoproject.org/docs/1.8/ref-manual/ref-manual.html#ref-features-image (see the 'allow-empty-password', 'debug-tweaks', 'empty-root-password', and 'post-install-logging' items in the feature list.

Since the bug is already set to RESOLVED, I am simply posting notice that these changes are in.  If there are issues, let me know here.  I also set the documentation flag to "done".

Thanks, 
Scott