Eric Biggers <ebiggers3@xxxxxxxxx> wrote: > If big_key_alloc_buffer() fails to allocate one of the pages then some of > the pages may still be NULL here, causing __free_page() to crash. You need > to check for NULL first. Ah, yes. I incorrectly used free_page() first - and that does check for a NULL pointer. Applied your other changes also. David