Re: [PATCH 2/2] integrity: replace call to integrity_read_file with kernel version

From: James Morris
Date: Thu Sep 14 2017 - 16:22:18 EST


On Tue, 12 Sep 2017, Mimi Zohar wrote:

> From: Christoph Hellwig <hch@xxxxxx>
>
> The CONFIG_IMA_LOAD_X509 and CONFIG_EVM_LOAD_X509 options permit
> loading x509 signed certificates onto the trusted keyrings without
> verifying the x509 certificate file's signature.
>
> This patch replaces the call to the integrity_read_file() specific
> function with the common kernel_read_file_from_path() function.
> To avoid verifying the file signature, this patch defines
> READING_X509_CERTFICATE.

So, to be clear, this patch solves the XFS deadlock using a different
approach (to the now reverted integrity_read approach), which Christoph
also says is more correct generally. Correct?

What testing has this had?

Should this go in with the rest of the security changes now or wait until
either -rc or the next merge window?


--
James Morris
<jmorris@xxxxxxxxx>