From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH v2 1/1] eal: fix resource leak of mapped memory Date: Wed, 15 Jun 2016 13:05:34 +0300 Message-ID: <65c6a8a7-ade6-e77a-1c1a-42c6e37f4682@redhat.com> References: <1461083251-31140-1-git-send-email-marcinx.kerlin@intel.com> <1465918435-30973-1-git-send-email-marcinx.kerlin@intel.com> <68D830D942438745AD09BAFA99E33E8125EB2F@IRSMSX102.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: "dev@dpdk.org" To: "Kerlin, MarcinX" , "Gonzalez Monroy, Sergio" , "david.marchand@6wind.com" Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id CF699ADC3 for ; Wed, 15 Jun 2016 12:05:36 +0200 (CEST) In-Reply-To: <68D830D942438745AD09BAFA99E33E8125EB2F@IRSMSX102.ger.corp.intel.com> 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" On 06/15/2016 12:35 PM, Kerlin, MarcinX wrote: > Hi Sergio, > > Thanks for tips, I removed double casting and I leave (void *) casting because pointer hp is const and compiler returns warnings. If hp is something that needs freeing then it shouldn't be const in the first place. Please fix that instead. - Panu -