[Asterisk-code-review] res rtp asterisk.c: Fix crash in RTCP DTLS operation. (asterisk[13])

[Richard Mudgett]

Richard Mudgett has posted comments on this change. ( https://gerrit.asterisk.org/5503 )

Change subject: res_rtp_asterisk.c: Fix crash in RTCP DTLS operation.
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.asterisk.org/#/c/5503/1/res/res_rtp_asterisk.c
File res/res_rtp_asterisk.c:

Line 5437: 				}
> Don't want to set rtp->rtcp->dtls.ssl = NULL as you did above?
No because immediately after freeing we are setting it to the same pointer as the rtp->dtls.ssl pointer.  rtcp-mux shares the same socket and encryption.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If6c64c79129961acfa4b3d63a864e8f6b664acc0
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: Yes