All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: Zubin Mithra <zsm@chromium.org>
Cc: stable@vger.kernel.org, tytso@mit.edu, natechancellor@gmail.com,
	groeck@chromium.org
Subject: Re: [PATCH v4.4.y] ext4: never move the system.data xattr out of the inode body
Date: Sun, 30 Sep 2018 08:28:01 -0700	[thread overview]
Message-ID: <20180930152801.GA2942@kroah.com> (raw)
In-Reply-To: <20180928162144.216318-1-zsm@chromium.org>

On Fri, Sep 28, 2018 at 09:21:44AM -0700, Zubin Mithra wrote:
> From: Theodore Ts'o <tytso@mit.edu>
> 
> commit 8cdb5240ec5928b20490a2bb34cb87e9a5f40226 upstream.
> 
> When expanding the extra isize space, we must never move the
> system.data xattr out of the inode body.  For performance reasons, it
> doesn't make any sense, and the inline data implementation assumes
> that system.data xattr is never in the external xattr block.
> 
> This addresses CVE-2018-10880
> 
> https://bugzilla.kernel.org/show_bug.cgi?id=200005
> 
> Backport Note:
> - dfa2064b22("ext4: factor out loop for freeing inode xattr space")
> factored out the loop from inside ext4_expand_extra_isize_ea to a
> separate function named ext4_xattr_make_inode_space. As the above commit
> is not present in 4.4.y, make the change inside ext4_expand_extra_isize_ea.


What about a 4.9.y version of this?  I can't just take a 4.4 version
because someone would suffer a regression if they moved from 4.4.y to
4.9.y :(

thanks,

greg k-h

      reply	other threads:[~2018-09-30 22:01 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-28 16:21 [PATCH v4.4.y] ext4: never move the system.data xattr out of the inode body Zubin Mithra
2018-09-30 15:28 ` Greg KH [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180930152801.GA2942@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=groeck@chromium.org \
    --cc=natechancellor@gmail.com \
    --cc=stable@vger.kernel.org \
    --cc=tytso@mit.edu \
    --cc=zsm@chromium.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.