<div dir="ltr"><div>Yes this is for both focal:linux and focal:linux-hwe-5.8. I knew that it would be a bit ambiguous so I put the comma instead of the slash but I can see that did not help at all. In the future is it just better to separate them like [F][F/hwe-5.8]? At least that is unambiguous :) <br></div><div><br></div><div>- Luke<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 30, 2021 at 1:39 AM Stefan Bader <<a href="mailto:stefan.bader@canonical.com">stefan.bader@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 30.09.21 01:57, Luke Nowakowski-Krijger wrote:<br>
> [Impact]<br>
> Mounting a crafted f2fs file system with a segment count in a section<br>
> that is less than segs_per_sec causes out-of-boundary memory access<br>
> during fs initalization.<br>
> <br>
> [Test case]<br>
> Reproduced bug with syzbot reproducer<br>
> (<a href="https://syzkaller.appspot.com/x/repro.c?x=102fbac5900000" rel="noreferrer" target="_blank">https://syzkaller.appspot.com/x/repro.c?x=102fbac5900000</a>)<br>
> with slight modification to target a valid loop device.<br>
> Confirmed that after the patches were applied the fs reports that there<br>
> are malformed segments/sections and mounting the file system fails,<br>
> which stops the initialization from continuing and preventing the<br>
> out-of-boundary memory access.<br>
> <br>
> [Potential regression]<br>
> The patches add checks that are a superset of the previous checks, which<br>
> might cause some filesystems that succeeded in mounting to now fail.<br>
> <br>
> Chao Yu (1):<br>
>    f2fs: fix to do sanity check on segment/section count<br>
> <br>
> Wang Xiaojun (1):<br>
>    f2fs: fix wrong total_sections check and fsmeta check<br>
> <br>
>   fs/f2fs/segment.h |  1 +<br>
>   fs/f2fs/super.c   | 14 ++++++++++----<br>
>   2 files changed, 11 insertions(+), 4 deletions(-)<br>
> <br>
Double checking because in that combination it is not fully clear:<br>
<br>
this is for focal:linux AND focal:linux-hwe-5.8?<br>
<br>
-Stefan<br>
<br>
</blockquote></div></div>