Commit 0c980660 authored Aug 03, 2022 by GangCao Committed by Jim Harris Aug 08, 2022

FTL: move assert earlier before accessing the field



Fix Klockwork issue.

Change-Id: Iae9557c152a745549c8963f4f0510ae829f871a4
Signed-off-by: GangCao <gang.cao@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/13860


Community-CI: Mellanox Build Bot
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Dong Yi <dongx.yi@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

parent 4070c65d

lib/ftl/mngt/ftl_mngt_md.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -103,10 +103,11 @@ persist(struct spdk_ftl_dev dev, struct ftl_mngt_process mngt,
		enum ftl_layout_region_type type)
		{
		struct ftl_layout *layout = &dev->layout;
		struct ftl_md *md = layout->md[type];
		struct ftl_md *md;

		assert(type < FTL_LAYOUT_REGION_TYPE_MAX);

		md = layout->md[type];
		if (!md) {
		ftl_mngt_fail_step(mngt);
		return;