On Fri, 2020-07-24 at 14:36 -0700, Kees Cook wrote:
FIRMWARE_PREALLOC_BUFFER is a "how", not a "what", and confuses the LSMs that are interested in filtering between types of things. The "how" should be an internal detail made uninteresting to the LSMs.
Fixes: a098ecd2fa7d ("firmware: support loading into a pre-allocated buffer") Fixes: fd90bc559bfb ("ima: based on policy verify firmware signatures (pre-allocated buffer)") Fixes: 4f0496d8ffa3 ("ima: based on policy warn about loading firmware (pre-allocated buffer)") Cc: stable@vger.kernel.org Acked-by: Scott Branden scott.branden@broadcom.com Signed-off-by: Kees Cook keescook@chromium.org
Thank you for updating the pre-allocated buffer comment.
Reviewed-by: Mimi Zohar zohar@linux.ibm.com