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

[reSIProcate] bugs found?


It looks like some of the changes made by the WebRTC patch may be
attempted bug fixes and not strictly WebRTC functionality.  Even without
concluding the WebSockets work, it may be desirable to merge some of
these into trunk, or fix the issues more formally somehow, as some of
the changes in the patch just suppress issues.

I've made a list of things that I notice:


http://svn.resiprocate.org/viewsvn/resiprocate/branches/b-webrtc/rutil/dns/RRCache.cxx?r1=9964&r2=9965&diff_format=l
(attempt to fix an assert)
I actually removed this change in the branch:
http://list.resiprocate.org/archive/resiprocate-commit/msg07140.html
and my instance of repro is running fine without it, so it is not a
strict dependency for WS to work.


http://svn.resiprocate.org/viewsvn/resiprocate/branches/b-webrtc/resip/stack/DnsResult.cxx?r1=9964&r2=9965&diff_format=l
(hiding an assert)
I've also removed that change and my own testing has been done without it:
http://list.resiprocate.org/archive/resiprocate-commit/msg07145.html


http://svn.resiprocate.org/viewsvn/resiprocate/branches/b-webrtc/resip/stack/Helper.cxx?r1=9964&r2=9965&diff_format=l
(hiding an assert)
also reverted as part of my own testing:
http://list.resiprocate.org/archive/resiprocate-commit/msg07147.html


http://svn.resiprocate.org/viewsvn/resiprocate/branches/b-webrtc/resip/stack/MessageFilterRule.cxx?r1=9964&r2=9965&diff_format=l
(commented out fix - do we need that code?)


http://svn.resiprocate.org/viewsvn/resiprocate/branches/b-webrtc/resip/stack/TransactionState.cxx?r1=9964&r2=9965&diff_format=l
(hiding an assert)