On Mon, Apr 08, 2019 at 10:46:37AM +0100, Bruce Richardson wrote: > Coverity points out that there is a check in the main thread loop for the > ctrlr->bdev being NULL, but by that stage the pointer has already been > dereferenced. Therefore, for safety, before we enter the loop do an > initial check on the parameter structure. > > Coverity issue: 158657 > Fixes: db75c7af19bb ("examples/vhost_scsi: introduce a new sample app") > CC: sta...@dpdk.org > CC: Maxime Coquelin <maxime.coque...@redhat.com> > CC: Tiwei Bie <tiwei....@intel.com> > CC: Zhihong Wang <zhihong.w...@intel.com> > > Signed-off-by: Bruce Richardson <bruce.richard...@intel.com> > --- > examples/vhost_scsi/vhost_scsi.c | 6 ++++++ > 1 file changed, 6 insertions(+)
Reviewed-by: Tiwei Bie <tiwei....@intel.com>