Re: [PATCH 1/2] Use correct g_free() function

From: Thomas Jarosch <thomas jarosch intra2net com>
To: Dan Williams <dcbw redhat com>
Cc: networkmanager-list gnome org
Subject: Re: [PATCH 1/2] Use correct g_free() function
Date: Wed, 05 Oct 2011 21:57:44 +0200

On 10/05/2011 07:40 PM, Dan Williams wrote:
> On Tue, 2011-10-04 at 22:22 +0200, Thomas Jarosch wrote:
>> Detected by cppcheck.
> 
> Thanks, applied.  Looks like the other two variables there (var1 and
> var2) also need to be freed with g_free() since they are allocated using
> glib functions.  I've added that to your patch.

Yes, very true. I've isolated the logic of this code
and reported it to the cppcheck project. Thanks!

Thomas

References:
- [PATCH 1/2] Use correct g_free() function
  - From: Thomas Jarosch
- Re: [PATCH 1/2] Use correct g_free() function
  - From: Dan Williams

[Date Prev][Date Next] [Thread Prev][Thread Next] [Thread Index] [Date Index] [Author Index]