-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3954/
-----------------------------------------------------------

(Updated Sept. 10, 2014, 5:18 p.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed jcolp's feedback.


Bugs: AFS-155 and ASTERISK-24295
    https://issues.asterisk.org/jira/browse/AFS-155
    https://issues.asterisk.org/jira/browse/ASTERISK-24295


Repository: Asterisk


Description
-------

The crash on the issues is a result of an invalid transport configuration 
change when asterisk is restarted.  The attempt to send the qualify request 
fails and we cleaned up.  However, the callback is also called which results in 
a double unref of the objects involved.

* Fixed send_request_cb() and qualify_contact_cb() to not cleanup the token 
when the PJSIP event is PJSIP_EVENT_TRANSPORT_ERROR since the initial function 
call will do the clean up.

* Made send_request_cb() able to handle repeated challenges (Up to 10).

* Fix periodic endpoint qualify OPTIONS sched deletion race by avoiding it.  
The sched entry will no longer self stop and must be externally stopped.

* Added REF_DEBUG description tags to struct sched_data in pjsip_options.c.

* Fix some off-nominal ref leaks in schedule_qualify(), qualify_and_schedule().

* Reordered pjsip_options.c module start/stop code to cleanup better on error.


Diffs (updated)
-----

  /branches/13/res/res_pjsip/pjsip_options.c 422963 
  /branches/13/res/res_pjsip.c 422963 

Diff: https://reviewboard.asterisk.org/r/3954/diff/


Testing
-------

* With the qualify_frequency option enabled, added and removed a "local_net=" 
line in the transport section and restarted asterisk via "core restart now".  
Before the latest patch version, asterisk would crash.  With the new patch, it 
keeps on going.

* Set the qualify_frequency option to different values and reloaded res_pjsip 
each time.  The OPTIONS poll frequency changed, started, and stopped according 
to the new qualify_frequency value.


Thanks,

rmudgett

-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to