Re: [PATCH] rootfs-postcommands.bbclass: Filter out dangling symlinks in ssh_allow_empty_password()

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Wed, 2017-08-09 at 09:45 -0700, Khem Raj wrote:
> In images built with pam in DISTRO_FEATURES, we end up with dangling
> symlinks
> if su is not packaged into image
> 
> $ ls /mnt/a/oe/build/tmp/work/raspberrypi3-bec-linux-gnueabi/core-
> image-minimal/1.0-r0/rootfs/etc/pam.d/su-l -l
> lrwxrwxrwx 1 kraj users 2 Aug  9 07:56
> /mnt/a/oe/build/tmp/work/raspberrypi3-bec-linux-gnueabi/core-image-
> minimal/1.0-r0/rootfs/etc/pam.d/su-l -> su
> 
> This causes image do_rootfs to fail
> 
> > 
> > sed: can't read /mnt/a/oe/build/tmp/work/raspberrypi3-bec-linux-
> > gnueabi/core-image-minimal/1.0-r0/rootfs/etc/pam.d/s
> u-l: No such file or directory
> > 
> > WARNING: /mnt/a/oe/build/tmp/work/raspberrypi3-bec-linux-
> > gnueabi/core-image-minimal/1.0-r0/temp/run.ssh_allow_empty_
> password.19238:1 exit 2 from 'sed -i 's/nullok_secure/nullok/'
> /mnt/a/oe/build/tmp/work/raspberrypi3-bec-linux-gnueabi
> /core-image-minimal/1.0-r0/rootfs/etc/pam.d/*'
> 
> Therefore we need to filter out dangling symlinks before sed'ing
> things out
> 
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> ---
>  meta/classes/rootfs-postcommands.bbclass | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)

This caused all rootfs sanity tests to fail in lsb images (for all
arches):

https://autobuilder.yocto.io/builders/nightly-x86-lsb/builds/410
https://autobuilder.yocto.io/builders/nightly-x86-64-lsb/builds/408
https://autobuilder.yocto.io/builders/nightly-ppc-lsb/builds/407
https://autobuilder.yocto.io/builders/nightly-mips-lsb/builds/394
https://autobuilder.yocto.io/builders/nightly-arm-lsb/builds/406

(I've confirmed dropping this patch made this issue go away)

Cheers,

Richard