pcp
[Top] [All Lists]

Re: [pcp] pcp updates

To: Max Matveev <makc@xxxxxxxxx>
Subject: Re: [pcp] pcp updates
From: Martin Hicks <mort@xxxxxxxx>
Date: Sat, 7 Mar 2009 12:18:07 -0500
Cc: Martin Hicks <mort@xxxxxxx>, pcp@xxxxxxxxxxx
In-reply-to: <18866.31663.926735.91528@xxxxxxxxxxxxxxxxx>
References: <20090306182731.GM30461@xxxxxxxxxxxxxxxxxxxxxxxxx> <18866.31663.926735.91528@xxxxxxxxxxxxxxxxx>
User-agent: Mutt/1.5.18 (2008-05-17)
On Sun, Mar 08, 2009 at 12:50:39AM +1100, Max Matveev wrote:
> On Fri, 6 Mar 2009 12:27:31 -0600, Martin Hicks wrote:
> 
> Strictly speaking you cannot rely on umad_release_port() not to stuff
> ca_name and portnum, the fact that it does not do it now, just like
> the fact that it didn't do any allocation at all in 1.2 timeframe,
> does not mean you're not going to get into some weird use-after-free
> next time OFED people change the library.

I realize that I'm exploiting a particular behaviour in the ibumad
library.  I've also filed a bug against the ibumad library and hopefully
someone will fix that in a future release.

As it stands now, I had a machine with about 300MB of leaked memory over
25 days due to this bug.

> PS. And it's tabs, not spaces for indentation.

It looks like the file is using a mix of tabs and spaces.

mh

<Prev in Thread] Current Thread [Next in Thread>