[reSIProcate] Crash bug in the DnsSub.cxx and AresDns.cxx

[Karlsson]

I have tested the patch, it still crashed
in FdPollItemBase::~FdPollItemBase()
{
    mPollGrp->delPollItem(mPollHandle);
}

The mPollGrp is NULL.



On Tue, Dec 18, 2012 at 9:49 PM, Scott Godin <sgodin at sipspectrum.com> wrote:

> Thanks Guys!  I have committed this patch to SVN mainline and will get
> back ported into 1.8.6.
>
> Scott
>
>
> On Tue, Dec 18, 2012 at 4:32 AM, Dmytro Bogovych <
> dmytro.bogovych at gmail.com> wrote:
>
>> I can confirm this.
>> My workaround:
>> https://dl.dropbox.com/u/16255670/AresDns.cxx
>>
>>
>> On Tue, Dec 18, 2012 at 11:06 AM, Karlsson <boost.regex at gmail.com> wrote:
>> > In the 1.8.5, I got the crashed often, once I debug it, I think there
>> is a
>> > bug:
>> >
>> > In the DnsSub.cxx:
>> >
>> > DnsStub::~DnsStub()
>> > {
>> >    for (set<Query*>::iterator it = mQueries.begin(); it !=
>> mQueries.end();
>> > ++it)
>> >    {
>> >       delete *it;
>> >    }
>> >
>> >    setPollGrp(0);
>> >    delete mDnsProvider;
>> > }
>> >
>> > The parameter of  "setPollGrp" function is 0, so in the
>> > AresDns::setPollGrp(FdPollGrp *grp) function, if the
>> > mPollItems is not empty, will cause AresDnsPollItem::resetPollGrp crash.
>> >
>> >
>> >
>> >
>> >
>> >
>> >
>> >
>> > _______________________________________________
>> > resiprocate-devel mailing list
>> > resiprocate-devel at resiprocate.org
>> > https://list.resiprocate.org/mailman/listinfo/resiprocate-devel
>> _______________________________________________
>> resiprocate-devel mailing list
>> resiprocate-devel at resiprocate.org
>> https://list.resiprocate.org/mailman/listinfo/resiprocate-devel
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://list.resiprocate.org/pipermail/resiprocate-devel/attachments/20121219/3f7398b5/attachment.htm>