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

[Karlsson <boost.regex@xxxxxxxxx>]

Thanks for share, yes, I did same patch in the FdPoll.cxx, it's works fine, just report it to mailist then Scott can submit it to mainline.

Best regards,

On Wed, Dec 19, 2012 at 1:35 PM, Dmytro Bogovych <dmytro.bogovych@xxxxxxxxx> wrote:

It should help:
https://dl.dropbox.com/u/16255670/FdPoll.cxx

On Wed, Dec 19, 2012 at 5:02 AM, Karlsson <boost.regex@xxxxxxxxx> wrote:
> 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@xxxxxxxxxxxxxxx> 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@xxxxxxxxx> 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@xxxxxxxxx> 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@xxxxxxxxxxxxxxx
>>> > https://list.resiprocate.org/mailman/listinfo/resiprocate-devel
>>> _______________________________________________
>>> resiprocate-devel mailing list
>>> resiprocate-devel@xxxxxxxxxxxxxxx
>>> https://list.resiprocate.org/mailman/listinfo/resiprocate-devel
>>
>>
>