Commit 488da916 authored by Marcin Spiewak's avatar Marcin Spiewak Committed by Tomasz Zawadzki
Browse files

bdevperf: fixed memory 'double free' 



Fixed issue detected by Coverity scan. The memory
allocated for 'config' structure could be freed
twice in some cases.
Now, when a memory for 'config ' is freed, the pointer
is set to NULL. If free(config) will be called second time,
it will take no action.

Change-Id: I1bd3f8b17503feae5975afbae9d96ad6ce19b67e
Signed-off-by: default avatarMarcin Spiewak <marcin.spiewak@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19974


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

examples/bdev/bdevperf/bdevperf.c

+1 −0
Original line number Diff line number Diff line
@@ -2319,6 +2319,7 @@ read_job_config(void) 
			config_set_cli_args(config);

			global_config = *config;

			free(config);

			config = NULL;

		} else {

			TAILQ_INSERT_TAIL(&job_config_list, config, link);

			n++;