Boost logo

Boost :

Subject: Re: [boost] [serialization][trunk] Latest patch for void_cast.cpp causes segfaults
From: Hartmut Kaiser (hartmut.kaiser_at_[hidden])
Date: 2009-07-11 13:42:21


> 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




Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk