From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yuanhan Liu Subject: Re: [PATCH] vhost: fix use after free Date: Wed, 19 Apr 2017 09:05:01 +0800 Message-ID: <20170419010501.GP7333@yliu-dev.sh.intel.com> References: <1492414024-18413-1-git-send-email-yuanhan.liu@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Maxime Coquelin Return-path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 8A9472BA2 for ; Wed, 19 Apr 2017 03:08:11 +0200 (CEST) Content-Disposition: inline In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Apr 18, 2017 at 10:20:41AM +0200, Maxime Coquelin wrote: > > > On 04/17/2017 09:27 AM, Yuanhan Liu wrote: > >A "return" is missing on error, which could lead to a "use after free" > >issue (about var "conn"). > > > >Fixes: 65388b43f592 ("vhost: fix fd leaks for vhost-user server mode") > >Coverity issue: 143476 > > > >Reported-by: John McNamara > >Signed-off-by: Yuanhan Liu > >--- > > lib/librte_vhost/socket.c | 1 + > > 1 file changed, 1 insertion(+) > > Reviewed-by: Maxime Coquelin Applied to dpdk-next-virtio. --yliu