[Asterisk-code-review] res pjsip mwi: Fix null pointer crash (asterisk[13])

Sean Bright asteriskteam at digium.com
Thu Feb 1 14:59:38 CST 2018


Sean Bright has posted comments on this change. ( https://gerrit.asterisk.org/8132 )

Change subject: res_pjsip_mwi: Fix null pointer crash
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/8132/2/res/res_pjsip_mwi.c
File res/res_pjsip_mwi.c:

https://gerrit.asterisk.org/#/c/8132/2/res/res_pjsip_mwi.c@a653
PS2, Line 653: 
Instead of the change you're proposing, just move this line into the:

 if (sub->is_solicited)

block below. resource is only used within that block anyway.



-- 
To view, visit https://gerrit.asterisk.org/8132
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: I78a0d38bfd8d0d82830f3d53da04872d6b67284d
Gerrit-Change-Number: 8132
Gerrit-PatchSet: 2
Gerrit-Owner: Joshua Elson <joshelson at gmail.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-Comment-Date: Thu, 01 Feb 2018 20:59:38 +0000
Gerrit-HasComments: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180201/e57d6394/attachment.html>


More information about the asterisk-code-review mailing list