BuildSystem: Remove unused variables.
[asterisk/asterisk.git] / main / udptl.c
index 42bf7af..5a491e6 100644 (file)
 /*
- * Asterisk -- A telephony toolkit for Linux.
+ * Asterisk -- An open source telephony toolkit.
  *
  * UDPTL support for T.38
- * 
+ *
  * Copyright (C) 2005, Steve Underwood, partly based on RTP code which is
- * Copyright (C) 1999-2006, Digium, Inc.
+ * Copyright (C) 1999-2009, Digium, Inc.
  *
  * Steve Underwood <steveu@coppice.org>
+ * Kevin P. Fleming <kpfleming@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
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
  *
  * A license has been granted to Digium (via disclaimer) for the use of
  * this code.
  */
 
-#include "asterisk.h"
+/*!
+ * \file
+ *
+ * \brief UDPTL support for T.38 faxing
+ *
+ *
+ * \author Mark Spencer <markster@digium.com>
+ * \author Steve Underwood <steveu@coppice.org>
+ * \author Kevin P. Fleming <kpfleming@digium.com>
+ *
+ * \page T38fax_udptl T.38 support :: UDPTL
+ *
+ * Asterisk supports T.38 fax passthrough, origination and termination. It does
+ * not support gateway operation. The only channel driver that supports T.38 at
+ * this time is chan_sip.
+ *
+ * UDPTL is handled very much like RTP. It can be reinvited to go directly between
+ * the endpoints, without involving Asterisk in the media stream.
+ *
+ * \b References:
+ * - chan_sip.c
+ * - udptl.c
+ * - app_fax.c
+ */
+
+/*! \li \ref udptl.c uses the configuration file \ref udptl.conf
+ * \addtogroup configuration_file Configuration Files
+ */
 
-ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
+/*!
+ * \page udptl.conf udptl.conf
+ * \verbinclude udptl.conf.sample
+ */
+
+/*** MODULEINFO
+       <support_level>core</support_level>
+ ***/
+
+#include "asterisk.h"
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
 #include <sys/time.h>
 #include <signal.h>
-#include <errno.h>
-#include <unistd.h>
-#include <netinet/in.h>
-#include <sys/time.h>
-#include <sys/socket.h>
-#include <arpa/inet.h>
 #include <fcntl.h>
 
 #include "asterisk/udptl.h"
 #include "asterisk/frame.h"
-#include "asterisk/logger.h"
-#include "asterisk/options.h"
 #include "asterisk/channel.h"
 #include "asterisk/acl.h"
-#include "asterisk/channel.h"
-#include "asterisk/config.h"
+#include "asterisk/config_options.h"
 #include "asterisk/lock.h"
 #include "asterisk/utils.h"
+#include "asterisk/netsock2.h"
 #include "asterisk/cli.h"
 #include "asterisk/unaligned.h"
-#include "asterisk/utils.h"
+
+/*** DOCUMENTATION
+       <configInfo name="udptl" language="en_US">
+               <configFile name="udptl.conf">
+                       <configObject name="global">
+                               <synopsis>Global options for configuring UDPTL</synopsis>
+                               <configOption name="udptlstart">
+                                       <synopsis>The start of the UDPTL port range</synopsis>
+                               </configOption>
+                               <configOption name="udptlend">
+                                       <synopsis>The end of the UDPTL port range</synopsis>
+                               </configOption>
+                               <configOption name="udptlchecksums">
+                                       <synopsis>Whether to enable or disable UDP checksums on UDPTL traffic</synopsis>
+                               </configOption>
+                               <configOption name="udptlfecentries">
+                                       <synopsis>The number of error correction entries in a UDPTL packet</synopsis>
+                               </configOption>
+                               <configOption name="udptlfecspan">
+                                       <synopsis>The span over which parity is calculated for FEC in a UDPTL packet</synopsis>
+                               </configOption>
+                               <configOption name="use_even_ports">
+                                       <synopsis>Whether to only use even-numbered UDPTL ports</synopsis>
+                               </configOption>
+                               <configOption name="t38faxudpec">
+                                       <synopsis>Removed</synopsis>
+                               </configOption>
+                               <configOption name="t38faxmaxdatagram">
+                                       <synopsis>Removed</synopsis>
+                               </configOption>
+                       </configObject>
+               </configFile>
+       </configInfo>
+***/
 
 #define UDPTL_MTU              1200
 
@@ -55,19 +121,17 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #define TRUE (!FALSE)
 #endif
 
-static int udptlstart;
-static int udptlend;
-static int udptldebug;                   /* Are we debugging? */
-static struct sockaddr_in udptldebugaddr;   /* Debug packets to/from this host */
-#ifdef SO_NO_CHECK
-static int nochecksums;
-#endif
-static int udptlfectype;
-static int udptlfecentries;
-static int udptlfecspan;
-static int udptlmaxdatagram;
+#define LOG_TAG(u) S_OR(u->tag, "no tag")
+
+#define DEFAULT_UDPTLSTART 4000
+#define DEFAULT_UDPTLEND 4999
 
-#define LOCAL_FAX_MAX_DATAGRAM      400
+static int udptldebug;                     /*!< Are we debugging? */
+static struct ast_sockaddr udptldebugaddr;   /*!< Debug packets to/from this host */
+
+#define LOCAL_FAX_MAX_DATAGRAM      1400
+#define DEFAULT_FAX_MAX_DATAGRAM    400
+#define FAX_MAX_DATAGRAM_LIMIT      1400
 #define MAX_FEC_ENTRIES             5
 #define MAX_FEC_SPAN                5
 
@@ -81,12 +145,13 @@ typedef struct {
 typedef struct {
        int buf_len;
        uint8_t buf[LOCAL_FAX_MAX_DATAGRAM];
-       int fec_len[MAX_FEC_ENTRIES];
+       unsigned int fec_len[MAX_FEC_ENTRIES];
        uint8_t fec[MAX_FEC_ENTRIES][LOCAL_FAX_MAX_DATAGRAM];
-       int fec_span;
-       int fec_entries;
+       unsigned int fec_span;
+       unsigned int fec_entries;
 } udptl_fec_rx_buffer_t;
 
+/*! \brief Structure for an UDPTL session */
 struct ast_udptl {
        int fd;
        char resp;
@@ -95,77 +160,127 @@ struct ast_udptl {
        unsigned int lasteventseqn;
        int nat;
        int flags;
-       struct sockaddr_in us;
-       struct sockaddr_in them;
+       struct ast_sockaddr us;
+       struct ast_sockaddr them;
        int *ioid;
-       uint16_t seqno;
-       struct sched_context *sched;
+       struct ast_sched_context *sched;
        struct io_context *io;
        void *data;
+       char *tag;
        ast_udptl_callback callback;
-       int udptl_offered_from_local;
 
        /*! This option indicates the error correction scheme used in transmitted UDPTL
-           packets. */
-       int error_correction_scheme;
+        * packets and expected in received UDPTL packets.
+        */
+       enum ast_t38_ec_modes error_correction_scheme;
 
        /*! This option indicates the number of error correction entries transmitted in
-           UDPTL packets. */
-       int error_correction_entries;
+        * UDPTL packets and expected in received UDPTL packets.
+        */
+       unsigned int error_correction_entries;
 
        /*! This option indicates the span of the error correction entries in transmitted
-           UDPTL packets (FEC only). */
-       int error_correction_span;
+        * UDPTL packets (FEC only).
+        */
+       unsigned int error_correction_span;
+
+       /*! The maximum size UDPTL packet that can be accepted by
+        * the remote device.
+        */
+       int far_max_datagram;
+
+       /*! The maximum size UDPTL packet that we are prepared to
+        * accept, or -1 if it hasn't been calculated since the last
+        * changes were applied to the UDPTL structure.
+        */
+       int local_max_datagram;
+
+       /*! The maximum IFP that can be submitted for sending
+        * to the remote device. Calculated from far_max_datagram,
+        * error_correction_scheme and error_correction_entries,
+        * or -1 if it hasn't been calculated since the last
+        * changes were applied to the UDPTL structure.
+        */
+       int far_max_ifp;
+
+       /*! The maximum IFP that the local endpoint is prepared
+        * to accept. Along with error_correction_scheme and
+        * error_correction_entries, used to calculate local_max_datagram.
+        */
+       int local_max_ifp;
+
+       unsigned int tx_seq_no;
+       unsigned int rx_seq_no;
 
-       /*! This option indicates the maximum size of a UDPTL packet that can be accepted by
-           the remote device. */
-       int far_max_datagram_size;
+       udptl_fec_tx_buffer_t tx[UDPTL_BUF_MASK + 1];
+       udptl_fec_rx_buffer_t rx[UDPTL_BUF_MASK + 1];
+};
 
-       /*! This option indicates the maximum size of a UDPTL packet that we are prepared to
-           accept. */
-       int local_max_datagram_size;
+struct udptl_global_options {
+       unsigned int start; /*< The UDPTL start port */
+       unsigned int end;   /*< The UDPTL end port */
+       unsigned int fecentries;
+       unsigned int fecspan;
+       unsigned int nochecksums;
+       unsigned int use_even_ports;
+};
 
-       int verbose;
+static AO2_GLOBAL_OBJ_STATIC(globals);
 
-       struct sockaddr_in far;
+struct udptl_config {
+       struct udptl_global_options *general;
+};
 
-       int tx_seq_no;
-       int rx_seq_no;
-       int rx_expected_seq_no;
+static void *udptl_snapshot_alloc(void);
+static int udptl_pre_apply_config(void);
 
-       udptl_fec_tx_buffer_t tx[UDPTL_BUF_MASK + 1];
-       udptl_fec_rx_buffer_t rx[UDPTL_BUF_MASK + 1];
+static struct aco_type general_option = {
+       .type = ACO_GLOBAL,
+       .name = "global",
+       .category_match = ACO_WHITELIST_EXACT,
+       .item_offset = offsetof(struct udptl_config, general),
+       .category = "general",
 };
 
-static struct ast_udptl_protocol *protos;
+static struct aco_type *general_options[] = ACO_TYPES(&general_option);
+
+static struct aco_file udptl_conf = {
+       .filename = "udptl.conf",
+       .types = ACO_TYPES(&general_option),
+};
 
-static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len);
-static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, int ifp_len);
+CONFIG_INFO_CORE("udptl", cfg_info, globals, udptl_snapshot_alloc,
+       .files = ACO_FILES(&udptl_conf),
+       .pre_apply_config = udptl_pre_apply_config,
+);
 
