From c2ef2448fceb240f0f5b18727a00e56f22c4c942 Mon Sep 17 00:00:00 2001 From: GangCao Date: Mon, 28 Nov 2022 21:39:28 -0500 Subject: [PATCH] bdev/lvol: add NULL return check when getting the lvs_bdev Change-Id: Ibdb7b79143f39a3d1396345f186d9cdb2d55596c Signed-off-by: GangCao Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/15674 Reviewed-by: Changpeng Liu Reviewed-by: Aleksey Marchuk Tested-by: SPDK CI Jenkins --- module/bdev/lvol/vbdev_lvol.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/module/bdev/lvol/vbdev_lvol.c b/module/bdev/lvol/vbdev_lvol.c index ce57896d3..d8b5cab9b 100644 --- a/module/bdev/lvol/vbdev_lvol.c +++ b/module/bdev/lvol/vbdev_lvol.c @@ -1690,6 +1690,12 @@ vbdev_lvs_grow(struct spdk_lvol_store *lvs, req->cb_arg = cb_arg; req->lvol_store = lvs; req->lvs_bdev = vbdev_get_lvs_bdev_by_lvs(lvs); + if (req->lvs_bdev == NULL) { + SPDK_ERRLOG("Cannot get valid lvs_bdev\n"); + cb_fn(cb_arg, -EINVAL); + free(req); + return; + } if (_vbdev_lvs_are_lvols_closed(lvs)) { spdk_lvs_unload(lvs, _vbdev_lvs_grow_unload_cb, req);