[Asterisk-code-review] res rtp asterisk: Issue NOTICE if res srtp is not found. (asterisk[11])

[Sean Bright]

Sean Bright has uploaded a new change for review.

  https://gerrit.asterisk.org/408

Change subject: res_rtp_asterisk: Issue NOTICE if res_srtp is not found.
......................................................................

res_rtp_asterisk: Issue NOTICE if res_srtp is not found.

While trying to get WebRTC working with chan_pjsip, I was running
into the following error:

    Attempted to set an invalid DTLS-SRTP configuration on RTP
    instance...

Josh helpfully pointed out that res_srtp.so might not be loaded, and
sure enough, it wasn't. This patch adds a NOTICE indiciating as much
to hopefully help others having a similar problem.

Change-Id: I13aa477b47b299876728a21b130998a0ea6cd19f
---
M res/res_rtp_asterisk.c
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/08/408/1

diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c
index 7389f9e..ca7fec2 100644
--- a/res/res_rtp_asterisk.c
+++ b/res/res_rtp_asterisk.c
@@ -1256,6 +1256,7 @@
 	}
 
 	if (!ast_rtp_engine_srtp_is_registered()) {
+		ast_log(LOG_NOTICE, "SRTP support module is not loaded or available. Try loading res_srtp.so.\n");
 		return -1;
 	}
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I13aa477b47b299876728a21b130998a0ea6cd19f
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>