[Asterisk-code-review] res pjsip: Remove spurious error logging when printing sile... (asterisk[13])

Richard Mudgett asteriskteam at digium.com
Tue Jul 17 10:36:35 CDT 2018


Richard Mudgett has uploaded a new patch set (#2) to the change originally created by Nick French. ( https://gerrit.asterisk.org/9483 )

Change subject: res_pjsip:  Remove spurious error logging when printing silent headers
......................................................................

res_pjsip:  Remove spurious error logging when printing silent headers

Asterisk patched the pjproject source to avoid crashing when pjproject
sip_msg headers are encountered with NULL vptr's, but the patch also
output error messages for some valid headers which simply did not need
to be added to the message itself, such as hidden route headers.

pjproject has since applied a similar patch to their baseline to avoid
crashes, but their version also avoids the spurious error logging.

Lets use their patch instead.

ASTERISK-27961 #close

Change-Id: I2ddbd82c8da10e0dcc9807a48089d1f3c2d6e389
---
M third-party/pjproject/patches/0060-sip_msg-Prevent-crash-on-header-without-vptr.patch
1 file changed, 29 insertions(+), 20 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/83/9483/2
-- 
To view, visit https://gerrit.asterisk.org/9483
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2ddbd82c8da10e0dcc9807a48089d1f3c2d6e389
Gerrit-Change-Number: 9483
Gerrit-PatchSet: 2
Gerrit-Owner: Nick French <naf at ou.edu>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180717/3d857526/attachment.html>


More information about the asterisk-code-review mailing list