[Openib-windows] [PATCH] access to freed object in ioc_pnp

[Jan Bottorff]

Hi,

I downloaded 440, and still seem to see a hang on shutdown. If I ignore
the assert at:

*** Assertion failed: cl_status == CL_SUCCESS
***   Source File: k:\windows-openib\src\winib-440\core\al\al_common.c,
line 535

I unfortunately get an unusable dump of references objects, as you are
using %016I64x in the format strings, which on 32-bit Windows prints
garbage for the object type (as the object type pointer was just printed
in hex as part of the previous pointer). The format %p is supposed to
"do the right thing" for pointers on 32 or 64 bit Windows.

I'll try to fix the format strings, or else shift over to 64-bit Windows
for a while (I spend about 50/50 on each) and see if I can get a
meaningful reference dump.

- Jan


> -----Original Message-----
> From: ftillier.sst at gmail.com [mailto:ftillier.sst at gmail.com] On Behalf
Of
> Fabian Tillier
> Sent: Friday, August 04, 2006 12:33 AM
> To: Jan Bottorff
> Cc: Yossi Leybovich; openib-windows at openib.org
> Subject: Re: [Openib-windows] [PATCH] access to freed object in
ioc_pnp
> 
> On 8/3/06, Fabian Tillier <ftillier at silverstorm.com> wrote:
> 
> <snip>
> 
> > Stay tuned for a patch.
> 
> I just checked in a fix for this, in revision 439.  I haven't seen the
> double-free crash before, so I can't be certain that it is solved.
> The code did look wrong, and now no longer does.  Please give this a
> shot and let me know if you see any issues.
> 
> Thanks,
> 
> - Fab