From f21b45db4920debe736b710951b4860fb420f306 Mon Sep 17 00:00:00 2001
From: Jonathan Rose <jrose@digium.com>
Date: Wed, 8 Apr 2015 18:15:33 +0000
Subject: [PATCH] res_pjsip_t38: Fix FAX failures when using PJSIP with
authentication
Without this patch, if a PJSIP endpoint with udptl enabled and authentication
set attempted to use sendFax, the FAX session would fail during setup. This
was because the invite issued in response to being auth challenged would cause
the PJSIP channel performing the FAX to receive a second T38 framehook and
this would cause frames to be consumed in an inappropriate manner.
ASTERISK-24933 #close
Reported by: Jonathan Rose
Review: https://reviewboard.asterisk.org/r/4577/
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/13@434425 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
res/res_pjsip_t38.c | 37 ++++++++++++++++++++++++++++++++-----
1 file changed, 32 insertions(+), 5 deletions(-)
diff --git a/res/res_pjsip_t38.c b/res/res_pjsip_t38.c
index 0298671377..c5cebc6c47 100644
--- a/res/res_pjsip_t38.c
+++ b/res/res_pjsip_t38.c
@@ -453,9 +453,15 @@ static void t38_masq(void *data, int framehook_id,
ast_framehook_detach(new_chan, framehook_id);
}
+static const struct ast_datastore_info t38_framehook_datastore = {
+ .type = "T38 framehook",
+};
+
/*! \brief Function called to attach T.38 framehook to channel when appropriate */
static void t38_attach_framehook(struct ast_sip_session *session)
{
+ int framehook_id;
+ struct ast_datastore *datastore = NULL;
static struct ast_framehook_interface hook = {
.version = AST_FRAMEHOOK_INTERFACE_VERSION,
.event_cb = t38_framehook,
@@ -463,17 +469,38 @@ static void t38_attach_framehook(struct ast_sip_session *session)
.chan_breakdown_cb = t38_masq,
};
- /* Only attach the framehook on the first outgoing INVITE or the first incoming INVITE */
- if ((session->inv_session->state != PJSIP_INV_STATE_NULL &&
- session->inv_session->state != PJSIP_INV_STATE_INCOMING) ||
- !session->endpoint->media.t38.enabled) {
+ /* Only attach the framehook if t38 is enabled for the endpoint */
+ if (!session->endpoint->media.t38.enabled) {
return;
}
- if (ast_framehook_attach(session->channel, &hook) < 0) {
+ /* Skip attaching the framehook if the T.38 datastore already exists for the channel */
+ ast_channel_lock(session->channel);
+ if ((datastore = ast_channel_datastore_find(session->channel, &t38_framehook_datastore, NULL))) {
+ ast_channel_unlock(session->channel);
+ return;
+ }
+ ast_channel_unlock(session->channel);
+
+ framehook_id = ast_framehook_attach(session->channel, &hook);
+ if (framehook_id < 0) {
ast_log(LOG_WARNING, "Could not attach T.38 Frame hook to channel, T.38 will be unavailable on '%s'\n",
ast_channel_name(session->channel));
+ return;
}
+
+ ast_channel_lock(session->channel);
+ datastore = ast_datastore_alloc(&t38_framehook_datastore, NULL);
+ if (!datastore) {
+ ast_log(LOG_ERROR, "Could not attach T.38 Frame hook to channel, T.38 will be unavailable on '%s'\n",
+ ast_channel_name(session->channel));
+ ast_framehook_detach(session->channel, framehook_id);
+ ast_channel_unlock(session->channel);
+ return;
+ }
+
+ ast_channel_datastore_add(session->channel, datastore);
+ ast_channel_unlock(session->channel);
}
/*! \brief Function called when an INVITE goes out */
--
GitLab