< Previous by Date Date Index Next by Date >
< Previous in Thread Thread Index Next in Thread >

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


https://dl.dropbox.com/u/16255670/transport_logger.zip

1) There is no DTLS support.
2) The source code is coupled with another change - iOS support.
3) I did not test with IPv6 yet.
You will have to use text compare software to see only transport logger changes.

iOS support is not finished piece of work yet.

If you want - I can prepare diffs in 1-2 weeks.


On Wed, Dec 19, 2012 at 7:10 PM, Karlsson <boost.regex@xxxxxxxxx> wrote:
> This is a very great job. Thanks
>
>
> On Wed, Dec 19, 2012 at 11:18 PM, Scott Godin <sgodin@xxxxxxxxxxxxxxx>
> wrote:
>>
>> That would be great thanks Dmytro!  At one time we had an idea to generate
>> a pcap file with all messaging, but it never got completed, this sounds like
>> a good framework for adding this type of functionality.
>>
>> Scott
>>
>>
>> On Wed, Dec 19, 2012 at 10:15 AM, Dmytro Bogovych
>> <dmytro.bogovych@xxxxxxxxx> wrote:
>>>
>>> BTW i have other mod for resiprocate - transport log interface.
>>>
>>> Interface is here:
>>> https://dl.dropbox.com/u/16255670/InternalTransport.hxx
>>>
>>> Its goal is to notify about incoming&outgoing SIP/STUN traffic.
>>> Useful for traffic dump without wireshark.
>>>
>>> Let me know if you want to apply it to svn - i will send all related
>>> source files.
>>>
>>>
>>>
>>> On Wed, Dec 19, 2012 at 3:38 PM, Scott Godin <sgodin@xxxxxxxxxxxxxxx>
>>> wrote:
>>> > Thanks again guys - new fix is also committed now.
>>> >
>>> > Scott
>>> >
>>> >
>>> > On Wed, Dec 19, 2012 at 4:13 AM, Karlsson <boost.regex@xxxxxxxxx>
>>> > wrote:
>>> >>
>>> >> 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
>>> >>> >>
>>> >>> >>
>>> >>> >
>>> >>
>>> >>
>>> >
>>
>>
>>
>> _______________________________________________
>> resiprocate-devel mailing list
>> resiprocate-devel@xxxxxxxxxxxxxxx
>> https://list.resiprocate.org/mailman/listinfo/resiprocate-devel
>
>