From patchwork Fri Sep 21 16:25:57 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Luse, Paul E" X-Patchwork-Id: 45146 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 51A821B109; Fri, 21 Sep 2018 18:28:53 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id AEA495F48 for ; Fri, 21 Sep 2018 18:28:51 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 21 Sep 2018 09:28:50 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.54,285,1534834800"; d="scan'208";a="264638925" Received: from pels-64.ch.intel.com ([143.182.136.79]) by fmsmga005.fm.intel.com with ESMTP; 21 Sep 2018 09:25:18 -0700 From: Paul Luse To: dev@dpdk.org Cc: paul luse Date: Fri, 21 Sep 2018 12:25:57 -0400 Message-Id: <20180921162557.48249-1-paul.e.luse@intel.com> X-Mailer: git-send-email 2.14.4 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" From: paul luse 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 Acked-by: Ferruh Yigit --- 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");