Re: [boost] [serialization][trunk] Latest patch for void_cast.cppcauses segfaults

Subject: Re: [boost] [serialization][trunk] Latest patch for void_cast.cppcauses segfaults
From: Robert Ramey (ramey_at_[hidden])
Date: 2009-07-11 16:33:28

If you're patch fixes it for you, It's still not right so I'll take another
look at it. I think your solution does in fact create a memory leak
which it is tempting to ignore.  Except that I'm concerned about
the dynamic unloading of DLLS which will need to remove
these records for the types no longer supported without that DLL.

Looking at it, I think I do see something that can be addressed.

Robert Ramey

Hartmut Kaiser wrote:
>>> I pretty sure I addressed this where it was first appearing.  All
>>> tests in
>>> test suite
>>> which were faililng before are now passing - EXCEPT one -
>>> test_no_rtti. It just happens that this test trips upon the same
>>> problem in a different
>>> place.  I'm looking into this now and hope to have it resolved soon.
>>
>> I'm attaching a patch fixing the problem for me, but I'm not sure if
>> this isn't introducing a memory leak. I still think the delete is
>> bogus. Doesn't the erase(*it) already delete the container element?
>>
>> Regards Hartmut
>>
>>>
>>> Robert Ramey
>>>
>>> Hartmut Kaiser wrote:
>>>> Robert,
>>>>
>>>>> I've got it on my desk top along with a couple others. I'm hoping
>>>>> to get all
>>>>> of these fixes in the next few days. I realize I'm overdue, but
>>>>> I'm stretched thin right now.
>>>>
>>>> Thanks for your fixes in SVN. I don't know whether those were
>>>> supposed to
>>>> fix this particular problem, but I'm afraid needing to report that
>>>> it's not fixed yet. I'm still seeing the same segmentation fault at
>>>> application shutdown :-(
>>>>
>>>> Thanks!
>>>> Regards Hartmut
>>>>
>>>>>
>>>>> Robert Ramey
>>>>>
>>>>> Hartmut Kaiser wrote:
>>>>>>> Sorry, I've been busy with some other stuff and a short trip.  I
>>>>> hope
>>>>>>> to
>>>>>>> look at this in the next few days.
>>>>>>
>>>>>> Any news on that? It would be really unfortunate if this bug
>>>>>> crept into the upcoming release, making it unusable for us.
>>>>>>
>>>>>> Regards Hartmut
>>>>>>
>>>>>>>
>>>>>>> Robert Ramey
>>>>>>>
>>>>>>> Hartmut Kaiser wrote:
>>>>>>>>>> apparently the latest patch to
>>>>>>>>>> boost/libs/serialization/src/void_cast.cpp
>>>>>>>>>> (rev.53078, svn:date: 2009-05-18T06:25:56.023190Z,
>>>>>>>>>> svn:author: ramey, svn:log: fix errors in extended_type_info
>>>>>>>>>> management) causes an existing application to segfault
>>>>>>>>>> during cleanup. The application was working properly before.
>>>>>>>>>> The attached patch fixes the issue.
>>>>>>>>>>
>>>>>>>>>> Ok to commit?
>>>>>>>>>
>>>>>>>>> Ping?
>>>>>>>>> Regards Hartmut
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>> _______________________________________________
>>>>>>>>> Unsubscribe & other changes:
>>>>>>>>> http://lists.boost.org/mailman/listinfo.cgi/boost
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Unsubscribe & other changes:
>>>>>>> http://lists.boost.org/mailman/listinfo.cgi/boost
>>>>>>
>>>>>> _______________________________________________
>>>>>> Unsubscribe & other changes:
>>>>>> http://lists.boost.org/mailman/listinfo.cgi/boost
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Unsubscribe & other changes:
>>>>> http://lists.boost.org/mailman/listinfo.cgi/boost
>>>>
>>>> _______________________________________________
>>>> Unsubscribe & other changes:
>>>> http://lists.boost.org/mailman/listinfo.cgi/boost
>>>
>>>
>>>
>>> _______________________________________________
>>> Unsubscribe & other changes:
>>> http://lists.boost.org/mailman/listinfo.cgi/boost
>>
>
>
>
>> _______________________________________________
>> Unsubscribe & other changes:
>> http://lists.boost.org/mailman/listinfo.cgi/boost