-static inline int udptl_debug_test_addr(struct sockaddr_in *addr)
+static inline int udptl_debug_test_addr(const struct ast_sockaddr *addr)
 {
        if (udptldebug == 0)
                return 0;
-       if (udptldebugaddr.sin_addr.s_addr) {
-               if (((ntohs(udptldebugaddr.sin_port) != 0)
-                       && (udptldebugaddr.sin_port != addr->sin_port))
-                       || (udptldebugaddr.sin_addr.s_addr != addr->sin_addr.s_addr))
-                       return 0;
+
+       if (ast_sockaddr_isnull(&udptldebugaddr)) {
+               return 1;
+       }
+
+       if (ast_sockaddr_port(&udptldebugaddr)) {
+               return !ast_sockaddr_cmp(&udptldebugaddr, addr);
+       } else {
+               return !ast_sockaddr_cmp_addr(&udptldebugaddr, addr);
        }
-       return 1;
 }
 
-static int decode_length(uint8_t *buf, int limit, int *len, int *pvalue)
+static int decode_length(uint8_t *buf, unsigned int limit, unsigned int *len, unsigned int *pvalue)
 {
+       if (*len >= limit)
+               return -1;
        if ((buf[*len] & 0x80) == 0) {
-               if (*len >= limit)
-                       return -1;
                *pvalue = buf[*len];
                (*len)++;
                return 0;
        }
        if ((buf[*len] & 0x40) == 0) {
-               if (*len >= limit - 1)
+               if (*len == limit - 1)
                        return -1;
                *pvalue = (buf[*len] & 0x3F) << 8;
                (*len)++;
@@ -173,48 +288,37 @@ static int decode_length(uint8_t *buf, int limit, int *len, int *pvalue)
                (*len)++;
                return 0;
        }
-       if (*len >= limit)
-               return -1;
        *pvalue = (buf[*len] & 0x3F) << 14;
        (*len)++;
-       /* Indicate we have a fragment */
+       /* We have a fragment.  Currently we don't process fragments. */
+       ast_debug(1, "UDPTL packet with length greater than 16K received, decoding will fail\n");
        return 1;
 }
 /*- End of function --------------------------------------------------------*/
 
-static int decode_open_type(uint8_t *buf, int limit, int *len, const uint8_t **p_object, int *p_num_octets)
+static int decode_open_type(uint8_t *buf, unsigned int limit, unsigned int *len, const uint8_t **p_object, unsigned int *p_num_octets)
 {
-       int octet_cnt;
-       int octet_idx;
-       int stat;
-       int i;
-       const uint8_t **pbuf;
-
-       for (octet_idx = 0, *p_num_octets = 0; ; octet_idx += octet_cnt) {
-               if ((stat = decode_length(buf, limit, len, &octet_cnt)) < 0)
-                       return -1;
-               if (octet_cnt > 0) {
-                       *p_num_octets += octet_cnt;
+       unsigned int octet_cnt = 0;
 
-                       pbuf = &p_object[octet_idx];
-                       i = 0;
-                       /* Make sure the buffer contains at least the number of bits requested */
-                       if ((*len + octet_cnt) > limit)
-                               return -1;
+       if (decode_length(buf, limit, len, &octet_cnt) != 0)
+               return -1;
 
-                       *pbuf = &buf[*len];
-                       *len += octet_cnt;
-               }
-               if (stat == 0)
-                       break;
+       /* Make sure the buffer contains at least the number of bits requested */
+       if ((*len + octet_cnt) > limit) {
+               return -1;
        }
+
+       *p_num_octets = octet_cnt;
+       *p_object = &buf[*len];
+       *len += octet_cnt;
+
        return 0;
 }
 /*- End of function --------------------------------------------------------*/
 
-static int encode_length(uint8_t *buf, int *len, int value)
+static unsigned int encode_length(uint8_t *buf, unsigned int *len, unsigned int value)
 {
-       int multiplier;
+       unsigned int multiplier;
 
        if (value < 0x80) {
                /* 1 octet */
@@ -240,10 +344,11 @@ static int encode_length(uint8_t *buf, int *len, int value)
 }
 /*- End of function --------------------------------------------------------*/
 
-static int encode_open_type(uint8_t *buf, int *len, const uint8_t *data, int num_octets)
+static int encode_open_type(const struct ast_udptl *udptl, uint8_t *buf, unsigned int buflen,
+                           unsigned int *len, const uint8_t *data, unsigned int num_octets)
 {
-       int enclen;
-       int octet_idx;
+       unsigned int enclen;
+       unsigned int octet_idx;
        uint8_t zero_byte;
 
        /* If open type is of zero length, add a single zero byte (10.1) */
@@ -254,8 +359,12 @@ static int encode_open_type(uint8_t *buf, int *len, const uint8_t *data, int num
        }
        /* Encode the open type */
        for (octet_idx = 0; ; num_octets -= enclen, octet_idx += enclen) {
-               if ((enclen = encode_length(buf, len, num_octets)) < 0)
+               enclen = encode_length(buf, len, num_octets);
+               if (enclen + *len > buflen) {
+                       ast_log(LOG_ERROR, "UDPTL (%s): Buffer overflow detected (%u + %u > %u)\n",
+                               LOG_TAG(udptl), enclen, *len, buflen);
                        return -1;
+               }
                if (enclen > 0) {
                        memcpy(&buf[*len], &data[octet_idx], enclen);
                        *len += enclen;
@@ -268,28 +377,19 @@ static int encode_open_type(uint8_t *buf, int *len, const uint8_t *data, int num
 }
 /*- End of function --------------------------------------------------------*/
 
-static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
+static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, unsigned int len)
 {
-       int stat;
+       int stat1;
        int stat2;
        int i;
-       int j;
-       int k;
-       int l;
-       int m;
-       int x;
-       int limit;
-       int which;
-       int ptr;
-       int count;
-       int total_count;
+       unsigned int ptr;       /* an index that keeps track of how much of the UDPTL packet has been processed */
        int seq_no;
-       const uint8_t *ifp;
-       const uint8_t *data;
-       int ifp_len;
+       const uint8_t *ifp = NULL;
+       const uint8_t *data = NULL;
+       unsigned int ifp_len = 0;
        int repaired[16];
-       const uint8_t *bufs[16];
-       int lengths[16];
+       const uint8_t *bufs[ARRAY_LEN(s->f) - 1];
+       unsigned int lengths[ARRAY_LEN(s->f) - 1];
        int span;
        int entries;
        int ifp_no;
@@ -305,7 +405,7 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
        ptr += 2;
 
        /* Break out the primary packet */
-       if ((stat = decode_open_type(buf, len, &ptr, &ifp, &ifp_len)) != 0)
+       if ((stat1 = decode_open_type(buf, len, &ptr, &ifp, &ifp_len)) != 0)
                return -1;
        /* Decode error_recovery */
        if (ptr + 1 > len)
@@ -315,30 +415,38 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
                if (seq_no > s->rx_seq_no) {
                        /* We received a later packet than we expected, so we need to check if we can fill in the gap from the
                           secondary packets. */
-                       total_count = 0;
+                       int total_count = 0;
                        do {
+                               unsigned int count;
                                if ((stat2 = decode_length(buf, len, &ptr, &count)) < 0)
                                        return -1;
-                               for (i = 0; i < count; i++) {
-                                       if ((stat = decode_open_type(buf, len, &ptr, &bufs[total_count + i], &lengths[total_count + i])) != 0)
+                               for (i = 0; i < count && total_count + i < ARRAY_LEN(bufs); i++) {
+                                       if ((stat1 = decode_open_type(buf, len, &ptr, &bufs[total_count + i], &lengths[total_count + i])) != 0) {
                                                return -1;
+                                       }
+                                       /* valid secondaries can contain zero-length packets that should be ignored */
+                                       if (!bufs[total_count + i] || !lengths[total_count + i]) {
+                                               /* drop the count of items to process and reuse the buffers that were just set */
+                                               i--;
+                                               count--;
+                                       }
                                }
-                               total_count += count;
+                               total_count += i;
                        }
-                       while (stat2 > 0);
+                       while (stat2 > 0 && total_count < ARRAY_LEN(bufs));
                        /* Step through in reverse order, so we go oldest to newest */
                        for (i = total_count; i > 0; i--) {
                                if (seq_no - i >= s->rx_seq_no) {
                                        /* This one wasn't seen before */
                                        /* Decode the secondary IFP packet */
-                                       //fprintf(stderr, "Secondary %d, len %d\n", seq_no - i, lengths[i - 1]);
+                                       ast_debug(3, "Recovering lost packet via secondary %d, len %u\n", seq_no - i, lengths[i - 1]);
                                        s->f[ifp_no].frametype = AST_FRAME_MODEM;
-                                       s->f[ifp_no].subclass = AST_MODEM_T38;
+                                       s->f[ifp_no].subclass.integer = AST_MODEM_T38;
 
                                        s->f[ifp_no].mallocd = 0;
-                                       //s->f[ifp_no].???seq_no = seq_no - i;
+                                       s->f[ifp_no].seqno = seq_no - i;
                                        s->f[ifp_no].datalen = lengths[i - 1];
-                                       s->f[ifp_no].data = (uint8_t *) bufs[i - 1];
+                                       s->f[ifp_no].data.ptr = (uint8_t *) bufs[i - 1];
                                        s->f[ifp_no].offset = 0;
                                        s->f[ifp_no].src = "UDPTL";
                                        if (ifp_no > 0)
@@ -348,26 +456,12 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
                                }
                        }
                }
-               /* If packets are received out of sequence, we may have already processed this packet from the error
-                  recovery information in a packet already received. */
-               if (seq_no >= s->rx_seq_no) {
-                       /* Decode the primary IFP packet */
-                       s->f[ifp_no].frametype = AST_FRAME_MODEM;
-                       s->f[ifp_no].subclass = AST_MODEM_T38;
-                       
-                       s->f[ifp_no].mallocd = 0;
-                       //s->f[ifp_no].???seq_no = seq_no;
-                       s->f[ifp_no].datalen = ifp_len;
-                       s->f[ifp_no].data = (uint8_t *) ifp;
-                       s->f[ifp_no].offset = 0;
-                       s->f[ifp_no].src = "UDPTL";
-                       if (ifp_no > 0)
-                               AST_LIST_NEXT(&s->f[ifp_no - 1], frame_list) = &s->f[ifp_no];
-                       AST_LIST_NEXT(&s->f[ifp_no], frame_list) = NULL;
-               }
        }
        else
        {
+               int j;
+               int l;
+               int x;
                /* FEC mode for error recovery */
                /* Our buffers cannot tolerate overlength IFP packets in FEC mode */
                if (ifp_len > LOCAL_FAX_MAX_DATAGRAM)
@@ -405,11 +499,14 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
                if (ptr + 1 > len)
                        return -1;
                entries = buf[ptr++];
+               if (entries > MAX_FEC_ENTRIES) {
+                       return -1;
+               }
                s->rx[x].fec_entries = entries;
 
                /* Decode the elements */
                for (i = 0; i < entries; i++) {
-                       if ((stat = decode_open_type(buf, len, &ptr, &data, &s->rx[x].fec_len[i])) != 0)
+                       if ((stat1 = decode_open_type(buf, len, &ptr, &data, &s->rx[x].fec_len[i])) != 0)
                                return -1;
                        if (s->rx[x].fec_len[i] > LOCAL_FAX_MAX_DATAGRAM)
                                return -1;
@@ -419,7 +516,7 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
 #if 0
                        fprintf(stderr, "FEC: ");
                        for (j = 0; j < s->rx[x].fec_len[i]; j++)
-                               fprintf(stderr, "%02X ", data[j]);
+                               fprintf(stderr, "%02hhX ", data[j]);
                        fprintf(stderr, "\n");
 #endif
                }
@@ -427,10 +524,19 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
                /* See if we can reconstruct anything which is missing */
                /* TODO: this does not comprehensively hunt back and repair everything that is possible */
                for (l = x; l != ((x - (16 - span*entries)) & UDPTL_BUF_MASK); l = (l - 1) & UDPTL_BUF_MASK) {
+                       int m;
                        if (s->rx[l].fec_len[0] <= 0)
                                continue;
                        for (m = 0; m < s->rx[l].fec_entries; m++) {
-                               limit = (l + m) & UDPTL_BUF_MASK;
+                               int k;
+                               int which;
+                               int limit = (l + m) & UDPTL_BUF_MASK;
+
+                               /* only repair buffers that actually exist! */
+                               if (seq_no <= (s->rx[l].fec_span * s->rx[l].fec_entries) - m) {
+                                       continue;
+                               }
+
                                for (which = -1, k = (limit - s->rx[l].fec_span * s->rx[l].fec_entries) & UDPTL_BUF_MASK; k != limit; k = (k + s->rx[l].fec_entries) & UDPTL_BUF_MASK) {
                                        if (s->rx[k].buf_len <= 0)
                                                which = (which == -1) ? k : -2;
@@ -452,12 +558,12 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
                        if (repaired[l]) {
                                //fprintf(stderr, "Fixed packet %d, len %d\n", j, l);
                                s->f[ifp_no].frametype = AST_FRAME_MODEM;
-                               s->f[ifp_no].subclass = AST_MODEM_T38;
-                       
+                               s->f[ifp_no].subclass.integer = AST_MODEM_T38;
+
                                s->f[ifp_no].mallocd = 0;
-                               //s->f[ifp_no].???seq_no = j;
+                               s->f[ifp_no].seqno = j;
                                s->f[ifp_no].datalen = s->rx[l].buf_len;
-                               s->f[ifp_no].data = s->rx[l].buf;
+                               s->f[ifp_no].data.ptr = s->rx[l].buf;
                                s->f[ifp_no].offset = 0;
                                s->f[ifp_no].src = "UDPTL";
                                if (ifp_no > 0)
@@ -466,29 +572,36 @@ static int udptl_rx_packet(struct ast_udptl *s, uint8_t *buf, int len)
                                ifp_no++;
                        }
                }
+       }
+
+       /* If packets are received out of sequence, we may have already processed this packet from the error
+          recovery information in a packet already received. */
+       if (seq_no >= s->rx_seq_no) {
                /* Decode the primary IFP packet */
                s->f[ifp_no].frametype = AST_FRAME_MODEM;
-               s->f[ifp_no].subclass = AST_MODEM_T38;
-                       
+               s->f[ifp_no].subclass.integer = AST_MODEM_T38;
+
                s->f[ifp_no].mallocd = 0;
-               //s->f[ifp_no].???seq_no = j;
+               s->f[ifp_no].seqno = seq_no;
                s->f[ifp_no].datalen = ifp_len;
-               s->f[ifp_no].data = (uint8_t *) ifp;
+               s->f[ifp_no].data.ptr = (uint8_t *) ifp;
                s->f[ifp_no].offset = 0;
                s->f[ifp_no].src = "UDPTL";
                if (ifp_no > 0)
                        AST_LIST_NEXT(&s->f[ifp_no - 1], frame_list) = &s->f[ifp_no];
                AST_LIST_NEXT(&s->f[ifp_no], frame_list) = NULL;
+
+               ifp_no++;
        }
 
        s->rx_seq_no = seq_no + 1;
-       return 0;
+       return ifp_no;
 }
 /*- End of function --------------------------------------------------------*/
 
-static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, int ifp_len)
+static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, unsigned int buflen, uint8_t *ifp, unsigned int ifp_len)
 {
-       uint8_t fec[LOCAL_FAX_MAX_DATAGRAM];
+       uint8_t fec[LOCAL_FAX_MAX_DATAGRAM * 2] = { 0, };
        int i;
        int j;
        int seq;
@@ -496,7 +609,7 @@ static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, i
        int entries;
        int span;
        int m;
-       int len;
+       unsigned int len;
        int limit;
        int high_tide;
 
@@ -509,7 +622,7 @@ static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, i
           redundancy sets later on. */
        s->tx[entry].buf_len = ifp_len;
        memcpy(s->tx[entry].buf, ifp, ifp_len);
-       
+
        /* Build the UDPTLPacket */
 
        len = 0;
@@ -518,7 +631,7 @@ static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, i
        buf[len++] = seq & 0xFF;
 
        /* Encode the primary IFP packet */
-       if (encode_open_type(buf, &len, ifp, ifp_len) < 0)
+       if (encode_open_type(s, buf, buflen, &len, ifp, ifp_len) < 0)
                return -1;
 
        /* Encode the appropriate type of error recovery information */
@@ -529,8 +642,7 @@ static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, i
                buf[len++] = 0x00;
                /* The number of entries will always be zero, so it is pointless allowing
                   for the fragmented case here. */
-               if (encode_length(buf, &len, 0) < 0)
-                       return -1;
+               encode_length(buf, &len, 0);
                break;
        case UDPTL_ERROR_CORRECTION_REDUNDANCY:
                /* Encode the error recovery type */
@@ -541,13 +653,15 @@ static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, i
                        entries = s->tx_seq_no;
                /* The number of entries will always be small, so it is pointless allowing
                   for the fragmented case here. */
-               if (encode_length(buf, &len, entries) < 0)
-                       return -1;
+               encode_length(buf, &len, entries);
                /* Encode the elements */
                for (i = 0; i < entries; i++) {
                        j = (entry - i - 1) & UDPTL_BUF_MASK;
-                       if (encode_open_type(buf, &len, s->tx[j].buf, s->tx[j].buf_len) < 0)
+                       if (encode_open_type(s, buf, buflen, &len, s->tx[j].buf, s->tx[j].buf_len) < 0) {
+                               ast_debug(1, "UDPTL (%s): Encoding failed at i=%d, j=%d\n",
+                                         LOG_TAG(s), i, j);
                                return -1;
+                       }
                }
                break;
        case UDPTL_ERROR_CORRECTION_FEC:
@@ -584,20 +698,17 @@ static int udptl_build_packet(struct ast_udptl *s, uint8_t *buf, uint8_t *ifp, i
                                                fec[j] ^= s->tx[i].buf[j];
                                }
                        }
-                       if (encode_open_type(buf, &len, fec, high_tide) < 0)
+                       if (encode_open_type(s, buf, buflen, &len, fec, high_tide) < 0)
                                return -1;
                }
                break;
        }
 
-       if (s->verbose)
-               fprintf(stderr, "\n");
-
        s->tx_seq_no++;
        return len;
 }
 
-int ast_udptl_fd(struct ast_udptl *udptl)
+int ast_udptl_fd(const struct ast_udptl *udptl)
 {
        return udptl->fd;
 }
@@ -632,195 +743,341 @@ static int udptlread(int *id, int fd, short events, void *cbdata)
 struct ast_frame *ast_udptl_read(struct ast_udptl *udptl)
 {
        int res;
-       struct sockaddr_in sin;
-       socklen_t len;
-       uint16_t seqno = 0;
-       uint16_t *udptlheader;
+       struct ast_sockaddr addr;
+       uint8_t *buf;
+
+       buf = udptl->rawdata + AST_FRIENDLY_OFFSET;
 
-       len = sizeof(sin);
-       
        /* Cache where the header will go */
-       res = recvfrom(udptl->fd,
-                       udptl->rawdata + AST_FRIENDLY_OFFSET,
+       res = ast_recvfrom(udptl->fd,
+                       buf,
                        sizeof(udptl->rawdata) - AST_FRIENDLY_OFFSET,
                        0,
-                       (struct sockaddr *) &sin,
-                       &len);
-       udptlheader = (uint16_t *)(udptl->rawdata + AST_FRIENDLY_OFFSET);
+                       &addr);
        if (res < 0) {
                if (errno != EAGAIN)
-                       ast_log(LOG_WARNING, "UDPTL read error: %s\n", strerror(errno));
-               if (errno == EBADF)
-                       CRASH;
+                       ast_log(LOG_WARNING, "UDPTL (%s): read error: %s\n",
+                               LOG_TAG(udptl), strerror(errno));
+               ast_assert(errno != EBADF);
                return &ast_null_frame;
        }
 
        /* Ignore if the other side hasn't been given an address yet. */
-       if (!udptl->them.sin_addr.s_addr || !udptl->them.sin_port)
+       if (ast_sockaddr_isnull(&udptl->them)) {
+               return &ast_null_frame;
+       }
+
+       /*
+        * If early media isn't turned on for the channel driver, it's going to
+        * drop this frame.  By that time though, udptl has already incremented
+        * the expected sequence number so if the CPE re-sends, the second frame
+        * will be dropped as a dup even though the first frame never went through.
+        * So we drop the frame here if the channel isn't up. 'tag' is set by the
+        * channel drivers on T38_ENABLED or T38_PEER_REINVITE.
+        */
+       if (udptl->tag == NULL) {
                return &ast_null_frame;
+       }
 
        if (udptl->nat) {
                /* Send to whoever sent to us */
-               if ((udptl->them.sin_addr.s_addr != sin.sin_addr.s_addr) ||
-                       (udptl->them.sin_port != sin.sin_port)) {
-                       memcpy(&udptl->them, &sin, sizeof(udptl->them));
-                       if (option_debug)
-                               ast_log(LOG_DEBUG, "UDPTL NAT: Using address %s:%d\n", ast_inet_ntoa(udptl->them.sin_addr), ntohs(udptl->them.sin_port));
+               if (ast_sockaddr_cmp(&udptl->them, &addr)) {
+                       ast_sockaddr_copy(&udptl->them, &addr);
+                       ast_debug(1, "UDPTL (%s): NAT, Using address %s\n",
+                                 LOG_TAG(udptl), ast_sockaddr_stringify(&udptl->them));
                }
        }
 
-       if (udptl_debug_test_addr(&sin)) {
-               if (option_verbose)
-                       ast_verbose("Got UDPTL packet from %s:%d (type %d, seq %d, len %d)\n",
-                               ast_inet_ntoa(sin.sin_addr), ntohs(sin.sin_port), 0, seqno, res);
+       if (udptl_debug_test_addr(&addr)) {
+               int seq_no;
+
+               /* Decode sequence number just for verbose message. */
+               if (res < 2) {
+                       /* Short packet. */
+                       seq_no = -1;
+               } else {
+                       seq_no = (buf[0] << 8) | buf[1];
+               }
+
+               ast_verb(1, "UDPTL (%s): packet from %s (seq %d, len %d)\n",
+                       LOG_TAG(udptl), ast_sockaddr_stringify(&addr), seq_no, res);
+       }
+       if (udptl_rx_packet(udptl, buf, res) < 1) {
+               return &ast_null_frame;
        }
-#if 0
-       printf("Got UDPTL packet from %s:%d (seq %d, len = %d)\n", ast_inet_ntoa(sin.sin_addr), ntohs(sin.sin_port), seqno, res);
-#endif
-       udptl_rx_packet(udptl, udptl->rawdata + AST_FRIENDLY_OFFSET, res);
 
        return &udptl->f[0];
 }
 
-void ast_udptl_offered_from_local(struct ast_udptl* udptl, int local)
+static void calculate_local_max_datagram(struct ast_udptl *udptl)
 {
-       if (udptl)
-               udptl->udptl_offered_from_local = local;
-       else
-               ast_log(LOG_WARNING, "udptl structure is null\n");
+       unsigned int new_max = 0;
+
+       if (udptl->local_max_ifp == -1) {
+               ast_log(LOG_WARNING, "UDPTL (%s): Cannot calculate local_max_datagram before local_max_ifp has been set.\n",
+                       LOG_TAG(udptl));
+               udptl->local_max_datagram = -1;
+               return;
+       }
+
+       /* calculate the amount of space required to receive an IFP
+        * of the maximum size supported by the application/endpoint
+        * that we are delivering them to (local endpoint), and add
+        * the amount of space required to support the selected
+        * error correction mode
+        */
+       switch (udptl->error_correction_scheme) {
+       case UDPTL_ERROR_CORRECTION_NONE:
+               /* need room for sequence number, length indicator, redundancy
+                * indicator and following length indicator
+                */
+               new_max = 5 + udptl->local_max_ifp;
+               break;
+       case UDPTL_ERROR_CORRECTION_REDUNDANCY:
+               /* need room for sequence number, length indicators, plus
+                * room for up to 3 redundancy packets
+                */
+               new_max = 5 + udptl->local_max_ifp + 2 + (3 * udptl->local_max_ifp);
+               break;
+       case UDPTL_ERROR_CORRECTION_FEC:
+               /* need room for sequence number, length indicators and a
+                * a single IFP of the maximum size expected
+                */
+               new_max = 5 + udptl->local_max_ifp + 4 + udptl->local_max_ifp;
+               break;
+       }
+       /* add 5% extra space for insurance, but no larger than LOCAL_FAX_MAX_DATAGRAM */
+       udptl->local_max_datagram = MIN(new_max * 1.05, LOCAL_FAX_MAX_DATAGRAM);
 }
 
-int ast_udptl_get_error_correction_scheme(struct ast_udptl* udptl)
+static void calculate_far_max_ifp(struct ast_udptl *udptl)
 {
-       if (udptl)
-               return udptl->error_correction_scheme;
-       else {
-               ast_log(LOG_WARNING, "udptl structure is null\n");
-               return -1;
+       unsigned new_max = 0;
+
+       if (udptl->far_max_datagram == -1) {
+               ast_log(LOG_WARNING, "UDPTL (%s): Cannot calculate far_max_ifp before far_max_datagram has been set.\n",
+                       LOG_TAG(udptl));
+               udptl->far_max_ifp = -1;
+               return;
        }
+
+       /* the goal here is to supply the local endpoint (application
+        * or bridged channel) a maximum IFP value that will allow it
+        * to effectively and efficiently transfer image data at its
+        * selected bit rate, taking into account the selected error
+        * correction mode, but without overrunning the far endpoint's
+        * datagram buffer. this is complicated by the fact that some
+        * far endpoints send us bogus (small) max datagram values,
+        * which would result in either buffer overrun or no error
+        * correction. we try to accomodate those, but if the supplied
+        * value is too small to do so, we'll emit warning messages and
+        * the user will have to use configuration options to override
+        * the max datagram value supplied by the far endpoint.
+        */
+       switch (udptl->error_correction_scheme) {
+       case UDPTL_ERROR_CORRECTION_NONE:
+               /* need room for sequence number, length indicator, redundancy
+                * indicator and following length indicator
+                */
+               new_max = udptl->far_max_datagram - 5;
+               break;
+       case UDPTL_ERROR_CORRECTION_REDUNDANCY:
+               /* for this case, we'd like to send as many error correction entries
+                * as possible (up to the number we're configured for), but we'll settle
+                * for sending fewer if the configured number would cause the
+                * calculated max IFP to be too small for effective operation
+                *
+                * need room for sequence number, length indicators and the
+                * configured number of redundant packets
+                *
+                * note: we purposely don't allow error_correction_entries to drop to
+                * zero in this loop; we'd rather send smaller IFPs (and thus reduce
+                * the image data transfer rate) than sacrifice redundancy completely
+                */
+               for (;;) {
+                       new_max = (udptl->far_max_datagram - 8) / (udptl->error_correction_entries + 1);
+
+                       if ((new_max < 80) && (udptl->error_correction_entries > 1)) {
+                               /* the max ifp is not large enough, subtract an
+                                * error correction entry and calculate again
+                                * */
+                               --udptl->error_correction_entries;
+                       } else {
+                               break;
+                       }
+               }
+               break;
+       case UDPTL_ERROR_CORRECTION_FEC:
+               /* need room for sequence number, length indicators and a
+                * a single IFP of the maximum size expected
+                */
+               new_max = (udptl->far_max_datagram - 10) / 2;
+               break;
+       }
+       /* subtract 5% of space for insurance */
+       udptl->far_max_ifp = new_max * 0.95;
 }
 
-void ast_udptl_set_error_correction_scheme(struct ast_udptl* udptl, int ec)
+enum ast_t38_ec_modes ast_udptl_get_error_correction_scheme(const struct ast_udptl *udptl)
 {
-       if (udptl) {
-               switch (ec) {
-               case UDPTL_ERROR_CORRECTION_FEC:
-                       udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_FEC;
-                       break;
-               case UDPTL_ERROR_CORRECTION_REDUNDANCY:
-                       udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_REDUNDANCY;
-                       break;
-               case UDPTL_ERROR_CORRECTION_NONE:
-                       udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_NONE;
-                       break;
-               default:
-                       ast_log(LOG_WARNING, "error correction parameter invalid\n");
-               };
-       } else
-               ast_log(LOG_WARNING, "udptl structure is null\n");
+       return udptl->error_correction_scheme;
 }
 
-int ast_udptl_get_local_max_datagram(struct ast_udptl* udptl)
+void ast_udptl_set_error_correction_scheme(struct ast_udptl *udptl, enum ast_t38_ec_modes ec)
 {
-       if (udptl)
-               return udptl->local_max_datagram_size;
-       else {
-               ast_log(LOG_WARNING, "udptl structure is null\n");
-               return -1;
+       udptl->error_correction_scheme = ec;
+       switch (ec) {
+       case UDPTL_ERROR_CORRECTION_FEC:
+               udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_FEC;
+               if (udptl->error_correction_entries == 0) {
+                       udptl->error_correction_entries = 3;
+               }
+               if (udptl->error_correction_span == 0) {
+                       udptl->error_correction_span = 3;
+               }
+               break;
+       case UDPTL_ERROR_CORRECTION_REDUNDANCY:
+               udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_REDUNDANCY;
+               if (udptl->error_correction_entries == 0) {
+                       udptl->error_correction_entries = 3;
+               }
+               break;
+       default:
+               /* nothing to do */
+               break;
+       };
+       /* reset calculated values so they'll be computed again */
+       udptl->local_max_datagram = -1;
+       udptl->far_max_ifp = -1;
+}
+
+void ast_udptl_set_local_max_ifp(struct ast_udptl *udptl, unsigned int max_ifp)
+{
+       /* make sure max_ifp is a positive value since a cast will take place when
+        * when setting local_max_ifp */
+       if ((signed int) max_ifp > 0) {
+               udptl->local_max_ifp = max_ifp;
+               /* reset calculated values so they'll be computed again */
+               udptl->local_max_datagram = -1;
        }
 }
 
-int ast_udptl_get_far_max_datagram(struct ast_udptl* udptl)
+unsigned int ast_udptl_get_local_max_datagram(struct ast_udptl *udptl)
 {
-       if (udptl)
-               return udptl->far_max_datagram_size;
-       else {
-               ast_log(LOG_WARNING, "udptl structure is null\n");
-               return -1;
+       if (udptl->local_max_datagram == -1) {
+               calculate_local_max_datagram(udptl);
        }
+
+       /* this function expects a unsigned value in return. */
+       if (udptl->local_max_datagram < 0) {
+               return 0;
+       }
+       return udptl->local_max_datagram;
 }
 
-void ast_udptl_set_local_max_datagram(struct ast_udptl* udptl, int max_datagram)
+void ast_udptl_set_far_max_datagram(struct ast_udptl *udptl, unsigned int max_datagram)
 {
-       if (udptl)
-               udptl->local_max_datagram_size = max_datagram;
-       else
-               ast_log(LOG_WARNING, "udptl structure is null\n");
+       if (!max_datagram || (max_datagram > FAX_MAX_DATAGRAM_LIMIT)) {
+               udptl->far_max_datagram = DEFAULT_FAX_MAX_DATAGRAM;
+       } else {
+               udptl->far_max_datagram = max_datagram;
+       }
+       /* reset calculated values so they'll be computed again */
+       udptl->far_max_ifp = -1;
 }
 
-void ast_udptl_set_far_max_datagram(struct ast_udptl* udptl, int max_datagram)
+unsigned int ast_udptl_get_far_max_datagram(const struct ast_udptl *udptl)
 {
-       if (udptl)
-               udptl->far_max_datagram_size = max_datagram;
-       else
-               ast_log(LOG_WARNING, "udptl structure is null\n");
+       if (udptl->far_max_datagram < 0) {
+               return 0;
+       }
+       return udptl->far_max_datagram;
 }
 
-struct ast_udptl *ast_udptl_new_with_bindaddr(struct sched_context *sched, struct io_context *io, int callbackmode, struct in_addr addr)
+unsigned int ast_udptl_get_far_max_ifp(struct ast_udptl *udptl)
+{
+       if (udptl->far_max_ifp == -1) {
+               calculate_far_max_ifp(udptl);
+       }
+
+       if (udptl->far_max_ifp < 0) {
+               return 0;
+       }
+       return udptl->far_max_ifp;
+}
+
+struct ast_udptl *ast_udptl_new_with_bindaddr(struct ast_sched_context *sched, struct io_context *io, int callbackmode, struct ast_sockaddr *addr)
 {
        struct ast_udptl *udptl;
        int x;
        int startplace;
        int i;
-       long int flags;
+       RAII_VAR(struct udptl_config *, cfg, ao2_global_obj_ref(globals), ao2_cleanup);
 
-       if (!(udptl = ast_calloc(1, sizeof(*udptl))))
+       if (!cfg || !cfg->general) {
+               ast_log(LOG_ERROR, "Could not access global udptl options!\n");
                return NULL;
+       }
+
+       if (!(udptl = ast_calloc(1, sizeof(*udptl)))) {
+               return NULL;
+       }
+
+       udptl->error_correction_span = cfg->general->fecspan;
+       udptl->error_correction_entries = cfg->general->fecentries;
+
+       udptl->far_max_datagram = -1;
+       udptl->far_max_ifp = -1;
+       udptl->local_max_ifp = -1;
+       udptl->local_max_datagram = -1;
 
-       if (udptlfectype == 2)
-               udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_FEC;
-       else if (udptlfectype == 1)
-               udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_REDUNDANCY;
-       else
-               udptl->error_correction_scheme = UDPTL_ERROR_CORRECTION_NONE;
-       udptl->error_correction_span = udptlfecspan;
-       udptl->error_correction_entries = udptlfecentries;
-       
-       udptl->far_max_datagram_size = udptlmaxdatagram;
-       udptl->local_max_datagram_size = udptlmaxdatagram;
-
-       memset(&udptl->rx, 0, sizeof(udptl->rx));
-       memset(&udptl->tx, 0, sizeof(udptl->tx));
        for (i = 0; i <= UDPTL_BUF_MASK; i++) {
                udptl->rx[i].buf_len = -1;
                udptl->tx[i].buf_len = -1;
        }
 
-       udptl->seqno = ast_random() & 0xffff;
-       udptl->them.sin_family = AF_INET;
-       udptl->us.sin_family = AF_INET;
-
-       if ((udptl->fd = socket(AF_INET, SOCK_DGRAM, 0)) < 0) {
-               free(udptl);
+       if ((udptl->fd = socket(ast_sockaddr_is_ipv6(addr) ?
+                                       AF_INET6 : AF_INET, SOCK_DGRAM, 0)) < 0) {
+               ast_free(udptl);
                ast_log(LOG_WARNING, "Unable to allocate socket: %s\n", strerror(errno));
                return NULL;
        }
-       flags = fcntl(udptl->fd, F_GETFL);
-       fcntl(udptl->fd, F_SETFL, flags | O_NONBLOCK);
+       ast_fd_set_flags(udptl->fd, O_NONBLOCK);
+
 #ifdef SO_NO_CHECK
-       if (nochecksums)
-               setsockopt(udptl->fd, SOL_SOCKET, SO_NO_CHECK, &nochecksums, sizeof(nochecksums));
+       if (cfg->general->nochecksums)
+               setsockopt(udptl->fd, SOL_SOCKET, SO_NO_CHECK, &cfg->general->nochecksums, sizeof(cfg->general->nochecksums));
 #endif
+
        /* Find us a place */
-       x = (ast_random() % (udptlend - udptlstart)) + udptlstart;
+       x = (cfg->general->start == cfg->general->end) ? cfg->general->start : (ast_random() % (cfg->general->end - cfg->general->start)) + cfg->general->start;
+       if (cfg->general->use_even_ports && (x & 1)) {
+               ++x;
+       }
        startplace = x;
        for (;;) {
-               udptl->us.sin_port = htons(x);
-               udptl->us.sin_addr = addr;
-               if (bind(udptl->fd, (struct sockaddr *) &udptl->us, sizeof(udptl->us)) == 0)
+               ast_sockaddr_copy(&udptl->us, addr);
+               ast_sockaddr_set_port(&udptl->us, x);
+               if (ast_bind(udptl->fd, &udptl->us) == 0) {
                        break;
-               if (errno != EADDRINUSE) {
+               }
+               if (errno != EADDRINUSE && errno != EACCES) {
                        ast_log(LOG_WARNING, "Unexpected bind error: %s\n", strerror(errno));
                        close(udptl->fd);
-                       free(udptl);
+                       ast_free(udptl);
                        return NULL;
                }
-               if (++x > udptlend)
-                       x = udptlstart;
+               if (cfg->general->use_even_ports) {
+                       x += 2;
+               } else {
+                       ++x;
+               }
+               if (x > cfg->general->end)
+                       x = cfg->general->start;
                if (x == startplace) {
                        ast_log(LOG_WARNING, "No UDPTL ports remaining\n");
                        close(udptl->fd);
-                       free(udptl);
+                       ast_free(udptl);
                        return NULL;
                }
        }
@@ -830,47 +1087,46 @@ struct ast_udptl *ast_udptl_new_with_bindaddr(struct sched_context *sched, struc
                udptl->io = io;
                udptl->ioid = ast_io_add(udptl->io, udptl->fd, udptlread, AST_IO_IN, udptl);
        }
+
        return udptl;
 }
 
-struct ast_udptl *ast_udptl_new(struct sched_context *sched, struct io_context *io, int callbackmode)
+void ast_udptl_set_tag(struct ast_udptl *udptl, const char *format, ...)
 {
-       struct in_addr ia;
-       memset(&ia, 0, sizeof(ia));
-       return ast_udptl_new_with_bindaddr(sched, io, callbackmode, ia);
+       va_list ap;
+
+       ast_free(udptl->tag);
+       udptl->tag = NULL;
+       va_start(ap, format);
+       if (ast_vasprintf(&udptl->tag, format, ap) == -1) {
+               udptl->tag = NULL;
+       }
+       va_end(ap);
 }
 
-int ast_udptl_settos(struct ast_udptl *udptl, int tos)
+int ast_udptl_setqos(struct ast_udptl *udptl, unsigned int tos, unsigned int cos)
 {
-       int res;
-
-       if ((res = setsockopt(udptl->fd, IPPROTO_IP, IP_TOS, &tos, sizeof(tos)))) 
-               ast_log(LOG_WARNING, "UDPTL unable to set TOS to %d\n", tos);
-       return res;
+       return ast_set_qos(udptl->fd, tos, cos, "UDPTL");
 }
 
-void ast_udptl_set_peer(struct ast_udptl *udptl, struct sockaddr_in *them)
+void ast_udptl_set_peer(struct ast_udptl *udptl, const struct ast_sockaddr *them)
 {
-       udptl->them.sin_port = them->sin_port;
-       udptl->them.sin_addr = them->sin_addr;
+       ast_sockaddr_copy(&udptl->them, them);
 }
 
-void ast_udptl_get_peer(struct ast_udptl *udptl, struct sockaddr_in *them)
+void ast_udptl_get_peer(const struct ast_udptl *udptl, struct ast_sockaddr *them)
 {
-       them->sin_family = AF_INET;
-       them->sin_port = udptl->them.sin_port;
-       them->sin_addr = udptl->them.sin_addr;
+       ast_sockaddr_copy(them, &udptl->them);
 }
 
-void ast_udptl_get_us(struct ast_udptl *udptl, struct sockaddr_in *us)
+void ast_udptl_get_us(const struct ast_udptl *udptl, struct ast_sockaddr *us)
 {
-       memcpy(us, &udptl->us, sizeof(udptl->us));
+       ast_sockaddr_copy(us, &udptl->us);
 }
 
 void ast_udptl_stop(struct ast_udptl *udptl)
 {
-       memset(&udptl->them.sin_addr, 0, sizeof(udptl->them.sin_addr));
-       memset(&udptl->them.sin_port, 0, sizeof(udptl->them.sin_port));
+       ast_sockaddr_setnull(&udptl->them);
 }
 
 void ast_udptl_destroy(struct ast_udptl *udptl)
@@ -879,380 +1135,277 @@ void ast_udptl_destroy(struct ast_udptl *udptl)
                ast_io_remove(udptl->io, udptl->ioid);
        if (udptl->fd > -1)
                close(udptl->fd);
-       free(udptl);
+       if (udptl->tag)
+               ast_free(udptl->tag);
+       ast_free(udptl);
 }
 
 int ast_udptl_write(struct ast_udptl *s, struct ast_frame *f)
 {
-       int len;
-       int res;
-       uint8_t buf[LOCAL_FAX_MAX_DATAGRAM];
+       unsigned int seq;
+       unsigned int len = f->datalen;
+       /* if no max datagram size is provided, use default value */
+       const int bufsize = (s->far_max_datagram > 0) ? s->far_max_datagram : DEFAULT_FAX_MAX_DATAGRAM;
+       uint8_t buf[bufsize];
 
-       /* If we have no peer, return immediately */    
-       if (s->them.sin_addr.s_addr == INADDR_ANY)
+       memset(buf, 0, sizeof(buf));
+
+       /* If we have no peer, return immediately */
+       if (ast_sockaddr_isnull(&s->them)) {
                return 0;
+       }
 
        /* If there is no data length, return immediately */
        if (f->datalen == 0)
                return 0;
-       
-       if (f->frametype != AST_FRAME_MODEM) {
-               ast_log(LOG_WARNING, "UDPTL can only send T.38 data\n");
+
+       if ((f->frametype != AST_FRAME_MODEM) ||
+           (f->subclass.integer != AST_MODEM_T38)) {
+               ast_log(LOG_WARNING, "UDPTL (%s): UDPTL can only send T.38 data.\n",
+                       LOG_TAG(s));
                return -1;
        }
 
+       if (len > s->far_max_ifp) {
+               ast_log(LOG_WARNING,
+                       "UDPTL (%s): UDPTL asked to send %u bytes of IFP when far end only prepared to accept %d bytes; data loss will occur."
+                       "You may need to override the T38FaxMaxDatagram value for this endpoint in the channel driver configuration.\n",
+                       LOG_TAG(s), len, s->far_max_ifp);
+               len = s->far_max_ifp;
+       }
+
+       /* Save seq_no for debug output because udptl_build_packet increments it */
+       seq = s->tx_seq_no & 0xFFFF;
+
        /* Cook up the UDPTL packet, with the relevant EC info. */
-       len = udptl_build_packet(s, buf, f->data, f->datalen);
+       len = udptl_build_packet(s, buf, sizeof(buf), f->data.ptr, len);
 
-       if (len > 0 && s->them.sin_port && s->them.sin_addr.s_addr) {
-               if ((res = sendto(s->fd, buf, len, 0, (struct sockaddr *) &s->them, sizeof(s->them))) < 0)
-                       ast_log(LOG_NOTICE, "UDPTL Transmission error to %s:%d: %s\n", ast_inet_ntoa(s->them.sin_addr), ntohs(s->them.sin_port), strerror(errno));
-#if 0
-               printf("Sent %d bytes of UDPTL data to %s:%d\n", res, ast_inet_ntoa(udptl->them.sin_addr), ntohs(udptl->them.sin_port));
-#endif
-               if (udptl_debug_test_addr(&s->them))
-                       ast_verbose("Sent UDPTL packet to %s:%d (type %d, seq %d, len %d)\n",
-                                       ast_inet_ntoa(s->them.sin_addr),
-                                       ntohs(s->them.sin_port), 0, s->seqno, len);
+       if ((signed int) len > 0 && !ast_sockaddr_isnull(&s->them)) {
+               if (ast_sendto(s->fd, buf, len, 0, &s->them) < 0) {
+                       ast_log(LOG_NOTICE, "UDPTL (%s): Transmission error to %s: %s\n",
+                               LOG_TAG(s), ast_sockaddr_stringify(&s->them), strerror(errno));
+               }
+               if (udptl_debug_test_addr(&s->them)) {
+                       ast_verb(1, "UDPTL (%s): packet to %s (seq %u, len %u)\n",
+                               LOG_TAG(s), ast_sockaddr_stringify(&s->them), seq, len);
+               }
        }
-               
+
        return 0;
 }
 
-void ast_udptl_proto_unregister(struct ast_udptl_protocol *proto)
+static char *handle_cli_udptl_set_debug(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
 {
-       struct ast_udptl_protocol *cur;
-       struct ast_udptl_protocol *prev;
-
-       cur = protos;
-       prev = NULL;
-       while (cur) {
-               if (cur == proto) {
-                       if (prev)
-                               prev->next = proto->next;
-                       else
-                               protos = proto->next;
-                       return;
+       switch (cmd) {
+       case CLI_INIT:
+               e->command = "udptl set debug {on|off|ip}";
+               e->usage =
+                       "Usage: udptl set debug {on|off|ip host[:port]}\n"
+                       "       Enable or disable dumping of UDPTL packets.\n"
+                       "       If ip is specified, limit the dumped packets to those to and from\n"
+                       "       the specified 'host' with optional port.\n";
+               return NULL;
+       case CLI_GENERATE:
+               return NULL;
+       }
+
+       if (a->argc < 4 || a->argc > 5)
+               return CLI_SHOWUSAGE;
+
+       if (a->argc == 4) {
+               if (!strncasecmp(a->argv[3], "on", 2)) {
+                       udptldebug = 1;
+                       memset(&udptldebugaddr, 0, sizeof(udptldebugaddr));
+                       ast_cli(a->fd, "UDPTL Debugging Enabled\n");
+               } else if (!strncasecmp(a->argv[3], "off", 3)) {
+                       udptldebug = 0;
+                       ast_cli(a->fd, "UDPTL Debugging Disabled\n");
+               } else {
+                       return CLI_SHOWUSAGE;
+               }
+       } else {
+               struct ast_sockaddr *addrs;
+               if (strncasecmp(a->argv[3], "ip", 2))
+                       return CLI_SHOWUSAGE;
+               if (!ast_sockaddr_resolve(&addrs, a->argv[4], 0, 0)) {
+                       return CLI_SHOWUSAGE;
                }
-               prev = cur;
-               cur = cur->next;
+               ast_sockaddr_copy(&udptldebugaddr, &addrs[0]);
+                       ast_cli(a->fd, "UDPTL Debugging Enabled for IP: %s\n", ast_sockaddr_stringify(&udptldebugaddr));
+               udptldebug = 1;
+               ast_free(addrs);
        }
+
+       return CLI_SUCCESS;
 }
 
-int ast_udptl_proto_register(struct ast_udptl_protocol *proto)
+static char *handle_cli_show_config(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
 {
-       struct ast_udptl_protocol *cur;
+       RAII_VAR(struct udptl_config *, cfg, NULL, ao2_cleanup);
+
+       switch (cmd) {
+       case CLI_INIT:
+               e->command = "udptl show config";
+               e->usage =
+                       "Usage: udptl show config\n"
+                       "       Display UDPTL configuration options\n";
+               return NULL;
+       case CLI_GENERATE:
+               return NULL;
+       }
 
-       cur = protos;
-       while (cur) {
-               if (cur->type == proto->type) {
-                       ast_log(LOG_WARNING, "Tried to register same protocol '%s' twice\n", cur->type);
-                       return -1;
-               }
-               cur = cur->next;
+       if (!(cfg = ao2_global_obj_ref(globals))) {
+               return CLI_FAILURE;
        }
-       proto->next = protos;
-       protos = proto;
-       return 0;
+
+       ast_cli(a->fd, "UDPTL Global options\n");
+       ast_cli(a->fd, "--------------------\n");
+       ast_cli(a->fd, "udptlstart:      %u\n", cfg->general->start);
+       ast_cli(a->fd, "udptlend:        %u\n", cfg->general->end);
+       ast_cli(a->fd, "udptlfecentries: %u\n", cfg->general->fecentries);
+       ast_cli(a->fd, "udptlfecspan:    %u\n", cfg->general->fecspan);
+       ast_cli(a->fd, "use_even_ports:  %s\n", AST_CLI_YESNO(cfg->general->use_even_ports));
+       ast_cli(a->fd, "udptlchecksums: %s\n", AST_CLI_YESNO(!cfg->general->nochecksums));
+
+       return CLI_SUCCESS;
+}
+
+static struct ast_cli_entry cli_udptl[] = {
+       AST_CLI_DEFINE(handle_cli_udptl_set_debug, "Enable/Disable UDPTL debugging"),
+       AST_CLI_DEFINE(handle_cli_show_config, "Show UDPTL config options"),
+};
+
+static void udptl_config_destructor(void *obj)
+{
+       struct udptl_config *cfg = obj;
+       ao2_cleanup(cfg->general);
 }
 
-static struct ast_udptl_protocol *get_proto(struct ast_channel *chan)
+static void *udptl_snapshot_alloc(void)
 {
-       struct ast_udptl_protocol *cur;
+       struct udptl_config *cfg;
 
-       cur = protos;
-       while (cur) {
-               if (cur->type == chan->tech->type)
-                       return cur;
-               cur = cur->next;
+       if (!(cfg = ao2_alloc(sizeof(*cfg), udptl_config_destructor))) {
+               return NULL;
        }
-       return NULL;
+       if (!(cfg->general = ao2_alloc(sizeof(*cfg->general), NULL))) {
+               ao2_ref(cfg, -1);
+               return NULL;
+       }
+
+       return cfg;
 }
 
-int ast_udptl_bridge(struct ast_channel *c0, struct ast_channel *c1, int flags, struct ast_frame **fo, struct ast_channel **rc)
+static int removed_options_handler(const struct aco_option *opt, struct ast_variable *var, void *obj)
 {
-       struct ast_frame *f;
-       struct ast_channel *who;
-       struct ast_channel *cs[3];
-       struct ast_udptl *p0;
-       struct ast_udptl *p1;
-       struct ast_udptl_protocol *pr0;
-       struct ast_udptl_protocol *pr1;
-       struct sockaddr_in ac0;
-       struct sockaddr_in ac1;
-       struct sockaddr_in t0;
-       struct sockaddr_in t1;
-       void *pvt0;
-       void *pvt1;
-       int to;
-       
-       ast_channel_lock(c0);
-       while (ast_channel_trylock(c1)) {
-               ast_channel_unlock(c0);
-               usleep(1);
-               ast_channel_lock(c0);
+       if (!strcasecmp(var->name, "t38faxudpec")) {
+               ast_log(LOG_WARNING, "t38faxudpec in udptl.conf is no longer supported; use the t38pt_udptl configuration option in sip.conf instead.\n");
+       } else if (!strcasecmp(var->name, "t38faxmaxdatagram")) {
+               ast_log(LOG_WARNING, "t38faxmaxdatagram in udptl.conf is no longer supported; value is now supplied by T.38 applications.\n");
        }
-       pr0 = get_proto(c0);
-       pr1 = get_proto(c1);
-       if (!pr0) {
-               ast_log(LOG_WARNING, "Can't find native functions for channel '%s'\n", c0->name);
-               ast_channel_unlock(c0);
-               ast_channel_unlock(c1);
-               return -1;
+       return 0;
+}
+
+static void __ast_udptl_reload(int reload)
+{
+       if (aco_process_config(&cfg_info, reload) == ACO_PROCESS_ERROR) {
+               if (!reload) {
+                       RAII_VAR(struct udptl_config *, udptl_cfg, udptl_snapshot_alloc(), ao2_cleanup);
+
+                       if (aco_set_defaults(&general_option, "general", udptl_cfg->general)) {
+                               ast_log(LOG_ERROR, "Failed to load udptl.conf and failed to initialize defaults.\n");
+                               return;
+                       }
+
+                       ast_log(LOG_NOTICE, "Could not load udptl config; using defaults\n");
+                       ao2_global_obj_replace_unref(globals, udptl_cfg);
+               }
        }
-       if (!pr1) {
-               ast_log(LOG_WARNING, "Can't find native functions for channel '%s'\n", c1->name);
-               ast_channel_unlock(c0);
-               ast_channel_unlock(c1);
+}
+
+static int udptl_pre_apply_config(void) {
+       struct udptl_config *cfg = aco_pending_config(&cfg_info);
+
+       if (!cfg->general) {
                return -1;
        }
-       pvt0 = c0->tech_pvt;
-       pvt1 = c1->tech_pvt;
-       p0 = pr0->get_udptl_info(c0);
-       p1 = pr1->get_udptl_info(c1);
-       if (!p0 || !p1) {
-               /* Somebody doesn't want to play... */
-               ast_channel_unlock(c0);
-               ast_channel_unlock(c1);
-               return -2;
+
+#ifndef SO_NO_CHECK
+       if (cfg->general->nochecksums) {
+               ast_log(LOG_WARNING, "Disabling UDPTL checksums is not supported on this operating system!\n");
+               cfg->general->nochecksums = 0;
        }
-       if (pr0->set_udptl_peer(c0, p1)) {
-               ast_log(LOG_WARNING, "Channel '%s' failed to talk to '%s'\n", c0->name, c1->name);
-       } else {
-               /* Store UDPTL peer */
-               ast_udptl_get_peer(p1, &ac1);
+#endif
+
+       /* Fix up any global config values that we can handle before replacing the config */
+       if (cfg->general->use_even_ports && (cfg->general->start & 1)) {
+               ++cfg->general->start;
+               ast_log(LOG_NOTICE, "Odd numbered udptlstart specified but use_even_ports enabled. udptlstart is now %u\n", cfg->general->start);
        }
-       if (pr1->set_udptl_peer(c1, p0))
-               ast_log(LOG_WARNING, "Channel '%s' failed to talk back to '%s'\n", c1->name, c0->name);
-       else {
-               /* Store UDPTL peer */
-               ast_udptl_get_peer(p0, &ac0);
+       if (cfg->general->start > cfg->general->end) {
+               ast_log(LOG_WARNING, "Unreasonable values for UDPTL start/end ports; defaulting to %s-%s.\n", __stringify(DEFAULT_UDPTLSTART), __stringify(DEFAULT_UDPTLEND));
+               cfg->general->start = DEFAULT_UDPTLSTART;
+               cfg->general->end = DEFAULT_UDPTLEND;
        }
-       ast_channel_unlock(c0);
-       ast_channel_unlock(c1);
-       cs[0] = c0;
-       cs[1] = c1;
-       cs[2] = NULL;
-       for (;;) {
-               if ((c0->tech_pvt != pvt0) ||
-                       (c1->tech_pvt != pvt1) ||
-                       (c0->masq || c0->masqr || c1->masq || c1->masqr)) {
-                               if (option_debug)
-                                       ast_log(LOG_DEBUG, "Oooh, something is weird, backing out\n");
-                               /* Tell it to try again later */
-                               return -3;
-               }
-               to = -1;
-               ast_udptl_get_peer(p1, &t1);
-               ast_udptl_get_peer(p0, &t0);
-               if (inaddrcmp(&t1, &ac1)) {
-                       if (option_debug) {
-                               ast_log(LOG_DEBUG, "Oooh, '%s' changed end address to %s:%d\n", 
-                                       c1->name, ast_inet_ntoa(t1.sin_addr), ntohs(t1.sin_port));
-                               ast_log(LOG_DEBUG, "Oooh, '%s' was %s:%d\n", 
-                                       c1->name, ast_inet_ntoa(ac1.sin_addr), ntohs(ac1.sin_port));
-                       }
-                       memcpy(&ac1, &t1, sizeof(ac1));
-               }
-               if (inaddrcmp(&t0, &ac0)) {
-                       if (option_debug) {
-                               ast_log(LOG_DEBUG, "Oooh, '%s' changed end address to %s:%d\n", 
-                                       c0->name, ast_inet_ntoa(t0.sin_addr), ntohs(t0.sin_port));
-                               ast_log(LOG_DEBUG, "Oooh, '%s' was %s:%d\n", 
-                                       c0->name, ast_inet_ntoa(ac0.sin_addr), ntohs(ac0.sin_port));
-                       }
-                       memcpy(&ac0, &t0, sizeof(ac0));
-               }
-               who = ast_waitfor_n(cs, 2, &to);
-               if (!who) {
-                       if (option_debug)
-                               ast_log(LOG_DEBUG, "Ooh, empty read...\n");
-                       /* check for hangup / whentohangup */
-                       if (ast_check_hangup(c0) || ast_check_hangup(c1))
-                               break;
-                       continue;
-               }
-               f = ast_read(who);
-               if (!f) {
-                       *fo = f;
-                       *rc = who;
-                       if (option_debug)
-                               ast_log(LOG_DEBUG, "Oooh, got a %s\n", f ? "digit" : "hangup");
-                       /* That's all we needed */
-                       return 0;
-               } else {
-                       if (f->frametype == AST_FRAME_MODEM) {
-                               /* Forward T.38 frames if they happen upon us */
-                               if (who == c0) {
-                                       ast_write(c1, f);
-                               } else if (who == c1) {
-                                       ast_write(c0, f);
-                               }
-                       }
-                       ast_frfree(f);
-               }
-               /* Swap priority. Not that it's a big deal at this point */
-               cs[2] = cs[0];
-               cs[0] = cs[1];
-               cs[1] = cs[2];
+       if (cfg->general->use_even_ports && (cfg->general->end & 1)) {
+               --cfg->general->end;
+               ast_log(LOG_NOTICE, "Odd numbered udptlend specified but use_even_ports enabled. udptlend is now %u\n", cfg->general->end);
        }
-       return -1;
+
+       return 0;
 }
 
-static int udptl_do_debug_ip(int fd, int argc, char *argv[])
+int ast_udptl_reload(void)
 {
-       struct hostent *hp;
-       struct ast_hostent ahp;
-       int port;
-       char *p;
-       char *arg;
-
-       port = 0;
-       if (argc != 4)
-               return RESULT_SHOWUSAGE;
-       arg = argv[3];
-       p = strstr(arg, ":");
-       if (p) {
-               *p = '\0';
-               p++;
-               port = atoi(p);
-       }
-       hp = ast_gethostbyname(arg, &ahp);
-       if (hp == NULL)
-               return RESULT_SHOWUSAGE;
-       udptldebugaddr.sin_family = AF_INET;
-       memcpy(&udptldebugaddr.sin_addr, hp->h_addr, sizeof(udptldebugaddr.sin_addr));
-       udptldebugaddr.sin_port = htons(port);
-       if (port == 0)
-               ast_cli(fd, "UDPTL Debugging Enabled for IP: %s\n", ast_inet_ntoa(udptldebugaddr.sin_addr));
-       else
-               ast_cli(fd, "UDPTL Debugging Enabled for IP: %s:%d\n", ast_inet_ntoa(udptldebugaddr.sin_addr), port);
-       udptldebug = 1;
-       return RESULT_SUCCESS;
+       __ast_udptl_reload(1);
+       return 0;
 }
 
-static int udptl_do_debug(int fd, int argc, char *argv[])
+/*!
+ * \internal
+ * \brief Clean up resources on Asterisk shutdown
+ */
+static void udptl_shutdown(void)
 {
-       if (argc != 2) {
-               if (argc != 4)
-                       return RESULT_SHOWUSAGE;
-               return udptl_do_debug_ip(fd, argc, argv);
-       }
-       udptldebug = 1;
-       memset(&udptldebugaddr,0,sizeof(udptldebugaddr));
-       ast_cli(fd, "UDPTL Debugging Enabled\n");
-       return RESULT_SUCCESS;
+       ast_cli_unregister_multiple(cli_udptl, ARRAY_LEN(cli_udptl));
+       ao2_t_global_obj_release(globals, "Unref udptl global container in shutdown");
+       aco_info_destroy(&cfg_info);
 }
 
-static int udptl_nodebug(int fd, int argc, char *argv[])
+void ast_udptl_init(void)
 {
-       if (argc != 3)
-               return RESULT_SHOWUSAGE;
-       udptldebug = 0;
-       ast_cli(fd,"UDPTL Debugging Disabled\n");
-       return RESULT_SUCCESS;
-}
+       if (aco_info_init(&cfg_info)) {
+               return;
+       }
 
-static const char debug_usage[] =
-  "Usage: udptl debug [ip host[:port]]\n"
-  "       Enable dumping of all UDPTL packets to and from host.\n";
+       aco_option_register(&cfg_info, "udptlstart", ACO_EXACT, general_options, __stringify(DEFAULT_UDPTLSTART),
+               OPT_UINT_T, PARSE_IN_RANGE | PARSE_DEFAULT,
+               FLDSET(struct udptl_global_options, start), DEFAULT_UDPTLSTART, 1024, 65535);
 
-static const char nodebug_usage[] =
-  "Usage: udptl debug off\n"
-  "       Disable all UDPTL debugging\n";
+       aco_option_register(&cfg_info, "udptlend", ACO_EXACT, general_options, __stringify(DEFAULT_UDPTLEND),
+               OPT_UINT_T, PARSE_IN_RANGE | PARSE_DEFAULT,
+               FLDSET(struct udptl_global_options, end), DEFAULT_UDPTLEND, 1024, 65535);
 
-static struct ast_cli_entry cli_udptl[] = {
-       { { "udptl", "debug", NULL },
-       udptl_do_debug, "Enable UDPTL debugging",
-       debug_usage },
+       aco_option_register(&cfg_info, "udptlfecentries", ACO_EXACT, general_options, NULL,
+               OPT_UINT_T, PARSE_IN_RANGE | PARSE_RANGE_DEFAULTS,
+               FLDSET(struct udptl_global_options, fecentries), 1, MAX_FEC_ENTRIES);
 
-       { { "udptl", "debug", "ip", NULL },
-       udptl_do_debug, "Enable UDPTL debugging on IP",
-       debug_usage },
+       aco_option_register(&cfg_info, "udptlfecspan", ACO_EXACT, general_options, NULL,
+               OPT_UINT_T, PARSE_IN_RANGE | PARSE_RANGE_DEFAULTS,
+               FLDSET(struct udptl_global_options, fecspan), 1, MAX_FEC_SPAN);
 
-       { { "udptl", "debug", "off", NULL },
-       udptl_nodebug, "Disable UDPTL debugging",
-       nodebug_usage },
-};
+       aco_option_register(&cfg_info, "udptlchecksums", ACO_EXACT, general_options, "yes",
+               OPT_BOOL_T, 0, FLDSET(struct udptl_global_options, nochecksums));
 
-void ast_udptl_reload(void)
-{
-       struct ast_config *cfg;
-       const char *s;
-
-       udptlstart = 4500;
-       udptlend = 4999;
-       udptlfectype = 0;
-       udptlfecentries = 0;
-       udptlfecspan = 0;
-       udptlmaxdatagram = 0;
-
-       if ((cfg = ast_config_load("udptl.conf"))) {
-               if ((s = ast_variable_retrieve(cfg, "general", "udptlstart"))) {
-                       udptlstart = atoi(s);
-                       if (udptlstart < 1024)
-                               udptlstart = 1024;
-                       if (udptlstart > 65535)
-                               udptlstart = 65535;
-               }
-               if ((s = ast_variable_retrieve(cfg, "general", "udptlend"))) {
-                       udptlend = atoi(s);
-                       if (udptlend < 1024)
-                               udptlend = 1024;
-                       if (udptlend > 65535)
-                               udptlend = 65535;
-               }
-               if ((s = ast_variable_retrieve(cfg, "general", "udptlchecksums"))) {
-#ifdef SO_NO_CHECK
-                       if (ast_false(s))
-                               nochecksums = 1;
-                       else
-                               nochecksums = 0;
-#else
-                       if (ast_false(s))
-                               ast_log(LOG_WARNING, "Disabling UDPTL checksums is not supported on this operating system!\n");
-#endif
-               }
-               if ((s = ast_variable_retrieve(cfg, "general", "T38FaxUdpEC"))) {
-                       if (strcmp(s, "t38UDPFEC") == 0)
-                               udptlfectype = 2;
-                       else if (strcmp(s, "t38UDPRedundancy") == 0)
-                               udptlfectype = 1;
-               }
-               if ((s = ast_variable_retrieve(cfg, "general", "T38FaxMaxDatagram"))) {
-                       udptlmaxdatagram = atoi(s);
-                       if (udptlmaxdatagram < 0)
-                               udptlmaxdatagram = 0;
-                       if (udptlmaxdatagram > LOCAL_FAX_MAX_DATAGRAM)
-                               udptlmaxdatagram = LOCAL_FAX_MAX_DATAGRAM;
-               }
-               if ((s = ast_variable_retrieve(cfg, "general", "UDPTLFECentries"))) {
-                       udptlfecentries = atoi(s);
-                       if (udptlfecentries < 0)
-                               udptlfecentries = 0;
-                       if (udptlfecentries > MAX_FEC_ENTRIES)
-                               udptlfecentries = MAX_FEC_ENTRIES;
-               }
-               if ((s = ast_variable_retrieve(cfg, "general", "UDPTLFECspan"))) {
-                       udptlfecspan = atoi(s);
-                       if (udptlfecspan < 0)
-                               udptlfecspan = 0;
-                       if (udptlfecspan > MAX_FEC_SPAN)
-                               udptlfecspan = MAX_FEC_SPAN;
-               }
-               ast_config_destroy(cfg);
-       }
-       if (udptlstart >= udptlend) {
-               ast_log(LOG_WARNING, "Unreasonable values for UDPTL start/end\n");
-               udptlstart = 4500;
-               udptlend = 4999;
-       }
-       if (option_verbose > 1)
-               ast_verbose(VERBOSE_PREFIX_2 "UDPTL allocating from port range %d -> %d\n", udptlstart, udptlend);
-}
+       aco_option_register(&cfg_info, "use_even_ports", ACO_EXACT, general_options, "no",
+               OPT_BOOL_T, 1, FLDSET(struct udptl_global_options, use_even_ports));
 
-void ast_udptl_init(void)
-{
-       ast_cli_register_multiple(cli_udptl, sizeof(cli_udptl) / sizeof(struct ast_cli_entry));
-       ast_udptl_reload();
+       aco_option_register_custom(&cfg_info, "t38faxudpec", ACO_EXACT, general_options, NULL, removed_options_handler, 0);
+       aco_option_register_custom(&cfg_info, "t38faxmaxdatagram", ACO_EXACT, general_options, NULL, removed_options_handler, 0);
+
+       __ast_udptl_reload(0);
+
+       ast_cli_register_multiple(cli_udptl, ARRAY_LEN(cli_udptl));
+
+       ast_register_cleanup(udptl_shutdown);
 }