linux-kernel - Re:Re: [PATCH 4.19.y 0/9] Fix the UAF issue caused by the loop driver

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 linux-cve-announce 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]

Message-ID: <1d74f009.2179.18e16c2d99f.Coremail.zhanggenjian@126.com>
Date: Thu, 7 Mar 2024 10:34:12 +0800 (CST)
From: "genjian zhang" <zhanggenjian@....com>
To: "Greg KH" <gregkh@...uxfoundation.org>
Cc: stable@...r.kernel.org, axboe@...nel.dk, stable@...nel.org, 
	linux-block@...r.kernel.org, linux-kernel@...r.kernel.org, 
	zhanggenjian123@...il.com, "Genjian Zhang" <zhanggenjian@...inos.cn>
Subject: Re:Re: [PATCH 4.19.y 0/9] Fix the UAF issue caused by the loop
 driver


At 2024-03-04 21:31:20, "Greg KH" <gregkh@...uxfoundation.org> wrote:
>On Fri, Mar 01, 2024 at 09:30:19AM +0800, Genjian wrote:
>> From: Genjian Zhang <zhanggenjian@...inos.cn>
>> 
>> Hello!
>> 
>> We found that 2035c770bfdb ("loop: Check for overflow while configuring loop") lost a unlock loop_ctl_mutex in loop_get_status(...).
>> which caused syzbot to report a UAF issue. However, the upstream patch does not have this issue.
>> So, we revert this patch and directly apply the unmodified upstream patch.
>> 
>> Risk use-after-free as reported by syzbot：
>
>This looks good, but you are backporting commits that are NOT in newer
>stable releases (i.e. from 5.8 but the commit is not in 5.4.y), is that
>intentional?
>
>Does 5.4.y also have this problem?  If so, can you send a series that
>fixes that up so I can take both of them at the same time?
>
>thanks,
>
>greg k-h

Thank you for your advice. This problem also exists in 5.4.y. 
I will send a series of patches for 5.4.y.

thanks,

Genjian