From sgrubb at redhat.com  Fri Jul 21 19:40:44 2017
From: sgrubb at redhat.com (Steve Grubb)
Date: Fri, 21 Jul 2017 13:40:44 -0400
Subject: [Rd] [PATCH] Fix bad free in connections
In-Reply-To: <25a319a3-d822-671f-06e9-5e2034f56c16@roswellpark.org>
References: <2125974.DtE9TkYUl0@x2>
 <25a319a3-d822-671f-06e9-5e2034f56c16@roswellpark.org>
Message-ID: <5540301.g3gtRBd9oT@x2>

On Friday, July 21, 2017 5:03:09 AM EDT Martin Morgan wrote:
> b gets reallocated when
> 
>      res = vasprintf(&b, format, ap);
> 
> is successful and res >= 0. usedVasprintf is then set to TRUE, and
> free(b) called.
> 
> It seems like the code is correct as written?

Yes, I think I see the issue. vasprintf seems to not be tracked for memory 
allocation. So, yes I agree its correct as written. Sorry for the noise.

-Steve



[Rd] [PATCH] Fix bad free in connections
Steve Grubb