From patchwork Thu Sep 20 23:29:30 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: paul luse X-Patchwork-Id: 45079 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 41B224CA1; Fri, 21 Sep 2018 12:52:43 +0200 (CEST) Received: from fed1rmfepo202.cox.net (fed1rmfepo202.cox.net [68.230.241.147]) by dpdk.org (Postfix) with ESMTP id 7FAD8255 for ; Fri, 21 Sep 2018 01:29:34 +0200 (CEST) Received: from fed1rmimpo109.cox.net ([68.230.241.158]) by fed1rmfepo202.cox.net (InterMail vM.8.01.05.28 201-2260-151-171-20160122) with ESMTP id <20180920232932.PMVP21918.fed1rmfepo202.cox.net@fed1rmimpo109.cox.net> for ; Thu, 20 Sep 2018 19:29:32 -0400 Received: from peluse-mac01.local.cox.net ([72.208.200.83]) by fed1rmimpo109.cox.net with cox id dzVX1y00H1oTjk801zVYXW; Thu, 20 Sep 2018 19:29:32 -0400 X-CT-Class: Clean X-CT-Score: 0.00 X-CT-RefID: str=0001.0A09020D.5BA42D5C.0026, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0 X-CT-Spam: 0 X-Authority-Analysis: v=2.3 cv=LZ+IFQXi c=1 sm=1 tr=0 a=6l8qiDExQlB81PhAZ4kONg==:117 a=x7bEGLp0ZPQA:10 a=5vm-TzuqCy0A:10 a=kviXuzpPAAAA:8 a=XmRP4r3XG-JDnMdskkIA:9 a=qrIFiuKZe2vaD64auk6j:22 X-CM-Score: 0.00 Authentication-Results: cox.net; auth=pass (LOGIN) smtp.auth=peluse@cox.net From: paul luse To: dev@dpdk.org Date: Thu, 20 Sep 2018 16:29:30 -0700 Message-Id: <20180920232930.68311-1-peluse@cox.net> X-Mailer: git-send-email 2.17.1 (Apple Git-112) X-Mailman-Approved-At: Fri, 21 Sep 2018 12:52:40 +0200 Subject: [dpdk-dev] [PATCH] vdev: free reply.msgs memory for secondary process X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This patch fixes an issue caught with ASAN where a vdev_scan() to a secondary bus was failing to free some memory. Signed-off-by: paul luse Acked-by: Anatoly Burakov --- drivers/bus/vdev/vdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c index 69dee89a8..2566d6c0d 100644 --- a/drivers/bus/vdev/vdev.c +++ b/drivers/bus/vdev/vdev.c @@ -423,6 +423,7 @@ vdev_scan(void) mp_rep = &mp_reply.msgs[0]; resp = (struct vdev_param *)mp_rep->param; VDEV_LOG(INFO, "Received %d vdevs", resp->num); + free(mp_reply.msgs); } else VDEV_LOG(ERR, "Failed to request vdev from primary");