Commit 3be9febd authored by Claire J. In's avatar Claire J. In Committed by Darek Stojaczyk
Browse files

lib/ftl: check for null 



Fixes #726.
This prevents a segmentation fault by checking the pointer
before accessing it.

Change-Id: I18a3776fbcd6242c6efd1b88044debaa88c7342f
Signed-off-by: default avatarClaire J. In <claire.in@circuitblvd.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/449591


Reviewed-by: default avatarwuzhouhui <wuzhouhui@kingsoft.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 643af88f

lib/ftl/ftl_io.c

+4 −0
Original line number Diff line number Diff line
@@ -256,6 +256,10 @@ ftl_io_erase_init(struct ftl_band *band, size_t lbk_cnt, spdk_ftl_fn cb) 
	};



	io = ftl_io_init_internal(&opts);

	if (!io) {

		return NULL;

	}



	io->lbk_cnt = lbk_cnt;



	return io;

lib/ftl/ftl_reloc.c

+6 −0
Original line number Diff line number Diff line
@@ -454,7 +454,13 @@ ftl_reloc_io_init(struct ftl_band_reloc *breloc, struct ftl_io *io, 
	}



	io = ftl_io_init_internal(&opts);

	if (!io) {

		spdk_dma_free(opts.data);

		return -1;

	}



	io->ppa = ppa;



	return 0;

}