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>

Reply via email to