<div dir="ltr"><div dir="ltr"><div>Hi Thadeu, <br></div><div><br></div><div>Not hard. I already had 
one prepared but saw that those commits were easy enough to find and 
apply. They do add a lot but the changes seemed straightforward enough. 
Do you think it would be better to exclude those other patches? <br></div><font color="#888888"><div><br></div><div>(p.s. resent the message because forgot to cc ML) <br></div><div><br></div><div>- Luke</div></font></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 8, 2021 at 6:04 AM Thadeu Lima de Souza Cascardo <<a href="mailto:cascardo@canonical.com">cascardo@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 Thu, Oct 07, 2021 at 01:05:46PM -0700, Luke Nowakowski-Krijger wrote:<br>
> [Impact]<br>
> Mounting a crafted ext4 filesystem can trigger an integer overflow<br>
> that occurs in ext4_es_cache_extent(). This yields a kernel bug that can<br>
> lead to a system crash and denial of service.<br>
> <br>
> [Backports]<br>
> Updated minor context changes in fs/ext4/ext4.h to include sbi_array_rcu_deref<br>
> define.<br>
> <br>
<br>
Hey, Luke.<br>
<br>
How hard would the backport of "ext4: check journal inode extents more<br>
carefully" be without the other commits?<br>
<br>
Cascardo.<br>
<br>
> [Test case]<br>
> Reproduced the bug using the reproducer here<br>
> (<a href="https://bugzilla.suse.com/show_bug.cgi?id=1173485" rel="noreferrer" target="_blank">https://bugzilla.suse.com/show_bug.cgi?id=1173485</a>),<br>
> confirmed that after the patches are applied that the system reports a<br>
> malformed filesystem and mounting fails.<br>
> <br>
> [Potential regression]<br>
> Journal inodes are no longer a special case when checking extent trees<br>
> which means that some filesystems that could be mounted could now fail.<br>
> <br>
> Jan Kara (1):<br>
>   ext4: check journal inode extents more carefully<br>
> <br>
> Theodore Ts'o (3):<br>
>   ext4: save the error code which triggered an ext4_error() in the<br>
>     superblock<br>
>   ext4: simulate various I/O and checksum errors when reading metadata<br>
>   ext4: save all error info in save_error_info() and drop<br>
>     ext4_set_errno()<br>
> <br>
>  fs/ext4/balloc.c         |  10 ++--<br>
>  fs/ext4/block_validity.c |  59 +++++++++---------<br>
>  fs/ext4/ext4.h           | 125 ++++++++++++++++++++++++++++++++-------<br>
>  fs/ext4/ext4_jbd2.c      |  10 ++--<br>
>  fs/ext4/extents.c        |  42 ++++++-------<br>
>  fs/ext4/ialloc.c         |  15 +++--<br>
>  fs/ext4/indirect.c       |   8 +--<br>
>  fs/ext4/inline.c         |  11 ++--<br>
>  fs/ext4/inode.c          |  38 ++++++------<br>
>  fs/ext4/mballoc.c        |  21 +++----<br>
>  fs/ext4/mmp.c            |  13 ++--<br>
>  fs/ext4/move_extent.c    |   4 +-<br>
>  fs/ext4/namei.c          |  31 ++++++----<br>
>  fs/ext4/super.c          | 106 +++++++++++++++++++++++++--------<br>
>  fs/ext4/sysfs.c          |  23 +++++++<br>
>  fs/ext4/xattr.c          |  12 ++--<br>
>  16 files changed, 351 insertions(+), 177 deletions(-)<br>
> <br>
> -- <br>
> 2.30.2<br>
> <br>
> <br>
> -- <br>
> kernel-team mailing list<br>
> <a href="mailto:kernel-team@lists.ubuntu.com" target="_blank">kernel-team@lists.ubuntu.com</a><br>
> <a href="https://lists.ubuntu.com/mailman/listinfo/kernel-team" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/kernel-team</a><br>
</blockquote></div></div>