Jitindar Singh, Suraj <surajjs@xxxxxxxxxx> wrote: > When testing the v6.1.69 kernel I bisected an issue to the below commit > which was added in v6.1.68. When running the xfstests[1] on cifs I > observe a null pointer dereference in cifs_flush_folio() because folio > is null and dereferenced in size = folio_size(folio). Yeah. __filemap_get_folio() works differently in v6.1.y. There it returns a folio or NULL. In 6.7 it returns a folio or a negative error code. The error check in cifs_flush_folio() needs to change to something like: folio = filemap_get_folio(inode->i_mapping, index); if (!folio) return -ENOMEM; David
