From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcin Kerlin Subject: [PATCH 1/1] vhost: fix null pointer dereference Date: Wed, 15 Jun 2016 11:47:22 +0200 Message-ID: <1465984042-21634-1-git-send-email-marcinx.kerlin@intel.com> Cc: huawei.xie@intel.com, yuanhan.liu@linux.intel.com, dev@dpdk.org, Marcin Kerlin To: changchun.ouyang@intel.com Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 73988ADED for ; Wed, 15 Jun 2016 10:38:41 +0200 (CEST) List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Return value of function get_device() is not checking before dereference. Fix this problem by adding checking condition. Coverity issue: 119262 Fixes: 77d20126b4c2 ("vhost-user: handle message to enable vring") Signed-off-by: Marcin Kerlin --- lib/librte_vhost/vhost_user/virtio-net-user.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/librte_vhost/vhost_user/virtio-net-user.c b/lib/librte_vhost/vhost_user/virtio-net-user.c index f5248bc..94959f2 100644 --- a/lib/librte_vhost/vhost_user/virtio-net-user.c +++ b/lib/librte_vhost/vhost_user/virtio-net-user.c @@ -332,9 +332,13 @@ int user_set_vring_enable(struct vhost_device_ctx ctx, struct vhost_vring_state *state) { - struct virtio_net *dev = get_device(ctx); + struct virtio_net *dev; int enable = (int)state->num; + dev = get_device(ctx); + if (dev == NULL) + return -1; + RTE_LOG(INFO, VHOST_CONFIG, "set queue enable: %d to qp idx: %d\n", enable, state->index); -- 1.9.1