Re: [PATCH] bcache: revert replacing IS_ERR_OR_NULL with IS_ERR again
Jens Axboe <[email protected]>
| Newsgroups | org.kernel.vger.linux-bcache,org.kernel.vger.linux-block,org.kernel.vger.stable |
|---|---|
| Message-ID | <[email protected]> |
On Mon, 02 Dec 2024 19:56:38 +0800, Coly Li wrote:
> Commit 028ddcac477b ("bcache: Remove unnecessary NULL point check in
> node allocations") leads a NULL pointer deference in cache_set_flush().
>
> 1721 if (!IS_ERR_OR_NULL(c->root))
> 1722 list_add(&c->root->list, &c->btree_cache);
>
> >From the above code in cache_set_flush(), if previous registration code
> fails before allocating c->root, it is possible c->root is NULL as what
> it is initialized. __bch_btree_node_alloc() never returns NULL but
> c->root is possible to be NULL at above line 1721.
>
> [...]
Applied, thanks!
[1/1] bcache: revert replacing IS_ERR_OR_NULL with IS_ERR again
commit: b2e382ae12a63560fca35050498e19e760adf8c0
Best regards,
--
Jens Axboe