[asterisk-dev] [Code Review] 3998: res_pjsip: ami: Fix error in AMI output when no transport is associated to an endpoint

George Joseph reviewboard at asterisk.org
Wed Sep 17 18:43:28 CDT 2014


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

(Updated Sept. 17, 2014, 5:43 p.m.)


Review request for Asterisk Developers and Kevin Harwell.


Changes
-------

Added ASTERISK-24161.


Bugs: 24161 and 24331
    https://issues.asterisk.org/jira/browse/24161
    https://issues.asterisk.org/jira/browse/24331


Repository: Asterisk


Description
-------

When no transport is associated to an endpoint, the AMI output for PJSIPShowEndpoint indicates an error instead of silently ignoring the missing transport.

This patch causes the error to appear only if a transport was specified on the endpoint and the transport doesn't exist.  It also fixes an issue with counting the objects that were actually found.


Diffs
-----

  branches/12/res/res_pjsip_endpoint_identifier_ip.c 423274 
  branches/12/res/res_pjsip/pjsip_configuration.c 423274 
  branches/12/res/res_pjsip/location.c 423274 
  branches/12/res/res_pjsip/config_transport.c 423274 
  branches/12/res/res_pjsip/config_auth.c 423274 
  branches/12/include/asterisk/res_pjsip.h 423274 

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


Testing
-------

Checked the output of PJSIPShowEndpoint to make sure the error appears (or doesn't) correctly and that ListItems is set correctly.


Thanks,

George Joseph

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140917/52e5c091/attachment.html>


More information about the asterisk-dev mailing list