lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Thu, 10 Dec 2020 15:09:46 -0800
From:   Libo Chen <libo.chen@...cle.com>
To:     Steven Rostedt <rostedt@...dmis.org>
Cc:     linux-kernel@...r.kernel.org, m.mizuma@...fujitsu.com,
        Masayoshi Mizuma <msys.mizuma@...il.com>
Subject: Re: [PATCH 1/1] ktest.pl: Fix incorrect reboot for grub2bls

On 11/30/20 7:25 PM, Steven Rostedt wrote:

> On Sun, 29 Nov 2020 20:32:56 -0500
> Masayoshi Mizuma <msys.mizuma@...il.com> wrote:
>
>> On Fri, Nov 20, 2020 at 06:12:43PM -0800, Libo Chen wrote:
>>> This issue was first noticed when I was testing different kernels on
>>> Oracle Linux 8 which as Fedora 30+ adopts BLS as default. Even though a
>>> kernel entry was added successfully and the index of that kernel entry was
>>> retrieved correctly, ktest still wouldn't reboot the system into
>>> user-specified kernel.
>>>
>>> The bug was spotted in subroutine reboot_to where the if-statement never
>>> checks for REBOOT_TYPE "grub2bls", therefore the desired entry will not be
>>> set for the next boot.
>>>
>>> Add a check for "grub2bls" so that $grub_reboot $grub_number can
>>> be run before a reboot if REBOOT_TYPE is "grub2bls" then we can boot to
>>> the correct kernel.
>>>
>>> Fixes: ac2466456eaa ("ktest: introduce grub2bls REBOOT_TYPE option")
>>>
>>> Signed-off-by: Libo Chen <libo.chen@...cle.com>
>> Thank you for the fix!
>> I tested the patch with fedora33. It works well.
>>
>> Please feel free to add:
>>
>> 	Tested-by: Masayoshi Mizuma <m.mizuma@...fujitsu.com>
>>
>>
> Thanks.
>
> I may push this upstream by the end of the week.
>
> -- Steve

(Mistakenly sent the last one privately so resend it now)

Hi Steven,


Just wanna check in and check on the status of it.


Thanks,

Libo

Powered by blists - more mailing lists