[Asterisk-code-review] Add SDP translator and PJMEDIA implementation. (asterisk[master])
Mark Michelson
asteriskteam at digium.com
Tue Feb 14 10:30:19 CST 2017
Mark Michelson has uploaded a new change for review. ( https://gerrit.asterisk.org/4946 )
Change subject: Add SDP translator and PJMEDIA implementation.
......................................................................
Add SDP translator and PJMEDIA implementation.
This creates the following:
* Asterisk's internal representation of an SDP
* An API for translating SDPs from one format to another
* An implementation of a translator for PJMEDIA
Change-Id: Ie2ecd3cbebe76756577be9b133e84d2ee356d46b
---
M include/asterisk/sdp_options.h
A include/asterisk/sdp_priv.h
A include/asterisk/sdp_translator.h
A main/sdp_repr.c
A main/sdp_translator.c
A res/res_sdp_translator_pjmedia.c
6 files changed, 1,021 insertions(+), 0 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/46/4946/3
diff --git a/include/asterisk/sdp_options.h b/include/asterisk/sdp_options.h
index e780f8d..c77464f 100644
--- a/include/asterisk/sdp_options.h
+++ b/include/asterisk/sdp_options.h
@@ -110,6 +110,8 @@
/*! SDP is represented as a pjmedia_sdp_session */
AST_SDP_REPR_PJMEDIA,
#endif
+ /*! End of the list */
+ AST_SDP_REPR_END,
};
/*!
diff --git a/include/asterisk/sdp_priv.h b/include/asterisk/sdp_priv.h
new file mode 100644
index 0000000..000d111
--- /dev/null
+++ b/include/asterisk/sdp_priv.h
@@ -0,0 +1,130 @@
+/*
+ * Asterisk -- An open source telephony toolkit.
+ *
+ * Copyright (C) 2017, Digium, Inc.
+ *
+ * Mark Michelson <mmichelson at digium.com>
+ *
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
+ *
+ * This program is free software, distributed under the terms of
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
+ */
+
+/* NOTE: It is unlikely that you need to include this file. You probably will only need
+ * this if you are an SDP translator, or if you are an inner part of the SDP API
+ */
+
+#ifndef _SDP_PRIV_H
+#define _SDP_PRIV_H
+
+#include "asterisk/vector.h"
+
+/*!
+ * \brief Structure representing an SDP attribute
+ */
+struct ast_sdp_a_line {
+ /*! Attribute name */
+ char *name;
+ /*! Attribute value. For attributes that have no value, this will be an empty string */
+ char *value;
+};
+
+/*!
+ * \brief Structure representing an SDP connection
+ */
+struct ast_sdp_c_line {
+ /* IP family string (e.g. IP4 or IP6) */
+ char *family;
+ /* Connection address. Can be an IP address or FQDN */
+ char *addr;
+};
+
+/*!
+ * \brief A collection of SDP attributes
+ */
+AST_VECTOR(ast_sdp_a_line_vector, struct ast_sdp_a_line);
+
+/*!
+ * \brief An SDP media stream
+ *
+ * This contains both the m line, as well as its
+ * constituent a lines.
+ */
+struct ast_sdp_m_line {
+ /*! Media type (e.g. "audio" or "video") */
+ char *type;
+ /*! Port number in m line */
+ uint16_t port;
+ /*! Number of ports specified in m line */
+ uint16_t port_count;
+ /*! RTP profile string (e.g. "RTP/AVP") */
+ char *profile;
+ /*! RTP payloads */
+ AST_VECTOR(, char *) payloads;
+ /*! Connection information for this media stream */
+ struct ast_sdp_c_line c_line;
+ /*! The attributes for this media stream */
+ struct ast_sdp_a_line_vector a_lines;
+};
+
+/*!
+ * \brief SDP time information
+ */
+struct ast_sdp_t_line {
+ /*! Session start time */
+ uint32_t start;
+ /*! Session end time */
+ uint32_t end;
+};
+
+/*!
+ * \brief An SDP
+ */
+struct ast_sdp {
+ /*! SDP Origin line */
+ struct {
+ /*! Origin user name */
+ char *user;
+ /*! Origin id */
+ uint32_t id;
+ /*! Origin version */
+ uint32_t version;
+ /*! Origin IP address family (e.g. "IP4" or "IP6") */
+ char *family;
+ /*! Origin address. Can be an IP address or FQDN */
+ char *addr;
+ } o_line;
+ /*! SDP Session name */
+ char *s_line;
+ /*! SDP top-level connection information */
+ struct ast_sdp_c_line c_line;
+ /*! SDP timing information */
+ struct ast_sdp_t_line t_line;
+ /*! SDP top-level attributes */
+ struct ast_sdp_a_line_vector a_lines;
+ /*! SDP media streams */
+ AST_VECTOR(, struct ast_sdp_m_line) m_lines;
+};
+
+/*!
+ * \brief Allocate a new SDP.
+ *
+ * \note This does not perform any initialization.
+ *
+ * \retval NULL FAIL
+ * \retval non-NULL New SDP
+ */
+struct ast_sdp *ast_sdp_alloc(void);
+
+/*!
+ * \brief Free an SDP and all its constituent parts
+ */
+void ast_sdp_free(struct ast_sdp *dead);
+
+#endif /* _SDP_PRIV_H */
diff --git a/include/asterisk/sdp_translator.h b/include/asterisk/sdp_translator.h
new file mode 100644
index 0000000..cc1bd26
--- /dev/null
+++ b/include/asterisk/sdp_translator.h
@@ -0,0 +1,102 @@
+/*
+ * Asterisk -- An open source telephony toolkit.
+ *
+ * Copyright (C) 2017, Digium, Inc.
+ *
+ * Mark Michelson <mmichelson at digium.com>
+ *
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
+ *
+ * This program is free software, distributed under the terms of
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
+ */
+
+#ifndef _ASTERISK_SDP_STATE_H
+#define _ASTERISK_SDP_STATE_H
+
+#include "asterisk/sdp_options.h"
+
+struct sdp;
+
+/*!
+ * \brief SDP translator operations
+ */
+struct ast_sdp_translator_ops {
+ /*! The SDP representation on which this translator operates */
+ enum ast_sdp_options_repr repr;
+ /*! Allocate new translator private data for a translator */
+ void *(*translator_new)(void);
+ /*! Free translator private data */
+ void (*translator_free)(void *translator_priv);
+ /*! Convert the channel-native SDP into an internal Asterisk SDP */
+ struct ast_sdp *(*to_sdp)(void *repr_sdp, void *translator_priv);
+ /*! Convert an internal Asterisk SDP into a channel-native SDP */
+ void *(*from_sdp)(struct ast_sdp *sdp, void *translator_priv);
+};
+
+/*!
+ * \brief An SDP translator
+ *
+ * An SDP translator is responsible for converting between Asterisk's internal
+ * representation of an SDP and the representation that is native to the channel
+ * driver. Translators are allocated per-use.
+ */
+struct ast_sdp_translator {
+ /*! The operations this translator uses */
+ struct ast_sdp_translator_ops *ops;
+ /*! Private data this translator uses */
+ void *translator_priv;
+};
+
+/*!
+ * \brief Register an SDP translator
+ * \param ops The SDP operations defined by this translator
+ * \retval 0 Success
+ * \retval -1 FAIL
+ */
+int ast_sdp_register_translator(struct ast_sdp_translator_ops *ops);
+
+/*!
+ * \brief Unregister an SDP translator
+ */
+void ast_sdp_unregister_translator(struct ast_sdp_translator_ops *ops);
+
+/*!
+ * \brief Allocate a new SDP translator
+ * \param Representation corresponding to the translator_ops to use
+ * \retval NULL FAIL
+ * \retval non-NULL New SDP translator
+ */
+struct ast_sdp_translator *ast_sdp_translator_new(enum ast_sdp_options_repr repr);
+
+/*!
+ * \brief Free an SDP translator
+ */
+void ast_sdp_translator_free(struct ast_sdp_translator *translator);
+
+/*!
+ * \brief Translate a native SDP to internal Asterisk SDP
+ *
+ * \param translator The translator to use when translating
+ * \param native_sdp The SDP from the channel driver
+ * \retval NULL FAIL
+ * \retval Non-NULL The translated SDP
+ */
+struct ast_sdp *ast_sdp_translator_to_sdp(struct ast_sdp_translator *translator, void *native_sdp);
+
+/*!
+ * \brief Translate an internal Asterisk SDP to a native SDP
+ *
+ * \param translator The translator to use when translating
+ * \param ast_sdp The Asterisk SDP to translate
+ * \retval NULL FAIL
+ * \retval non-NULL The translated SDP
+ */
+void *ast_sdp_translator_from_sdp(struct ast_sdp_translator *translator, struct ast_sdp *ast_sdp);
+
+#endif /* _ASTERISK_SDP_STATE_H */
diff --git a/main/sdp_repr.c b/main/sdp_repr.c
new file mode 100644
index 0000000..6df243b
--- /dev/null
+++ b/main/sdp_repr.c
@@ -0,0 +1,111 @@
+/*
+ * Asterisk -- An open source telephony toolkit.
+ *
+ * Copyright (C) 2017, Digium, Inc.
+ *
+ * Mark Michelson <mmichelson at digium.com>
+ *
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
+ *
+ * This program is free software, distributed under the terms of
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
+ */
+
+#include "asterisk.h"
+#include "asterisk/sdp_priv.h"
+#include "asterisk/utils.h"
+
+struct ast_sdp *ast_sdp_alloc(void)
+{
+ struct ast_sdp *new_sdp;
+
+ new_sdp = ast_calloc(1, sizeof *new_sdp);
+ return new_sdp;
+}
+
+static void free_o_line(struct ast_sdp *dead)
+{
+ ast_free(dead->o_line.user);
+ ast_free(dead->o_line.family);
+ ast_free(dead->o_line.addr);
+}
+
+static void free_s_line(struct ast_sdp *dead)
+{
+ ast_free(dead->s_line);
+}
+
+static void free_c_line(struct ast_sdp_c_line *c_line)
+{
+ ast_free(c_line->family);
+ ast_free(c_line->addr);
+}
+
+static void free_t_line(struct ast_sdp_t_line *t_line)
+{
+ return;
+}
+
+static void free_a_line(struct ast_sdp_a_line *a_line)
+{
+ ast_free(a_line->name);
+ ast_free(a_line->value);
+}
+
+static void free_a_lines(struct ast_sdp_a_line_vector *a_lines)
+{
+ int i;
+
+ for (i = 0; i < AST_VECTOR_SIZE(a_lines); ++i) {
+ free_a_line(AST_VECTOR_GET_ADDR(a_lines, i));
+ }
+ AST_VECTOR_FREE(a_lines);
+}
+
+static void free_m_line(struct ast_sdp_m_line *m_line)
+{
+ int i;
+
+ ast_free(m_line->type);
+ ast_free(m_line->profile);
+ free_c_line(&m_line->c_line);
+
+ for (i = 0; i < AST_VECTOR_SIZE(&m_line->payloads); ++i) {
+ ast_free(AST_VECTOR_GET(&m_line->payloads, i));
+ }
+ AST_VECTOR_FREE(&m_line->payloads);
+
+ free_a_lines(&m_line->a_lines);
+}
+
+static void free_m_lines(struct ast_sdp *dead)
+{
+ int i;
+
+ for (i = 0; i < AST_VECTOR_SIZE(&dead->m_lines); ++i) {
+ free_m_line(AST_VECTOR_GET_ADDR(&dead->m_lines, i));
+ }
+
+ AST_VECTOR_FREE(&dead->m_lines);
+}
+
+void ast_sdp_free(struct ast_sdp *dead)
+{
+ if (!dead) {
+ return;
+ }
+
+ free_o_line(dead);
+ free_s_line(dead);
+ free_c_line(&dead->c_line);
+ free_t_line(&dead->t_line);
+ free_a_lines(&dead->a_lines);
+ free_m_lines(dead);
+ ast_free(dead);
+}
+
diff --git a/main/sdp_translator.c b/main/sdp_translator.c
new file mode 100644
index 0000000..5426ae9
--- /dev/null
+++ b/main/sdp_translator.c
@@ -0,0 +1,99 @@
+/*
+ * Asterisk -- An open source telephony toolkit.
+ *
+ * Copyright (C) 2017, Digium, Inc.
+ *
+ * Mark Michelson <mmichelson at digium.com>
+ *
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
+ *
+ * This program is free software, distributed under the terms of
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
+ */
+
+#include "asterisk.h"
+#include "asterisk/sdp_options.h"
+#include "asterisk/sdp_translator.h"
+#include "asterisk/logger.h"
+#include "asterisk/utils.h"
+#include "asterisk/lock.h"
+
+AST_RWLOCK_DEFINE_STATIC(registered_ops_lock);
+static struct ast_sdp_translator_ops *registered_ops[AST_SDP_REPR_END];
+
+int ast_sdp_register_translator(struct ast_sdp_translator_ops *ops)
+{
+ SCOPED_WRLOCK(lock, ®istered_ops_lock);
+
+ if (ops->repr >= AST_SDP_REPR_END) {
+ ast_log(LOG_ERROR, "SDP translator has unrecognized representation\n");
+ return -1;
+ }
+
+ if (registered_ops[ops->repr] != NULL) {
+ ast_log(LOG_ERROR, "SDP_translator with this representation already registered\n");
+ return -1;
+ }
+
+ registered_ops[ops->repr] = ops;
+ ast_log(LOG_NOTICE, "Placed ops %p at slot %d\n", ops, ops->repr);
+ return 0;
+}
+
+void ast_sdp_unregister_translator(struct ast_sdp_translator_ops *ops)
+{
+ SCOPED_WRLOCK(lock, ®istered_ops_lock);
+
+ if (ops->repr >= AST_SDP_REPR_END) {
+ return;
+ }
+
+ registered_ops[ops->repr] = NULL;
+}
+
+struct ast_sdp_translator *ast_sdp_translator_new(enum ast_sdp_options_repr repr)
+{
+ struct ast_sdp_translator *translator;
+ SCOPED_RDLOCK(lock, ®istered_ops_lock);
+
+ if (registered_ops[repr] == NULL) {
+ ast_log(LOG_NOTICE, "No registered SDP translator with representation %d\n", repr);
+ return NULL;
+ }
+
+ translator = ast_calloc(1, sizeof(*translator));
+ if (!translator) {
+ return NULL;
+ }
+
+ translator->ops = registered_ops[repr];
+
+ translator->translator_priv = translator->ops->translator_new();
+ if (!translator->translator_priv) {
+ ast_free(translator);
+ return NULL;
+ }
+
+ return translator;
+}
+
+void ast_sdp_translator_free(struct ast_sdp_translator *translator)
+{
+ translator->ops->translator_free(translator->translator_priv);
+ ast_free(translator);
+}
+
+struct ast_sdp *ast_sdp_translator_to_sdp(struct ast_sdp_translator *translator, void *native_sdp)
+{
+ return translator->ops->to_sdp(native_sdp, translator->translator_priv);
+}
+
+void *ast_sdp_translator_from_sdp(struct ast_sdp_translator *translator, struct ast_sdp *ast_sdp)
+{
+ return translator->ops->from_sdp(ast_sdp, translator->translator_priv);
+}
diff --git a/res/res_sdp_translator_pjmedia.c b/res/res_sdp_translator_pjmedia.c
new file mode 100644
index 0000000..3f16a74
--- /dev/null
+++ b/res/res_sdp_translator_pjmedia.c
@@ -0,0 +1,577 @@
+/*
+ * Asterisk -- An open source telephony toolkit.
+ *
+ * Copyright (C) 2017, Digium, Inc.
+ *
+ * Mark Michelson <mmichelson at digium.com>
+ *
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
+ *
+ * This program is free software, distributed under the terms of
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
+ */
+
+#include "asterisk.h"
+#include "asterisk/sdp_translator.h"
+#include "asterisk/sdp_options.h"
+#include "asterisk/sdp_priv.h"
+#include "asterisk/vector.h"
+#include "asterisk/netsock2.h"
+#include "asterisk/utils.h"
+#include "asterisk/config.h"
+#include "asterisk/test.h"
+#include "asterisk/module.h"
+#ifdef HAVE_PJPROJECT
+#include <pjlib.h>
+#include <pjmedia.h>
+#endif
+
+/*** MODULEINFO
+ <depend>pjproject</depend>
+ <support_level>core</support_level>
+ ***/
+
+static pj_caching_pool sdp_caching_pool;
+
+
+static void *pjmedia_new(void)
+{
+ pj_pool_t *pool;
+
+ pool = pj_pool_create(&sdp_caching_pool.factory, "pjmedia sdp translator", 1024, 1024, NULL);
+
+ return pool;
+}
+
+static void pjmedia_free(void *translator_priv)
+{
+ pj_pool_t *pool = translator_priv;
+
+ pj_pool_release(pool);
+}
+
+static void copy_pj_str(char *dest, const pj_str_t *src, size_t size)
+{
+ memcpy(dest, pj_strbuf(src), size);
+ dest[size] = '\0';
+}
+
+static void dup_pj_str(char **dest, const pj_str_t *src)
+{
+ *dest = ast_malloc(pj_strlen(src) + 1);
+ copy_pj_str(*dest, src, pj_strlen(src));
+}
+
+static void pjmedia_copy_o_line(struct ast_sdp *new_sdp, struct pjmedia_sdp_session * pjmedia_sdp)
+{
+ dup_pj_str(&new_sdp->o_line.user, &pjmedia_sdp->origin.user);
+ new_sdp->o_line.id = pjmedia_sdp->origin.id;
+ new_sdp->o_line.version = pjmedia_sdp->origin.version;
+ dup_pj_str(&new_sdp->o_line.family, &pjmedia_sdp->origin.addr_type);
+ dup_pj_str(&new_sdp->o_line.addr, &pjmedia_sdp->origin.addr);
+}
+
+static void pjmedia_copy_s_line(struct ast_sdp *new_sdp, struct pjmedia_sdp_session *pjmedia_sdp)
+{
+ dup_pj_str(&new_sdp->s_line, &pjmedia_sdp->name);
+}
+
+static void pjmedia_copy_t_line(struct ast_sdp_t_line *new_t_line, struct pjmedia_sdp_session *pjmedia_sdp)
+{
+ new_t_line->start = pjmedia_sdp->time.start;
+ new_t_line->end = pjmedia_sdp->time.stop;
+}
+
+static void pjmedia_copy_c_line(struct ast_sdp_c_line *new_c_line, struct pjmedia_sdp_conn *conn)
+{
+ /* It's perfectly reasonable for a c line not to be present, especially within a media description */
+ if (!conn) {
+ return;
+ }
+
+ dup_pj_str(&new_c_line->family, &conn->addr_type);
+ dup_pj_str(&new_c_line->addr, &conn->addr);
+}
+
+static void pjmedia_copy_m_line(struct ast_sdp_m_line *new_m_line, struct pjmedia_sdp_media *pjmedia_m_line)
+{
+ int i;
+
+ dup_pj_str(&new_m_line->type, &pjmedia_m_line->desc.media);
+ new_m_line->port = pjmedia_m_line->desc.port;
+ new_m_line->port_count = pjmedia_m_line->desc.port_count;
+ dup_pj_str(&new_m_line->profile, &pjmedia_m_line->desc.transport);
+ pjmedia_copy_c_line(&new_m_line->c_line, pjmedia_m_line->conn);
+
+ AST_VECTOR_INIT(&new_m_line->payloads, pjmedia_m_line->desc.fmt_count);
+ for (i = 0; i < pjmedia_m_line->desc.fmt_count; ++i) {
+ ++new_m_line->payloads.current;
+ dup_pj_str(AST_VECTOR_GET_ADDR(&new_m_line->payloads, i), &pjmedia_m_line->desc.fmt[i]);
+ }
+}
+
+static void pjmedia_copy_a_lines(struct ast_sdp_a_line_vector *new_a_lines, pjmedia_sdp_attr **attr, unsigned int attr_count)
+{
+ int i;
+
+ AST_VECTOR_INIT(new_a_lines, attr_count);
+
+ for (i = 0; i < attr_count; ++i) {
+ struct ast_sdp_a_line *a_line;
+
+ ++new_a_lines->current;
+ a_line = AST_VECTOR_GET_ADDR(new_a_lines, i);
+ dup_pj_str(&a_line->name, &attr[i]->name);
+ dup_pj_str(&a_line->value, &attr[i]->value);
+ }
+}
+
+static void pjmedia_copy_m_lines(struct ast_sdp *new_sdp, struct pjmedia_sdp_session *pjmedia_sdp)
+{
+ int i;
+
+ AST_VECTOR_INIT(&new_sdp->m_lines, pjmedia_sdp->media_count);
+
+ for (i = 0; i < pjmedia_sdp->media_count; ++i) {
+ ++new_sdp->m_lines.current;
+
+ pjmedia_copy_m_line(AST_VECTOR_GET_ADDR(&new_sdp->m_lines, i), pjmedia_sdp->media[i]);
+ pjmedia_copy_a_lines(&AST_VECTOR_GET_ADDR(&new_sdp->m_lines, i)->a_lines, pjmedia_sdp->media[i]->attr, pjmedia_sdp->media[i]->attr_count);
+ }
+}
+
+static struct ast_sdp *pjmedia_to_sdp(void *in, void *translator_priv)
+{
+ struct pjmedia_sdp_session *pjmedia_sdp = in;
+
+ struct ast_sdp *new_sdp = ast_sdp_alloc();
+
+ pjmedia_copy_o_line(new_sdp, pjmedia_sdp);
+ pjmedia_copy_s_line(new_sdp, pjmedia_sdp);
+ pjmedia_copy_t_line(&new_sdp->t_line, pjmedia_sdp);
+ pjmedia_copy_c_line(&new_sdp->c_line, pjmedia_sdp->conn);
+ pjmedia_copy_a_lines(&new_sdp->a_lines, pjmedia_sdp->attr, pjmedia_sdp->attr_count);
+ pjmedia_copy_m_lines(new_sdp, pjmedia_sdp);
+
+ return new_sdp;
+}
+
+static void copy_o_line_pjmedia(pj_pool_t *pool, pjmedia_sdp_session *pjmedia_sdp, struct ast_sdp *sdp)
+{
+ pjmedia_sdp->origin.id = sdp->o_line.id;
+ pjmedia_sdp->origin.version = sdp->o_line.version;
+ pj_strdup2(pool, &pjmedia_sdp->origin.user, sdp->o_line.user);
+ pj_strdup2(pool, &pjmedia_sdp->origin.addr_type, sdp->o_line.family);
+ pj_strdup2(pool, &pjmedia_sdp->origin.addr, sdp->o_line.addr);
+ pj_strdup2(pool, &pjmedia_sdp->origin.net_type, "IN");
+}
+
+static void copy_s_line_pjmedia(pj_pool_t *pool, pjmedia_sdp_session *pjmedia_sdp, struct ast_sdp *sdp)
+{
+ pj_strdup2(pool, &pjmedia_sdp->name, sdp->s_line);
+}
+
+static void copy_t_line_pjmedia(pj_pool_t *pool, pjmedia_sdp_session *pjmedia_sdp, struct ast_sdp_t_line *t_line)
+{
+ pjmedia_sdp->time.start = t_line->start;
+ pjmedia_sdp->time.stop = t_line->end;
+}
+
+static void copy_c_line_pjmedia(pj_pool_t *pool, pjmedia_sdp_conn **conn, struct ast_sdp_c_line *c_line)
+{
+ pjmedia_sdp_conn *local_conn;
+ local_conn = PJ_POOL_ZALLOC_T(pool, pjmedia_sdp_conn);
+ pj_strdup2(pool, &local_conn->addr_type, c_line->family);
+ pj_strdup2(pool, &local_conn->addr, c_line->addr);
+ pj_strdup2(pool, &local_conn->net_type, "IN");
+
+ *conn = local_conn;
+}
+
+static void copy_a_lines_pjmedia(pj_pool_t *pool, pjmedia_sdp_session *pjmedia_sdp, struct ast_sdp_a_line_vector *a_lines)
+{
+ int i;
+
+ for (i = 0; i < AST_VECTOR_SIZE(a_lines); ++i) {
+ pjmedia_sdp_attr *attr;
+ pj_str_t value;
+
+ pj_strdup2(pool, &value, AST_VECTOR_GET(a_lines, i).value);
+ attr = pjmedia_sdp_attr_create(pool, AST_VECTOR_GET(a_lines, i).name, &value);
+ pjmedia_sdp_session_add_attr(pjmedia_sdp, attr);
+ }
+}
+
+static void copy_a_lines_pjmedia_media(pj_pool_t *pool, pjmedia_sdp_media *media, struct ast_sdp_a_line_vector *a_lines)
+{
+ int i;
+
+ for (i = 0; i < AST_VECTOR_SIZE(a_lines); ++i) {
+ pjmedia_sdp_attr *attr;
+ pj_str_t value;
+
+ pj_strdup2(pool, &value, AST_VECTOR_GET(a_lines, i).value);
+ attr = pjmedia_sdp_attr_create(pool, AST_VECTOR_GET(a_lines, i).name, &value);
+ pjmedia_sdp_media_add_attr(media, attr);
+ }
+}
+
+static void copy_m_line_pjmedia(pj_pool_t *pool, pjmedia_sdp_media *media, struct ast_sdp_m_line *m_line)
+{
+ int i;
+
+ media->desc.port = m_line->port;
+ media->desc.port_count = m_line->port_count;
+ pj_strdup2(pool, &media->desc.transport, m_line->profile);
+ pj_strdup2(pool, &media->desc.media, m_line->type);
+
+ for (i = 0; i < AST_VECTOR_SIZE(&m_line->payloads); ++i) {
+ pj_strdup2(pool, &media->desc.fmt[i], AST_VECTOR_GET(&m_line->payloads, i));
+ ++media->desc.fmt_count;
+ }
+ if (m_line->c_line.addr) {
+ copy_c_line_pjmedia(pool, &media->conn, &m_line->c_line);
+ }
+ copy_a_lines_pjmedia_media(pool, media, &m_line->a_lines);
+}
+
+static void copy_m_lines_pjmedia(pj_pool_t *pool, pjmedia_sdp_session *pjmedia_sdp, struct ast_sdp *sdp)
+{
+ int i;
+
+ for (i = 0; i < AST_VECTOR_SIZE(&sdp->m_lines); ++i) {
+ pjmedia_sdp_media *media;
+
+ media = PJ_POOL_ZALLOC_T(pool, pjmedia_sdp_media);
+ copy_m_line_pjmedia(pool, media, AST_VECTOR_GET_ADDR(&sdp->m_lines, i));
+ pjmedia_sdp->media[pjmedia_sdp->media_count] = media;
+ ++pjmedia_sdp->media_count;
+ }
+}
+
+static void *sdp_to_pjmedia(struct ast_sdp *sdp, void *translator_priv)
+{
+ pj_pool_t *pool = translator_priv;
+ pjmedia_sdp_session *pjmedia_sdp;
+
+ pjmedia_sdp = PJ_POOL_ZALLOC_T(pool, pjmedia_sdp_session);
+ copy_o_line_pjmedia(pool, pjmedia_sdp, sdp);
+ copy_s_line_pjmedia(pool, pjmedia_sdp, sdp);
+ copy_t_line_pjmedia(pool, pjmedia_sdp, &sdp->t_line);
+ copy_c_line_pjmedia(pool, &pjmedia_sdp->conn, &sdp->c_line);
+ copy_a_lines_pjmedia(pool, pjmedia_sdp, &sdp->a_lines);
+ copy_m_lines_pjmedia(pool, pjmedia_sdp, sdp);
+ return pjmedia_sdp;
+}
+
+static struct ast_sdp_translator_ops pjmedia_translator = {
+ .repr = AST_SDP_REPR_PJMEDIA,
+ .translator_new = pjmedia_new,
+ .translator_free = pjmedia_free,
+ .to_sdp = pjmedia_to_sdp,
+ .from_sdp = sdp_to_pjmedia,
+};
+
+#ifdef TEST_FRAMEWORK
+
+static int verify_s_line(char *s_line, char *expected)
+{
+ return strcmp(s_line, expected) == 0;
+}
+
+static int verify_c_line(struct ast_sdp_c_line *c_line, char *family, char *addr)
+{
+ return strcmp(c_line->family, family) == 0 && strcmp(c_line->addr, addr) == 0;
+}
+
+static int verify_t_line(struct ast_sdp_t_line *t_line, uint32_t start, uint32_t end)
+{
+ return t_line->start == start && t_line->end == end;
+}
+
+static int verify_m_line(struct ast_sdp *sdp, int index, char *type, int port, int port_count, char *profile, ...)
+{
+ struct ast_sdp_m_line *m_line;
+ int res;
+ va_list ap;
+ int i;
+
+ m_line = AST_VECTOR_GET_ADDR(&sdp->m_lines, index);
+
+ res = strcmp(m_line->type, type) == 0;
+ res |= m_line->port == port;
+ res |= m_line->port_count == port_count;
+ res |= strcmp(m_line->profile, profile) == 0;
+
+ va_start(ap, profile);
+ for (i = 0; i < AST_VECTOR_SIZE(&m_line->payloads); ++i) {
+ char *payload;
+
+ payload = va_arg(ap, char *);
+ if (!payload) {
+ res = -1;
+ break;
+ }
+ res |= strcmp(AST_VECTOR_GET(&m_line->payloads, i), payload) == 0;
+ }
+ va_end(ap);
+ return res;
+}
+
+static int verify_a_line(struct ast_sdp *sdp, int m_index, int a_index, char *name, char *value)
+{
+ struct ast_sdp_m_line *m_line;
+ struct ast_sdp_a_line *a_line;
+
+ m_line = AST_VECTOR_GET_ADDR(&sdp->m_lines, m_index);
+ a_line = AST_VECTOR_GET_ADDR(&m_line->a_lines, a_index);
+
+ return strcmp(a_line->name, name) == 0 && strcmp(a_line->value, value) == 0;
+}
+
+AST_TEST_DEFINE(pjmedia_to_sdp_test)
+{
+ struct ast_sdp_translator *translator;
+ pj_pool_t *pool;
+ char *sdp_str =
+ "v=0\r\n"
+ "o=alice 2890844526 2890844526 IN IP4 host.atlanta.example.com\r\n"
+ "s= \r\n"
+ "c=IN IP4 host.atlanta.example.com\r\n"
+ "t=0 0\r\n"
+ "m=audio 49170 RTP/AVP 0 8 97\r\n"
+ "a=rtpmap:0 PCMU/8000\r\n"
+ "a=rtpmap:8 PCMA/8000\r\n"
+ "a=rtpmap:97 iLBC/8000\r\n"
+ "a=sendrecv\r\n"
+ "m=video 51372 RTP/AVP 31 32\r\n"
+ "a=rtpmap:31 H261/90000\r\n"
+ "a=rtpmap:32 MPV/90000\r\n";
+ pjmedia_sdp_session *pjmedia_sdp;
+ struct ast_sdp *sdp = NULL;
+ pj_status_t status;
+ enum ast_test_result_state res = AST_TEST_PASS;
+
+ switch (cmd) {
+ case TEST_INIT:
+ info->name = "pjmedia_to_sdp";
+ info->category = "/main/sdp/";
+ info->summary = "PJMEDIA to SDP unit test";
+ info->description =
+ "Ensures PJMEDIA SDPs are translated correctly";
+ return AST_TEST_NOT_RUN;
+ case TEST_EXECUTE:
+ break;
+ }
+
+ pool = pj_pool_create(&sdp_caching_pool.factory, "pjmedia to sdp test", 1024, 1024, NULL);
+
+ translator = ast_sdp_translator_new(AST_SDP_REPR_PJMEDIA);
+ if (!translator) {
+ ast_test_status_update(test, "Failed to create SDP translator\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+ status = pjmedia_sdp_parse(pool, sdp_str, strlen(sdp_str), &pjmedia_sdp);
+ if (status != PJ_SUCCESS) {
+ ast_test_status_update(test, "Error parsing SDP\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+ sdp = ast_sdp_translator_to_sdp(translator, pjmedia_sdp);
+
+ if (strcmp(sdp->o_line.user, "alice")) {
+ ast_test_status_update(test, "Unexpected SDP user '%s'\n", sdp->o_line.user);
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (sdp->o_line.id != 2890844526) {
+ ast_test_status_update(test, "Unexpected SDP id '%u'\n", sdp->o_line.id);
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (sdp->o_line.version != 2890844526) {
+ ast_test_status_update(test, "Unexpected SDP version '%u'\n", sdp->o_line.version);
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (strcmp(sdp->o_line.family, "IP4")) {
+ ast_test_status_update(test, "Unexpected address family '%s'\n", sdp->o_line.family);
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (strcmp(sdp->o_line.addr, "host.atlanta.example.com")) {
+ ast_test_status_update(test, "Unexpected address '%s'\n", sdp->o_line.addr);
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+ if (!verify_s_line(sdp->s_line, " ")) {
+ ast_test_status_update(test, "Bad s line\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_c_line(&sdp->c_line, "IP4", "host.atlanta.example.com")) {
+ ast_test_status_update(test, "Bad c line\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_t_line(&sdp->t_line, 0, 0)) {
+ ast_test_status_update(test, "Bad t line\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+ if (!verify_m_line(sdp, 0, "audio", 49170, 1, "RTP/AVP", "0", "8", "97", NULL)) {
+ ast_test_status_update(test, "Bad m line 1\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_a_line(sdp, 0, 0, "rtpmap", "0 PCMU/8000")) {
+ ast_test_status_update(test, "Bad a line 1\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_a_line(sdp, 0, 1, "rtpmap", "8 PCMA/8000")) {
+ ast_test_status_update(test, "Bad a line 2\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_a_line(sdp, 0, 2, "rtpmap", "97 iLBC/8000")) {
+ ast_test_status_update(test, "Bad a line 3\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_a_line(sdp, 0, 3, "sendrecv", "")) {
+ ast_test_status_update(test, "Bad a line 3\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_m_line(sdp, 1, "video", 51372, 1, "RTP/AVP", "31", "32", NULL)) {
+ ast_test_status_update(test, "Bad m line 2\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_a_line(sdp, 1, 0, "rtpmap", "31 H261/90000")) {
+ ast_test_status_update(test, "Bad a line 4\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ } else if (!verify_a_line(sdp, 1, 1, "rtpmap", "32 MPV/90000")) {
+ ast_test_status_update(test, "Bad a line 5\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+cleanup:
+ ast_sdp_free(sdp);
+ ast_sdp_translator_free(translator);
+ pj_pool_release(pool);
+ return res;
+}
+
+AST_TEST_DEFINE(sdp_to_pjmedia_test)
+{
+ struct ast_sdp_translator *translator;
+ char *sdp_str =
+ "v=0\r\n"
+ "o=alice 2890844526 2890844526 IN IP4 host.atlanta.example.com\r\n"
+ "s= \r\n"
+ "c=IN IP4 host.atlanta.example.com\r\n"
+ "t=0 0\r\n"
+ "m=audio 49170 RTP/AVP 0 8 97\r\n"
+ "a=rtpmap:0 PCMU/8000\r\n"
+ "a=rtpmap:8 PCMA/8000\r\n"
+ "a=rtpmap:97 iLBC/8000\r\n"
+ "a=sendrecv\r\n"
+ "m=video 51372 RTP/AVP 31 32\r\n"
+ "a=rtpmap:31 H261/90000\r\n"
+ "a=rtpmap:32 MPV/90000\r\n\r\n";
+ pj_pool_t *pool;
+ pjmedia_sdp_session *pjmedia_sdp_orig;
+ pjmedia_sdp_session *pjmedia_sdp_dup;
+ struct ast_sdp *sdp = NULL;
+ pj_status_t status;
+ enum ast_test_result_state res = AST_TEST_PASS;
+
+ switch (cmd) {
+ case TEST_INIT:
+ info->name = "sdp_to_pjmedia";
+ info->category = "/main/sdp/";
+ info->summary = "SDP to PJMEDIA unit test";
+ info->description =
+ "Ensures PJMEDIA SDPs are translated correctly";
+ return AST_TEST_NOT_RUN;
+ case TEST_EXECUTE:
+ break;
+ }
+
+ pool = pj_pool_create(&sdp_caching_pool.factory, "pjmedia to sdp test", 1024, 1024, NULL);
+
+ translator = ast_sdp_translator_new(AST_SDP_REPR_PJMEDIA);
+ if (!translator) {
+ ast_test_status_update(test, "Failed to create SDP translator\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+ status = pjmedia_sdp_parse(pool, sdp_str, strlen(sdp_str), &pjmedia_sdp_orig);
+ if (status != PJ_SUCCESS) {
+ ast_test_status_update(test, "Error parsing SDP\n");
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+ sdp = ast_sdp_translator_to_sdp(translator, pjmedia_sdp_orig);
+ pjmedia_sdp_dup = ast_sdp_translator_from_sdp(translator, sdp);
+
+ if ((status = pjmedia_sdp_session_cmp(pjmedia_sdp_orig, pjmedia_sdp_dup, 0)) != PJ_SUCCESS) {
+ char buf[2048];
+ char errbuf[256];
+ ast_test_status_update(test, "SDPs aren't equal\n");
+ pjmedia_sdp_print(pjmedia_sdp_orig, buf, sizeof(buf));
+ ast_log(LOG_NOTICE, "Original SDP is %s\n", buf);
+ pjmedia_sdp_print(pjmedia_sdp_dup, buf, sizeof(buf));
+ ast_log(LOG_NOTICE, "New SDP is %s\n", buf);
+ pjmedia_strerror(status, errbuf, sizeof(errbuf));
+ ast_log(LOG_NOTICE, "PJMEDIA says %d: '%s'\n", status, errbuf);
+ res = AST_TEST_FAIL;
+ goto cleanup;
+ }
+
+cleanup:
+ ast_sdp_free(sdp);
+ ast_sdp_translator_free(translator);
+ pj_pool_release(pool);
+ return res;
+}
+
+#endif /* TEST_FRAMEWORK */
+
+static int load_module(void)
+{
+ if (ast_sdp_register_translator(&pjmedia_translator)) {
+ return AST_MODULE_LOAD_DECLINE;
+ }
+ pj_caching_pool_init(&sdp_caching_pool, NULL, 1024 * 1024);
+ AST_TEST_REGISTER(pjmedia_to_sdp_test);
+ AST_TEST_REGISTER(sdp_to_pjmedia_test);
+
+ return AST_MODULE_LOAD_SUCCESS;
+}
+
+static int unload_module(void)
+{
+ ast_sdp_unregister_translator(&pjmedia_translator);
+ pj_caching_pool_destroy(&sdp_caching_pool);
+ AST_TEST_UNREGISTER(pjmedia_to_sdp_test);
+ AST_TEST_REGISTER(sdp_to_pjmedia_test);
+ return 0;
+}
+
+static int reload_module(void)
+{
+ return 0;
+}
+
+AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_LOAD_ORDER, "PJMEDIA SDP Translator",
+ .support_level = AST_MODULE_SUPPORT_CORE,
+ .load = load_module,
+ .unload = unload_module,
+ .reload = reload_module,
+ .load_pri = AST_MODPRI_CHANNEL_DEPEND,
+);
--
To view, visit https://gerrit.asterisk.org/4946
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie2ecd3cbebe76756577be9b133e84d2ee356d46b
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
More information about the asterisk-code-review
mailing list