From 69642141bb4b93cadc6f967355ecdbc3afb587a5 Mon Sep 17 00:00:00 2001 From: Darek Stojaczyk Date: Mon, 15 Jul 2019 07:50:57 +0200 Subject: [PATCH] blobstore: fix unused variable warning on non-debug builds MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit gcc complains: blobstore.c: In function ‘_spdk_blob_load_cpl’: blobstore.c:978:12: warning: unused variable ‘max_md_lba’ [-Wunused-variable] Change-Id: If2875d2d83edce6d1b544d6a4f51e78fa760d752 Signed-off-by: Darek Stojaczyk Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/461750 Reviewed-by: Seth Howell Reviewed-by: Shuhei Matsumoto Reviewed-by: Changpeng Liu Tested-by: SPDK CI Jenkins --- lib/blob/blobstore.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/blob/blobstore.c b/lib/blob/blobstore.c index 08ac7027f..7580c9dd2 100644 --- a/lib/blob/blobstore.c +++ b/lib/blob/blobstore.c @@ -977,7 +977,9 @@ _spdk_blob_load_cpl(spdk_bs_sequence_t *seq, void *cb_arg, int bserrno) uint64_t next_lba = _spdk_bs_page_to_lba(blob->bs, blob->bs->md_start + next_page); uint64_t max_md_lba = _spdk_bs_page_to_lba(blob->bs, blob->bs->md_start + blob->bs->md_len); - assert(next_lba < max_md_lba); + if (next_lba >= max_md_lba) { + assert(false); + } /* Read the next page */ ctx->num_pages++;