[Asterisk-code-review] res pjsip log forwarder: Add ast sip get pjproject buildopt (asterisk[master])

George Joseph asteriskteam at digium.com
Thu Jan 14 16:02:48 CST 2016


George Joseph has uploaded a new patch set (#2).

Change subject: res_pjsip_log_forwarder:  Add ast_sip_get_pjproject_buildopt
......................................................................

res_pjsip_log_forwarder:  Add ast_sip_get_pjproject_buildopt

As a follow-on to the recent 'Add CLI "pjsip show buildopts"' patch,
a new ast_sip_get_pjproject_buildopt function has been added.  It
allows the caller to get the value of one of the buildopts.

The initial use case is retrieving the runtime value of
PJ_MAX_HOSTNAME to insure we don't send a hostname greater
than pjproject can handle.  Since it can differ between
the version of pjproject that Asterisk was compiled against
and the version of pjproject that Asterisk is running against,
we can't use the PJ_MAX_HOSTNAME macro directly in Asterisk
source code.

Since this function parses the output of pj_dump_config,
it should never be used during call processing.  If a
buildopt is needed during cal processing, the value
should be retrieved at startup and saved for later use.

Change-Id: Icd23c289e1190edf9c6f07cabf382144ccef5b31
---
M include/asterisk/res_pjsip.h
M res/res_pjsip_log_forwarder.c
2 files changed, 85 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/13/2013/2
-- 
To view, visit https://gerrit.asterisk.org/2013
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icd23c289e1190edf9c6f07cabf382144ccef5b31
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Anonymous Coward #1000019



More information about the asterisk-code-review mailing list