Commit c8c1050e authored May 17, 2023 by Marcin Spiewak Committed by Jim Harris May 17, 2023

vbdev: Fixed wrong sizeof() argument



Fixed issue found by Coverity scan.
Wrong sizeof() argument was used when calling
spdk_mem_all_zero() function. It shall be sizeof(*uuid),
not sizeof(uuid).

Change-Id: Id33ddebff4a1e7cae21eba7102d0bf18f2ad9c69
Signed-off-by: Marcin Spiewak <marcin.spiewak@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18185


Community-CI: Mellanox Build Bot
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>

parent acb76c7a

module/bdev/delay/vbdev_delay.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -510,7 +510,7 @@ _delay_write_conf_values(struct vbdev_delay *delay_node, struct spdk_json_write_

		spdk_json_write_named_string(w, "name", spdk_bdev_get_name(&delay_node->delay_bdev));
		spdk_json_write_named_string(w, "base_bdev_name", spdk_bdev_get_name(delay_node->base_bdev));
		if (!spdk_mem_all_zero(uuid, sizeof(uuid))) {
		if (!spdk_mem_all_zero(uuid, sizeof(*uuid))) {
		spdk_uuid_fmt_lower(uuid_str, sizeof(uuid_str), uuid);
		spdk_json_write_named_string(w, "uuid", uuid_str);
		}