Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-28 Thread Paul Eggleton
On Tue, 29 Nov 2016 13:36:58 Robert Yang wrote: > On 11/29/2016 11:45 AM, Paul Eggleton wrote: > > On Tue, 29 Nov 2016 10:45:51 Robert Yang wrote: > >> We may also need something like IMAGE_FEATURES += "test" to make it can > >> work with -ctestimage. > > > > Not sure I follow your reasoning here

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-28 Thread Robert Yang
On 11/29/2016 11:45 AM, Paul Eggleton wrote: On Tue, 29 Nov 2016 10:45:51 Robert Yang wrote: On 11/29/2016 09:57 AM, Khem Raj wrote: On Nov 24, 2016, at 10:59 AM, Paul Eggleton wrote:>> On Thu, 24 Nov 2016 08:46:29 Patrick Ohly wrote: On Thu, 2016-11-24 at

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-28 Thread Paul Eggleton
On Tue, 29 Nov 2016 10:45:51 Robert Yang wrote: > On 11/29/2016 09:57 AM, Khem Raj wrote: > >> On Nov 24, 2016, at 10:59 AM, Paul Eggleton > >> wrote:>> > >> On Thu, 24 Nov 2016 08:46:29 Patrick Ohly wrote: > >>> On Thu, 2016-11-24 at 11:38 +0800, Robert Yang

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-28 Thread Robert Yang
On 11/29/2016 09:57 AM, Khem Raj wrote: On Nov 24, 2016, at 10:59 AM, Paul Eggleton wrote: On Thu, 24 Nov 2016 08:46:29 Patrick Ohly wrote: On Thu, 2016-11-24 at 11:38 +0800, Robert Yang wrote: Currently, debug-tweaks is in EXTRA_IMAGE_FEATURES by default

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-28 Thread Khem Raj
> On Nov 24, 2016, at 10:59 AM, Paul Eggleton > wrote: > > On Thu, 24 Nov 2016 08:46:29 Patrick Ohly wrote: >> On Thu, 2016-11-24 at 11:38 +0800, Robert Yang wrote: >>> Currently, debug-tweaks is in EXTRA_IMAGE_FEATURES by default for poky, >>> and >>> there is

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-24 Thread Paul Eggleton
On Thu, 24 Nov 2016 08:46:29 Patrick Ohly wrote: > On Thu, 2016-11-24 at 11:38 +0800, Robert Yang wrote: > > Currently, debug-tweaks is in EXTRA_IMAGE_FEATURES by default for poky, > > and > > there is no passwd, so that user can login easily without a passwd, I > > think > > that current status

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-24 Thread Patrick Ohly
On Thu, 2016-11-24 at 09:09 -0500, Philip Balister wrote: > On 11/24/2016 02:46 AM, Patrick Ohly wrote: > > On Thu, 2016-11-24 at 11:38 +0800, Robert Yang wrote: > >> Currently, debug-tweaks is in EXTRA_IMAGE_FEATURES by default for poky, and > >> there is no passwd, so that user can login easily

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-24 Thread Philip Balister
On 11/24/2016 02:46 AM, Patrick Ohly wrote: > On Thu, 2016-11-24 at 11:38 +0800, Robert Yang wrote: >> Currently, debug-tweaks is in EXTRA_IMAGE_FEATURES by default for poky, and >> there is no passwd, so that user can login easily without a passwd, I think >> that current status is more unsafe ?

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-24 Thread Mike Looijmans
On 24-11-16 03:01, Robert Yang wrote: On 11/23/2016 07:16 PM, Patrick Ohly wrote: On Tue, 2016-11-22 at 23:49 -0800, Robert Yang wrote: [YOCTO #10710] Otherwise, we can't login as root when debug-tweaks is not in IMAGE_FEATURES, and there is no other users to login by default, so there is

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-24 Thread Robert Yang
On 11/24/2016 03:46 PM, Patrick Ohly wrote: On Thu, 2016-11-24 at 11:38 +0800, Robert Yang wrote: Currently, debug-tweaks is in EXTRA_IMAGE_FEATURES by default for poky, and there is no passwd, so that user can login easily without a passwd, I think that current status is more unsafe ? Both

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-24 Thread Patrick Ohly
On Thu, 2016-11-24 at 11:38 +0800, Robert Yang wrote: > Currently, debug-tweaks is in EXTRA_IMAGE_FEATURES by default for poky, and > there is no passwd, so that user can login easily without a passwd, I think > that current status is more unsafe ? Both well-known password and no password are

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-23 Thread Robert Yang
On 11/24/2016 11:18 AM, Paul Eggleton wrote: On Thu, 24 Nov 2016 10:01:59 Robert Yang wrote: On 11/23/2016 07:16 PM, Patrick Ohly wrote: On Tue, 2016-11-22 at 23:49 -0800, Robert Yang wrote: [YOCTO #10710] Otherwise, we can't login as root when debug-tweaks is not in IMAGE_FEATURES, and

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-23 Thread Paul Eggleton
On Thu, 24 Nov 2016 10:01:59 Robert Yang wrote: > On 11/23/2016 07:16 PM, Patrick Ohly wrote: > > On Tue, 2016-11-22 at 23:49 -0800, Robert Yang wrote: > >> [YOCTO #10710] > >> > >> Otherwise, we can't login as root when debug-tweaks is not in > >> IMAGE_FEATURES, and there is no other users to

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-23 Thread Robert Yang
On 11/23/2016 07:16 PM, Patrick Ohly wrote: On Tue, 2016-11-22 at 23:49 -0800, Robert Yang wrote: [YOCTO #10710] Otherwise, we can't login as root when debug-tweaks is not in IMAGE_FEATURES, and there is no other users to login by default, so there is no way to login. Wait a second, are

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-23 Thread Burton, Ross
On 23 November 2016 at 11:16, Patrick Ohly wrote: > Wait a second, are you really suggesting that OE-core should have a > default root password in its default configuration? > > That's very bad practice and I'm against doing it this way. Having a > default password is one

Re: [OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-23 Thread Patrick Ohly
On Tue, 2016-11-22 at 23:49 -0800, Robert Yang wrote: > [YOCTO #10710] > > Otherwise, we can't login as root when debug-tweaks is not in > IMAGE_FEATURES, and there is no other users to login by default, so > there is no way to login. Wait a second, are you really suggesting that OE-core should

[OE-core] [PATCH 2/2] base-passwd: set root's default password to 'root'

2016-11-23 Thread Robert Yang
[YOCTO #10710] Otherwise, we can't login as root when debug-tweaks is not in IMAGE_FEATURES, and there is no other users to login by default, so there is no way to login. Signed-off-by: Robert Yang --- .../base-passwd/base-passwd/passwd_master.patch | 16