Commit 0fb65adc authored by Eugene Kobyak's avatar Eugene Kobyak Committed by Tomasz Zawadzki
Browse files

bdev/ocf: extend snprintf size for null terminator char 



Function snpritf always add null terminator to the end of string
To save original size of core and cache names was passed incremented buffor size

Fixes #3107

Change-Id: I3773d240e41a6c0ed55f4b4f45cf3a6562c159ce
Signed-off-by: default avatarEugene Kobyak <eugene.kobyak@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19719


Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarKrzysztof Karas <krzysztof.karas@intel.com>
parent 49be050e

module/bdev/ocf/vbdev_ocf.c

+8 −2
Original line number Diff line number Diff line
@@ -1167,8 +1167,14 @@ init_vbdev_config(struct vbdev_ocf *vbdev) 
	ocf_mngt_cache_config_set_default(&cfg->cache);

	ocf_mngt_core_config_set_default(&cfg->core);



	snprintf(cfg->cache.name, sizeof(cfg->cache.name), "%s", vbdev->name);

	snprintf(cfg->core.name, sizeof(cfg->core.name), "%s", vbdev->core.name);

	ret = snprintf(cfg->cache.name, sizeof(cfg->cache.name), "%s", vbdev->name);

	if (ret < 0 || (size_t) ret >= sizeof(cfg->cache.name)) {

		return -EINVAL;

	}

	ret = snprintf(cfg->core.name, sizeof(cfg->core.name), "%s", vbdev->core.name);

	if (ret < 0 || (size_t) ret >= sizeof(cfg->core.name)) {

		return -EINVAL;

	}



	cfg->attach.open_cores = false;

	cfg->attach.device.perform_test = false;