On Fri, Feb 10, 2023 at 9:35 AM Ondrej Mosnacek <omosnace@xxxxxxxxxx> wrote:
> On Thu, Feb 9, 2023 at 11:09 PM Paul Moore <paul@xxxxxxxxxxxxxx> wrote:
> > Hi all,
> >
> > I just noticed a selinux-testsuite failure in the tests/file test:
> >
> > # Test 8 got: "512" (file/test at line 103)
> > # Expected: "0"
> > # file/test line 103 is: ok( $result, 0 );
> > file/test ................... Failed 1/16 subtests
> >
> > Digging into this a bit more it looks like the the fnctl(TIOCSTI) call
> > in test_sigiotask.c is failing due to a recent Rawhide change to
> > undefine CONFIG_LEGACY_TIOCSTI, disabling TIOCSTI. Upstream kernel
> > commit 83efeeeb3d04 ("tty: Allow TIOCSTI to be disabled") has more
> > information on the Kconfig option.
> >
> > I'm not going to argue for reenabling CONFIG_LEGACY_TIOCSTI, I think
> > turning it off is a good idea, but it does mean we need to adjust the
> > selinux-testsuite.
>
> I noticed that in our CI yesterday and had a brief look. Unfortunately
> I don't (yet) fully understand the terminal device voodoo being done
> in test_sigiotask.c and I don't have any idea how to make it work
> without TIOCSTI. So the best fix for now seems to be to use the
> dev.tty.legacy_tiocsti sysctl (see the Kconfig). However, it seems to
> be broken currently:
>
> # sysctl dev.tty.legacy_tiocsti
> # sysctl -w dev.tty.legacy_tiocsti=1
> sysctl: setting key "/proc/sys/dev/tty/legacy_tiocsti": Invalid argument
> #
>
> I will look into fixing it...
https://lore.kernel.org/lkml/20230210145823.756906-1-omosnace@xxxxxxxxxx/
--
Ondrej Mosnacek
Senior Software Engineer, Linux Security - SELinux kernel
Red Hat, Inc.
