On Wed, May 1, 2019 at 9:07 AM Christoph Hellwig <hch@xxxxxx> wrote: > > Casting mapping->a_ops->readpage to filler_t causes an indirect call > type mismatch with Control-Flow Integrity checking. This change fixes > the mismatch in read_cache_page_gfp and read_mapping_page by adding > using a NULL filler argument as an indication to call ->readpage > directly, and by passing the right parameter callbacks in nfs and jffs2. Nice. This looks great; thanks for looking at this. For the series (including patch 5): Reviewed-by: Kees Cook <keescook@xxxxxxxxxxxx> -- Kees Cook
