[asterisk-bugs] [JIRA] (ASTERISK-27961) res_pjsip: Spurious ERROR logging when printing headers in sip_msg

[Richard Mudgett (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-27961?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=244105#comment-244105 ] 

Richard Mudgett edited comment on ASTERISK-27961 at 7/13/18 11:35 AM:
----------------------------------------------------------------------

The 0060-sip_msg-Prevent-crash-on-header-without-vptr.patch file in the asterisk code base is not the patch that actually got committed into pjproject's code base in svn -r5727.

To fix the issue it looks like we just need to replace that patch file with the patch that got committed to pjproject instead.


was (Author: rmudgett):
The 0060-sip_msg-Prevent-crash-on-header-without-vptr.patch file in the asterisk code base is not the patch that actually got committed into pjproject's code base in svn -r5727.

To fis the issue it looks like we just need to replace that patch file with the patch that got committed to pjproject instead.

> res_pjsip: Spurious ERROR logging when printing headers in sip_msg
> ------------------------------------------------------------------
>
>                 Key: ASTERISK-27961
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-27961
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: pjproject/pjsip
>    Affects Versions: 13.22.0, 15.5.0
>            Reporter: Nick French
>            Severity: Trivial
>              Labels: pjsip
>
> Asterisk patches the pjproject source to avoid crashing when pjproject sip_msg headers are encountered with NULL vptr's. This logic is correct.
> However, in the same patch, error logging was added to announce when these NULL-vtpr headers were seen, but the logic of the logging is to mark an error for every printed header that results in no header text being added to the sip msg, not just when an actual NULL vptr is encountered.
> There exists types of headers that are perfectly valid yet result in no header text being added to the sip msg, such as a 'hidden' Route header.
> These types should not be treated as an error.



--
This message was sent by Atlassian JIRA
(v6.2#6252)