<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7207">View Change</a></p><div style="white-space:pre-wrap">Approvals:
Joshua Colp: Looks good to me, but someone else must approve; Approved for Submit
George Joseph: Looks good to me, approved
</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bundled_pjproject: Update to 2.7.1<br><br>Update from 2.7 to 2.7.1 for bundled pjproject. Changed version<br>and removed patch files included in the update.<br><br>Change-Id: I55cea8e734b318c2df9daf86aa0802c559ec8357<br>---<br>D third-party/pjproject/patches/0010-r5665-svn-backport-ICE-Use-STUN-FINGERPRINT-attribut.patch<br>D third-party/pjproject/patches/0020-sip_parser-Add-validity-checking-for-numeric-header-.patch<br>M third-party/versions.mak<br>3 files changed, 1 insertion(+), 1,015 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/third-party/pjproject/patches/0010-r5665-svn-backport-ICE-Use-STUN-FINGERPRINT-attribut.patch b/third-party/pjproject/patches/0010-r5665-svn-backport-ICE-Use-STUN-FINGERPRINT-attribut.patch<br>deleted file mode 100644<br>index 3c83cc5..0000000<br>--- a/third-party/pjproject/patches/0010-r5665-svn-backport-ICE-Use-STUN-FINGERPRINT-attribut.patch<br>+++ /dev/null<br>@@ -1,41 +0,0 @@<br>-From 06bc834969a34d2c60e05beed3051b4a6d34c125 Mon Sep 17 00:00:00 2001<br>-From: Richard Mudgett <rmudgett@digium.com><br>-Date: Thu, 28 Sep 2017 10:57:23 -0500<br>-Subject: [PATCH 3/3] r5665 svn backport ICE: Use STUN FINGERPRINT attribute<br>- when sending keepalives<br>-<br>-Fixed #2046: ICE: Use STUN FINGERPRINT attribute when sending keepalives<br>-<br>-Per RFC 5245 Section 10:<br>-<br>- If STUN is being used for keepalives, a STUN Binding Indication is<br>- used [RFC5389]. The Indication MUST NOT utilize any authentication<br>- mechanism. It SHOULD contain the FINGERPRINT attribute to aid in<br>- demultiplexing, but SHOULD NOT contain any other attributes.<br>----<br>- pjnath/src/pjnath/ice_session.c | 8 +++++---<br>- 1 file changed, 5 insertions(+), 3 deletions(-)<br>-<br>-diff --git a/pjnath/src/pjnath/ice_session.c b/pjnath/src/pjnath/ice_session.c<br>-index 27a2950..63a0d1c 100644<br>---- a/pjnath/src/pjnath/ice_session.c<br>-+++ b/pjnath/src/pjnath/ice_session.c<br>-@@ -1217,10 +1217,12 @@ static void ice_keep_alive(pj_ice_sess *ice, pj_bool_t send_now)<br>- msg_data = PJ_POOL_ZALLOC_T(tdata->pool, pj_ice_msg_data);<br>- msg_data->transport_id = the_check->lcand->transport_id;<br>- <br>-- /* Temporarily disable FINGERPRINT. The Binding Indication <br>-- * SHOULD NOT contain any attributes.<br>-+ /* RFC 5245 Section 10:<br>-+ * The Binding Indication SHOULD contain the FINGERPRINT attribute<br>-+ * to aid in demultiplexing, but SHOULD NOT contain any other<br>-+ * attributes.<br>- */<br>-- saved = pj_stun_session_use_fingerprint(comp->stun_sess, PJ_FALSE);<br>-+ saved = pj_stun_session_use_fingerprint(comp->stun_sess, PJ_TRUE);<br>- <br>- /* Send to session */<br>- addr_len = pj_sockaddr_get_len(&the_check->rcand->addr);<br>--- <br>-2.7.4<br>-<br>diff --git a/third-party/pjproject/patches/0020-sip_parser-Add-validity-checking-for-numeric-header-.patch b/third-party/pjproject/patches/0020-sip_parser-Add-validity-checking-for-numeric-header-.patch<br>deleted file mode 100644<br>index dfee4b2..0000000<br>--- a/third-party/pjproject/patches/0020-sip_parser-Add-validity-checking-for-numeric-header-.patch<br>+++ /dev/null<br>@@ -1,973 +0,0 @@<br>-From b21042956dc4d3526052d5030953e5c565bb0895 Mon Sep 17 00:00:00 2001<br>-From: George Joseph <gjoseph@digium.com><br>-Date: Thu, 2 Nov 2017 08:23:00 -0600<br>-Subject: [PATCH] sip_parser: Add validity checking for numeric header values<br>-<br>-Parsing the numeric header fields like cseq, ttl, port, etc. all<br>-had the potential to overflow, either causing unintended values to<br>-be captured or, if the values were subsequently converted back to<br>-strings, a buffer overrun. To address this, new "strto" functions<br>-have been created that do range checking and those functions are<br>-used wherever possible in the parser.<br>-<br>- * Created pjlib/include/limits.h and pjlib/include/compat/limits.h<br>- to either include the system limits.h or define common numeric<br>- limits if there is no system limits.h.<br>-<br>- * Created strto*_validate functions in sip_parser that take bounds<br>- and on failure call the on_str_parse_error function which prints<br>- an error message and calls PJ_THROW.<br>-<br>- * Updated sip_parser to validate the numeric fields.<br>-<br>- * Fixed an issue in sip_transport that prevented error messages<br>- from being properly displayed.<br>-<br>- * Added "volatile" to some variables referenced in PJ_CATCH blocks<br>- as the optimizer was sometimes optimizing them away.<br>-<br>- * Fixed length calculation in sip_transaction/create_tsx_key_2543<br>- to account for signed ints being 11 characters, not 9.<br>-<br>-Reported by: Youngsung Kim at LINE Corporation<br>----<br>- pjlib/build/pjlib.vcproj | 14 +++-<br>- pjlib/build/pjlib.vcxproj | 4 +-<br>- pjlib/build/pjlib.vcxproj.filters | 6 ++<br>- pjlib/include/pj/compat/limits.h | 65 +++++++++++++++<br>- pjlib/include/pj/compat/os_win32.h | 1 +<br>- pjlib/include/pj/limits.h | 51 ++++++++++++<br>- pjlib/include/pj/string.h | 46 +++++++++-<br>- pjlib/include/pj/types.h | 3 -<br>- pjlib/src/pj/string.c | 119 +++++++++++++++++++++++++-<br>- pjlib/src/pj/timer.c | 1 +<br>- pjsip/include/pjsip/sip_parser.h | 25 ++++++<br>- pjsip/src/pjsip/sip_parser.c | 166 +++++++++++++++++++++++++++++--------<br>- pjsip/src/pjsip/sip_transaction.c | 4 +-<br>- pjsip/src/pjsip/sip_transport.c | 7 +-<br>- 14 files changed, 463 insertions(+), 49 deletions(-)<br>- create mode 100644 pjlib/include/pj/compat/limits.h<br>- create mode 100644 pjlib/include/pj/limits.h<br>-<br>-diff --git a/pjlib/build/pjlib.vcproj b/pjlib/build/pjlib.vcproj<br>-index 6a217a0b7..12592ef94 100644<br>---- a/pjlib/build/pjlib.vcproj<br>-+++ b/pjlib/build/pjlib.vcproj<br>-@@ -14967,7 +14967,11 @@<br>- </File><br>- <File<br>- RelativePath="..\include\pj\ip_helper.h"<br>-- ><br>-+ ><br>-+ </File><br>-+ <File<br>-+ RelativePath="..\include\pj\limits.h"<br>-+ ><br>- </File><br>- <File<br>- RelativePath="..\include\pj\list.h"<br>-@@ -15070,8 +15074,12 @@<br>- </File><br>- <File<br>- RelativePath="..\include\pj\compat\high_precision.h"<br>-- ><br>-- </File><br>-+ ><br>-+ </File><br>-+ <File<br>-+ RelativePath="..\include\pj\compat\limits.h"<br>-+ ><br>-+ </File><br>- <File<br>- RelativePath="..\include\pj\compat\m_alpha.h"<br>- ><br>-diff --git a/pjlib/build/pjlib.vcxproj b/pjlib/build/pjlib.vcxproj<br>-index abf09ec44..e41731e3c 100644<br>---- a/pjlib/build/pjlib.vcxproj<br>-+++ b/pjlib/build/pjlib.vcxproj<br>-@@ -494,7 +494,7 @@<br>- <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Release|ARM'">true</ExcludedFromBuild><br>- <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Release|x64'">true</ExcludedFromBuild><br>- </ClCompile><br>-- <ClCompile Include="..\src\pj\file_io_win32.c" /> <br>-+ <ClCompile Include="..\src\pj\file_io_win32.c" /><br>- <ClCompile Include="..\src\pj\guid.c" /><br>- <ClCompile Include="..\src\pj\guid_simple.c"><br>- <ExcludedFromBuild Condition="'$(Configuration)|$(Platform)'=='Debug-Dynamic|Win32'">true</ExcludedFromBuild><br>-@@ -890,6 +890,7 @@<br>- <ClInclude Include="..\include\pj\compat\ctype.h" /><br>- <ClInclude Include="..\include\pj\compat\errno.h" /><br>- <ClInclude Include="..\include\pj\compat\high_precision.h" /><br>-+ <ClInclude Include="..\include\pj\compat\limits.h" /><br>- <ClInclude Include="..\include\pj\compat\malloc.h" /><br>- <ClInclude Include="..\include\pj\compat\m_alpha.h" /><br>- <ClInclude Include="..\include\pj\compat\m_i386.h" /><br>-@@ -925,6 +926,7 @@<br>- <ClInclude Include="..\include\pj\hash.h" /><br>- <ClInclude Include="..\include\pj\ioqueue.h" /><br>- <ClInclude Include="..\include\pj\ip_helper.h" /><br>-+ <ClInclude Include="..\include\pj\limits.h" /><br>- <ClInclude Include="..\include\pj\list.h" /><br>- <ClInclude Include="..\include\pj\list_i.h" /><br>- <ClInclude Include="..\include\pj\lock.h" /><br>-diff --git a/pjlib/build/pjlib.vcxproj.filters b/pjlib/build/pjlib.vcxproj.filters<br>-index 0b5cbf109..6f343b019 100644<br>---- a/pjlib/build/pjlib.vcxproj.filters<br>-+++ b/pjlib/build/pjlib.vcxproj.filters<br>-@@ -439,5 +439,11 @@<br>- <ClInclude Include="..\include\pj\compat\os_winuwp.h"><br>- <Filter>Header Files\compat</Filter><br>- </ClInclude><br>-+ <ClInclude Include="..\include\pj\limits.h"><br>-+ <Filter>Header Files</Filter><br>-+ </ClInclude><br>-+ <ClInclude Include="..\include\pj\compat\limits.h"><br>-+ <Filter>Header Files\compat</Filter><br>-+ </ClInclude><br>- </ItemGroup><br>- </Project><br>-\ No newline at end of file<br>-diff --git a/pjlib/include/pj/compat/limits.h b/pjlib/include/pj/compat/limits.h<br>-new file mode 100644<br>-index 000000000..fba0625df<br>---- /dev/null<br>-+++ b/pjlib/include/pj/compat/limits.h<br>-@@ -0,0 +1,65 @@<br>-+/* $Id$ */<br>-+/*<br>-+ * Copyright (C) 2017 Teluu Inc. (http://www.teluu.com)<br>-+ * Copyright (C) 2017 George Joseph <gjoseph@digium.com><br>-+ *<br>-+ * This program is free software; you can redistribute it and/or modify<br>-+ * it under the terms of the GNU General Public License as published by<br>-+ * the Free Software Foundation; either version 2 of the License, or<br>-+ * (at your option) any later version.<br>-+ *<br>-+ * This program is distributed in the hope that it will be useful,<br>-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of<br>-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the<br>-+ * GNU General Public License for more details.<br>-+ *<br>-+ * You should have received a copy of the GNU General Public License<br>-+ * along with this program; if not, write to the Free Software<br>-+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA <br>-+ */<br>-+#ifndef __PJ_COMPAT_LIMITS_H__<br>-+#define __PJ_COMPAT_LIMITS_H__<br>-+<br>-+/**<br>-+ * @file limits.h<br>-+ * @brief Provides integer limits normally found in limits.h.<br>-+ */<br>-+<br>-+#if defined(PJ_HAS_LIMITS_H) && PJ_HAS_LIMITS_H != 0<br>-+# include <limits.h><br>-+#else<br>-+<br>-+# ifdef _MSC_VER<br>-+# pragma message("limits.h is not found or not supported. LONG_MIN and "\<br>-+ "LONG_MAX will be defined by the library in "\<br>-+ "pj/compats/limits.h and overridable in config_site.h")<br>-+# else<br>-+# warning "limits.h is not found or not supported. LONG_MIN and LONG_MAX " \<br>-+ "will be defined by the library in pj/compats/limits.h and "\<br>-+ "overridable in config_site.h"<br>-+# endif<br>-+<br>-+/* Minimum and maximum values a `signed long int' can hold. */<br>-+# ifndef LONG_MAX<br>-+# if __WORDSIZE == 64<br>-+# define LONG_MAX 9223372036854775807L<br>-+# else<br>-+# define LONG_MAX 2147483647L<br>-+# endif<br>-+# endif<br>-+<br>-+# ifndef LONG_MIN<br>-+# define LONG_MIN (-LONG_MAX - 1L)<br>-+# endif<br>-+<br>-+/* Maximum value an `unsigned long int' can hold. (Minimum is 0.) */<br>-+# ifndef ULONG_MAX<br>-+# if __WORDSIZE == 64<br>-+# define ULONG_MAX 18446744073709551615UL<br>-+# else <br>-+# define ULONG_MAX 4294967295UL<br>-+# endif<br>-+# endif<br>-+#endif<br>-+<br>-+#endif /* __PJ_COMPAT_LIMITS_H__ */<br>-diff --git a/pjlib/include/pj/compat/os_win32.h b/pjlib/include/pj/compat/os_win32.h<br>-index 4fa8b21ea..9b18e4eb1 100644<br>---- a/pjlib/include/pj/compat/os_win32.h<br>-+++ b/pjlib/include/pj/compat/os_win32.h<br>-@@ -57,6 +57,7 @@<br>- #define PJ_HAS_SYS_TYPES_H 1<br>- #define PJ_HAS_TIME_H 1<br>- #define PJ_HAS_UNISTD_H 0<br>-+#define PJ_HAS_LIMITS_H 1<br>- <br>- #define PJ_HAS_MSWSOCK_H 1<br>- #define PJ_HAS_WINSOCK_H 0<br>-diff --git a/pjlib/include/pj/limits.h b/pjlib/include/pj/limits.h<br>-new file mode 100644<br>-index 000000000..8b00ae52a<br>---- /dev/null<br>-+++ b/pjlib/include/pj/limits.h<br>-@@ -0,0 +1,51 @@<br>-+/* $Id$ */<br>-+/*<br>-+ * Copyright (C) 2017 Teluu Inc. (http://www.teluu.com)<br>-+ * Copyright (C) 2017 George Joseph <gjoseph@digium.com><br>-+ *<br>-+ * This program is free software; you can redistribute it and/or modify<br>-+ * it under the terms of the GNU General Public License as published by<br>-+ * the Free Software Foundation; either version 2 of the License, or<br>-+ * (at your option) any later version.<br>-+ *<br>-+ * This program is distributed in the hope that it will be useful,<br>-+ * but WITHOUT ANY WARRANTY; without even the implied warranty of<br>-+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the<br>-+ * GNU General Public License for more details.<br>-+ *<br>-+ * You should have received a copy of the GNU General Public License<br>-+ * along with this program; if not, write to the Free Software<br>-+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA <br>-+ */<br>-+#ifndef __PJ_LIMITS_H__<br>-+#define __PJ_LIMITS_H__<br>-+<br>-+/**<br>-+ * @file limits.h<br>-+ * @brief Common min and max values<br>-+ */<br>-+<br>-+#include <pj/compat/limits.h><br>-+<br>-+/** Maximum value for signed 32-bit integer. */<br>-+#define PJ_MAXINT32 0x7fffffff<br>-+<br>-+/** Minimum value for signed 32-bit integer. */<br>-+#define PJ_MININT32 0x80000000<br>-+<br>-+/** Maximum value for unsigned 16-bit integer. */<br>-+#define PJ_MAXUINT16 0xffff<br>-+<br>-+/** Maximum value for unsigned char. */<br>-+#define PJ_MAXUINT8 0xff<br>-+<br>-+/** Maximum value for long. */<br>-+#define PJ_MAXLONG LONG_MAX<br>-+<br>-+/** Minimum value for long. */<br>-+#define PJ_MINLONG LONG_MIN<br>-+<br>-+/** Minimum value for unsigned long. */<br>-+#define PJ_MAXULONG ULONG_MAX<br>-+<br>-+#endif /* __PJ_LIMITS_H__ */<br>-diff --git a/pjlib/include/pj/string.h b/pjlib/include/pj/string.h<br>-index 70a1d6c8c..5de236a65 100644<br>---- a/pjlib/include/pj/string.h<br>-+++ b/pjlib/include/pj/string.h<br>-@@ -28,7 +28,6 @@<br>- #include <pj/types.h><br>- #include <pj/compat/string.h><br>- <br>--<br>- PJ_BEGIN_DECL<br>- <br>- /**<br>-@@ -636,6 +635,29 @@ PJ_DECL(char*) pj_create_random_string(char *str, pj_size_t length);<br>- PJ_DECL(long) pj_strtol(const pj_str_t *str);<br>- <br>- /**<br>-+ * Convert string to signed long integer. The conversion will stop as<br>-+ * soon as non-digit character is found or all the characters have<br>-+ * been processed.<br>-+ *<br>-+ * @param str the string.<br>-+ * @param value Pointer to a long to receive the value.<br>-+ *<br>-+ * @return PJ_SUCCESS if successful. Otherwise:<br>-+ * PJ_ETOOSMALL if the value was an impossibly long negative number.<br>-+ * In this case *value will be set to LONG_MIN.<br>-+ * \n<br>-+ * PJ_ETOOBIG if the value was an impossibly long positive number.<br>-+ * In this case, *value will be set to LONG_MAX.<br>-+ * \n<br>-+ * PJ_EINVAL if the input string was NULL, the value pointer was NULL <br>-+ * or the input string could not be parsed at all such as starting with<br>-+ * a character other than a '+', '-' or not in the '0' - '9' range.<br>-+ * In this case, *value will be left untouched.<br>-+ */<br>-+PJ_DECL(pj_status_t) pj_strtol2(const pj_str_t *str, long *value);<br>-+<br>-+<br>-+/**<br>- * Convert string to unsigned integer. The conversion will stop as<br>- * soon as non-digit character is found or all the characters have<br>- * been processed.<br>-@@ -664,6 +686,27 @@ PJ_DECL(unsigned long) pj_strtoul2(const pj_str_t *str, pj_str_t *endptr,<br>- unsigned base);<br>- <br>- /**<br>-+ * Convert string to unsigned long integer. The conversion will stop as<br>-+ * soon as non-digit character is found or all the characters have<br>-+ * been processed.<br>-+ *<br>-+ * @param str The input string.<br>-+ * @param value Pointer to an unsigned long to receive the value.<br>-+ * @param base Number base to use.<br>-+ *<br>-+ * @return PJ_SUCCESS if successful. Otherwise:<br>-+ * PJ_ETOOBIG if the value was an impossibly long positive number.<br>-+ * In this case, *value will be set to ULONG_MAX.<br>-+ * \n<br>-+ * PJ_EINVAL if the input string was NULL, the value pointer was NULL <br>-+ * or the input string could not be parsed at all such as starting <br>-+ * with a character outside the base character range. In this case,<br>-+ * *value will be left untouched.<br>-+ */<br>-+PJ_DECL(pj_status_t) pj_strtoul3(const pj_str_t *str, unsigned long *value,<br>-+ unsigned base);<br>-+<br>-+/**<br>- * Convert string to float.<br>- *<br>- * @param str the string.<br>-@@ -786,7 +829,6 @@ PJ_INLINE(void*) pj_memchr(const void *buf, int c, pj_size_t size)<br>- return (void*)memchr((void*)buf, c, size);<br>- }<br>- <br>--<br>- /**<br>- * @}<br>- */<br>-diff --git a/pjlib/include/pj/types.h b/pjlib/include/pj/types.h<br>-index 0e0e2d9a7..8c9f78238 100644<br>---- a/pjlib/include/pj/types.h<br>-+++ b/pjlib/include/pj/types.h<br>-@@ -280,9 +280,6 @@ typedef int pj_exception_id_t;<br>- /** Utility macro to compute the number of elements in static array. */<br>- #define PJ_ARRAY_SIZE(a) (sizeof(a)/sizeof(a[0]))<br>- <br>--/** Maximum value for signed 32-bit integer. */<br>--#define PJ_MAXINT32 0x7FFFFFFFL<br>--<br>- /**<br>- * Length of object names.<br>- */<br>-diff --git a/pjlib/src/pj/string.c b/pjlib/src/pj/string.c<br>-index 307cfb47e..b95f141be 100644<br>---- a/pjlib/src/pj/string.c<br>-+++ b/pjlib/src/pj/string.c<br>-@@ -23,11 +23,14 @@<br>- #include <pj/ctype.h><br>- #include <pj/rand.h><br>- #include <pj/os.h><br>-+#include <pj/errno.h><br>-+#include <pj/limits.h><br>- <br>- #if PJ_FUNCTIONS_ARE_INLINED==0<br>- # include <pj/string_i.h><br>- #endif<br>- <br>-+<br>- PJ_DEF(pj_ssize_t) pj_strspn(const pj_str_t *str, const pj_str_t *set_char)<br>- {<br>- pj_ssize_t i, j, count = 0;<br>-@@ -230,6 +233,55 @@ PJ_DEF(long) pj_strtol(const pj_str_t *str)<br>- return pj_strtoul(str);<br>- }<br>- <br>-+<br>-+PJ_DEF(pj_status_t) pj_strtol2(const pj_str_t *str, long *value)<br>-+{<br>-+ pj_str_t s;<br>-+ unsigned long retval = 0;<br>-+ pj_bool_t is_negative = PJ_FALSE;<br>-+ int rc = 0;<br>-+<br>-+ PJ_CHECK_STACK();<br>-+<br>-+ if (!str || !value) {<br>-+ return PJ_EINVAL;<br>-+ }<br>-+<br>-+ s = *str;<br>-+ pj_strltrim(&s);<br>-+<br>-+ if (s.slen == 0)<br>-+ return PJ_EINVAL;<br>-+<br>-+ if (s.ptr[0] == '+' || s.ptr[0] == '-') {<br>-+ is_negative = (s.ptr[0] == '-');<br>-+ s.ptr += 1;<br>-+ s.slen -= 1;<br>-+ }<br>-+<br>-+ rc = pj_strtoul3(&s, &retval, 10);<br>-+ if (rc == PJ_EINVAL) {<br>-+ return rc;<br>-+ } else if (rc != PJ_SUCCESS) {<br>-+ *value = is_negative ? PJ_MINLONG : PJ_MAXLONG;<br>-+ return is_negative ? PJ_ETOOSMALL : PJ_ETOOBIG;<br>-+ }<br>-+<br>-+ if (retval > PJ_MAXLONG && !is_negative) {<br>-+ *value = PJ_MAXLONG;<br>-+ return PJ_ETOOBIG;<br>-+ }<br>-+<br>-+ if (retval > (PJ_MAXLONG + 1UL) && is_negative) {<br>-+ *value = PJ_MINLONG;<br>-+ return PJ_ETOOSMALL;<br>-+ }<br>-+<br>-+ *value = is_negative ? -(long)retval : retval;<br>-+<br>-+ return PJ_SUCCESS;<br>-+}<br>-+<br>- PJ_DEF(unsigned long) pj_strtoul(const pj_str_t *str)<br>- {<br>- unsigned long value;<br>-@@ -282,6 +334,71 @@ PJ_DEF(unsigned long) pj_strtoul2(const pj_str_t *str, pj_str_t *endptr,<br>- return value;<br>- }<br>- <br>-+PJ_DEF(pj_status_t) pj_strtoul3(const pj_str_t *str, unsigned long *value,<br>-+ unsigned base)<br>-+{<br>-+ pj_str_t s;<br>-+ unsigned i;<br>-+<br>-+ PJ_CHECK_STACK();<br>-+<br>-+ if (!str || !value) {<br>-+ return PJ_EINVAL;<br>-+ }<br>-+<br>-+ s = *str;<br>-+ pj_strltrim(&s);<br>-+<br>-+ if (s.slen == 0 || s.ptr[0] < '0' ||<br>-+ (base <= 10 && (unsigned)s.ptr[0] > ('0' - 1) + base) ||<br>-+ (base == 16 && !pj_isxdigit(s.ptr[0])))<br>-+ {<br>-+ return PJ_EINVAL;<br>-+ }<br>-+<br>-+ *value = 0;<br>-+ if (base <= 10) {<br>-+ for (i=0; i<(unsigned)s.slen; ++i) {<br>-+ unsigned c = s.ptr[i] - '0';<br>-+ if (s.ptr[i] < '0' || (unsigned)s.ptr[i] > ('0' - 1) + base) {<br>-+ break;<br>-+ }<br>-+ if (*value > PJ_MAXULONG / base) {<br>-+ *value = PJ_MAXULONG;<br>-+ return PJ_ETOOBIG;<br>-+ }<br>-+<br>-+ *value *= base;<br>-+ if ((PJ_MAXULONG - *value) < c) {<br>-+ *value = PJ_MAXULONG;<br>-+ return PJ_ETOOBIG;<br>-+ }<br>-+ *value += c;<br>-+ }<br>-+ } else if (base == 16) {<br>-+ for (i=0; i<(unsigned)s.slen; ++i) {<br>-+ unsigned c = pj_hex_digit_to_val(s.ptr[i]);<br>-+ if (!pj_isxdigit(s.ptr[i]))<br>-+ break;<br>-+<br>-+ if (*value > PJ_MAXULONG / base) {<br>-+ *value = PJ_MAXULONG;<br>-+ return PJ_ETOOBIG;<br>-+ }<br>-+ *value *= base;<br>-+ if ((PJ_MAXULONG - *value) < c) {<br>-+ *value = PJ_MAXULONG;<br>-+ return PJ_ETOOBIG;<br>-+ }<br>-+ *value += c;<br>-+ }<br>-+ } else {<br>-+ pj_assert(!"Unsupported base");<br>-+ return PJ_EINVAL;<br>-+ }<br>-+ return PJ_SUCCESS;<br>-+}<br>-+<br>- PJ_DEF(float) pj_strtof(const pj_str_t *str)<br>- {<br>- pj_str_t part;<br>-@@ -356,5 +473,3 @@ PJ_DEF(int) pj_utoa_pad( unsigned long val, char *buf, int min_dig, int pad)<br>- <br>- return len;<br>- }<br>--<br>--<br>-diff --git a/pjlib/src/pj/timer.c b/pjlib/src/pj/timer.c<br>-index 225be4498..399e114a8 100644<br>---- a/pjlib/src/pj/timer.c<br>-+++ b/pjlib/src/pj/timer.c<br>-@@ -36,6 +36,7 @@<br>- #include <pj/lock.h><br>- #include <pj/log.h><br>- #include <pj/rand.h><br>-+#include <pj/limits.h><br>- <br>- #define THIS_FILE "timer.c"<br>- <br>-diff --git a/pjsip/include/pjsip/sip_parser.h b/pjsip/include/pjsip/sip_parser.h<br>-index 0d767f0ad..5691fed3a 100644<br>---- a/pjsip/include/pjsip/sip_parser.h<br>-+++ b/pjsip/include/pjsip/sip_parser.h<br>-@@ -39,6 +39,26 @@ PJ_BEGIN_DECL<br>- */<br>- <br>- /**<br>-+ * Contants for limit checks<br>-+ */<br>-+#define PJSIP_MIN_CONTENT_LENGTH 0<br>-+#define PJSIP_MAX_CONTENT_LENGTH PJ_MAXINT32<br>-+#define PJSIP_MIN_PORT 0<br>-+#define PJSIP_MAX_PORT PJ_MAXUINT16<br>-+#define PJSIP_MIN_TTL 0<br>-+#define PJSIP_MAX_TTL PJ_MAXUINT8<br>-+#define PJSIP_MIN_STATUS_CODE 100<br>-+#define PJSIP_MAX_STATUS_CODE 999<br>-+#define PJSIP_MIN_Q1000 0<br>-+#define PJSIP_MAX_Q1000 PJ_MAXINT32 / 1000<br>-+#define PJSIP_MIN_EXPIRES 0<br>-+#define PJSIP_MAX_EXPIRES PJ_MAXINT32<br>-+#define PJSIP_MIN_CSEQ 0<br>-+#define PJSIP_MAX_CSEQ PJ_MAXINT32<br>-+#define PJSIP_MIN_RETRY_AFTER 0<br>-+#define PJSIP_MAX_RETRY_AFTER PJ_MAXINT32<br>-+<br>-+/**<br>- * URI Parsing options.<br>- */<br>- enum<br>-@@ -64,6 +84,11 @@ enum<br>- extern int PJSIP_SYN_ERR_EXCEPTION;<br>- <br>- /**<br>-+ * Invalid value error exception value.<br>-+ */<br>-+extern int PJSIP_EINVAL_ERR_EXCEPTION;<br>-+<br>-+/**<br>- * This structure is used to get error reporting from parser.<br>- */<br>- typedef struct pjsip_parser_err_report<br>-diff --git a/pjsip/src/pjsip/sip_parser.c b/pjsip/src/pjsip/sip_parser.c<br>-index cf3b879f6..f9a0e65b5 100644<br>---- a/pjsip/src/pjsip/sip_parser.c<br>-+++ b/pjsip/src/pjsip/sip_parser.c<br>-@@ -34,6 +34,7 @@<br>- #include <pj/string.h><br>- #include <pj/ctype.h><br>- #include <pj/assert.h><br>-+#include <pj/limits.h><br>- <br>- #define THIS_FILE "sip_parser.c"<br>- <br>-@@ -93,6 +94,7 @@ static unsigned uri_handler_count;<br>- * Global vars (also extern).<br>- */<br>- int PJSIP_SYN_ERR_EXCEPTION = -1;<br>-+int PJSIP_EINVAL_ERR_EXCEPTION = -2;<br>- <br>- /* Parser constants */<br>- static pjsip_parser_const_t pconst =<br>-@@ -205,7 +207,6 @@ static unsigned long pj_strtoul_mindigit(const pj_str_t *str,<br>- /* Case insensitive comparison */<br>- #define parser_stricmp(s1, s2) (s1.slen!=s2.slen || pj_stricmp_alnum(&s1, &s2))<br>- <br>--<br>- /* Get a token and unescape */<br>- PJ_INLINE(void) parser_get_and_unescape(pj_scanner *scanner, pj_pool_t *pool,<br>- const pj_cis_t *spec, <br>-@@ -223,8 +224,6 @@ PJ_INLINE(void) parser_get_and_unescape(pj_scanner *scanner, pj_pool_t *pool,<br>- #endif<br>- }<br>- <br>--<br>--<br>- /* Syntax error handler for parser. */<br>- static void on_syntax_error(pj_scanner *scanner)<br>- {<br>-@@ -232,6 +231,60 @@ static void on_syntax_error(pj_scanner *scanner)<br>- PJ_THROW(PJSIP_SYN_ERR_EXCEPTION);<br>- }<br>- <br>-+/* Syntax error handler for parser. */<br>-+static void on_str_parse_error(const pj_str_t *str, int rc)<br>-+{<br>-+ char *s;<br>-+<br>-+ switch(rc) {<br>-+ case PJ_EINVAL:<br>-+ s = "NULL input string, invalid input string, or NULL return "\<br>-+ "value pointer";<br>-+ break;<br>-+ case PJ_ETOOSMALL:<br>-+ s = "String value was less than the minimum allowed value.";<br>-+ break;<br>-+ case PJ_ETOOBIG:<br>-+ s = "String value was greater than the maximum allowed value.";<br>-+ break;<br>-+ default:<br>-+ s = "Unknown error";<br>-+ }<br>-+<br>-+ if (str) {<br>-+ PJ_LOG(1, (THIS_FILE, "Error parsing '%.*s': %s",<br>-+ (int)str->slen, str->ptr, s));<br>-+ } else {<br>-+ PJ_LOG(1, (THIS_FILE, "Can't parse input string: %s", s));<br>-+ }<br>-+ PJ_THROW(PJSIP_EINVAL_ERR_EXCEPTION);<br>-+}<br>-+<br>-+static void strtoi_validate(const pj_str_t *str, int min_val,<br>-+ int max_val, int *value)<br>-+{ <br>-+ long retval;<br>-+ pj_status_t status;<br>-+<br>-+ if (!str || !value) {<br>-+ on_str_parse_error(str, PJ_EINVAL);<br>-+ }<br>-+ status = pj_strtol2(str, &retval);<br>-+ if (status != PJ_EINVAL) {<br>-+ if (min_val > retval) {<br>-+ *value = min_val;<br>-+ status = PJ_ETOOSMALL;<br>-+ } else if (retval > max_val) {<br>-+ *value = max_val;<br>-+ status = PJ_ETOOBIG;<br>-+ } else<br>-+ *value = (int)retval;<br>-+ }<br>-+<br>-+ if (status != PJ_SUCCESS)<br>-+ on_str_parse_error(str, status);<br>-+}<br>-+<br>- /* Get parser constants. */<br>- PJ_DEF(const pjsip_parser_const_t*) pjsip_parser_const(void)<br>- {<br>-@@ -285,6 +338,14 @@ static pj_status_t init_parser()<br>- PJ_ASSERT_RETURN(status == PJ_SUCCESS, status);<br>- <br>- /*<br>-+ * Invalid value exception.<br>-+ */<br>-+ pj_assert (PJSIP_EINVAL_ERR_EXCEPTION == -2);<br>-+ status = pj_exception_id_alloc("PJSIP invalid value error", <br>-+ &PJSIP_EINVAL_ERR_EXCEPTION);<br>-+ PJ_ASSERT_RETURN(status == PJ_SUCCESS, status);<br>-+<br>-+ /*<br>- * Init character input spec (cis)<br>- */<br>- <br>-@@ -502,6 +563,9 @@ void deinit_sip_parser(void)<br>- /* Deregister exception ID */<br>- pj_exception_id_free(PJSIP_SYN_ERR_EXCEPTION);<br>- PJSIP_SYN_ERR_EXCEPTION = -1;<br>-+<br>-+ pj_exception_id_free(PJSIP_EINVAL_ERR_EXCEPTION);<br>-+ PJSIP_EINVAL_ERR_EXCEPTION = -2;<br>- }<br>- pj_leave_critical_section();<br>- }<br>-@@ -766,7 +830,7 @@ PJ_DEF(pjsip_msg *) pjsip_parse_rdata( char *buf, pj_size_t size,<br>- }<br>- <br>- /* Determine if a message has been received. */<br>--PJ_DEF(pj_bool_t) pjsip_find_msg( const char *buf, pj_size_t size, <br>-+PJ_DEF(pj_status_t) pjsip_find_msg( const char *buf, pj_size_t size, <br>- pj_bool_t is_datagram, pj_size_t *msg_size)<br>- {<br>- #if PJ_HAS_TCP<br>-@@ -776,6 +840,7 @@ PJ_DEF(pj_bool_t) pjsip_find_msg( const char *buf, pj_size_t size,<br>- const char *line;<br>- int content_length = -1;<br>- pj_str_t cur_msg;<br>-+ pj_status_t status = PJ_SUCCESS;<br>- const pj_str_t end_hdr = { "\n\r\n", 3};<br>- <br>- *msg_size = size;<br>-@@ -836,9 +901,16 @@ PJ_DEF(pj_bool_t) pjsip_find_msg( const char *buf, pj_size_t size,<br>- pj_scan_get_newline(&scanner);<br>- <br>- /* Found a valid Content-Length header. */<br>-- content_length = pj_strtoul(&str_clen);<br>-+ strtoi_validate(&str_clen, PJSIP_MIN_CONTENT_LENGTH,<br>-+ PJSIP_MAX_CONTENT_LENGTH, &content_length);<br>- }<br>- PJ_CATCH_ANY {<br>-+ int eid = PJ_GET_EXCEPTION();<br>-+ if (eid == PJSIP_SYN_ERR_EXCEPTION) {<br>-+ status = PJSIP_EMISSINGHDR;<br>-+ } else if (eid == PJSIP_EINVAL_ERR_EXCEPTION) {<br>-+ status = PJSIP_EINVALIDHDR;<br>-+ }<br>- content_length = -1;<br>- }<br>- PJ_END<br>-@@ -858,7 +930,7 @@ PJ_DEF(pj_bool_t) pjsip_find_msg( const char *buf, pj_size_t size,<br>- <br>- /* Found Content-Length? */<br>- if (content_length == -1) {<br>-- return PJSIP_EMISSINGHDR;<br>-+ return status;<br>- }<br>- <br>- /* Enough packet received? */<br>-@@ -938,10 +1010,14 @@ static pj_bool_t is_next_sip_version(pj_scanner *scanner)<br>- static pjsip_msg *int_parse_msg( pjsip_parse_ctx *ctx,<br>- pjsip_parser_err_report *err_list)<br>- {<br>-- pj_bool_t parsing_headers;<br>-- pjsip_msg *msg = NULL;<br>-+ /* These variables require "volatile" so their values get<br>-+ * preserved when re-entering the PJ_TRY block after an error.<br>-+ */<br>-+ volatile pj_bool_t parsing_headers;<br>-+ pjsip_msg *volatile msg = NULL;<br>-+ pjsip_ctype_hdr *volatile ctype_hdr = NULL;<br>-+<br>- pj_str_t hname;<br>-- pjsip_ctype_hdr *ctype_hdr = NULL;<br>- pj_scanner *scanner = ctx->scanner;<br>- pj_pool_t *pool = ctx->pool;<br>- PJ_USE_EXCEPTION;<br>-@@ -1023,7 +1099,6 @@ parse_headers:<br>- hdr->name = hdr->sname = hname;<br>- }<br>- <br>-- <br>- /* Single parse of header line can produce multiple headers.<br>- * For example, if one Contact: header contains Contact list<br>- * separated by comma, then these Contacts will be split into<br>-@@ -1267,7 +1342,7 @@ static void int_parse_uri_host_port( pj_scanner *scanner,<br>- pj_str_t port;<br>- pj_scan_get_char(scanner);<br>- pj_scan_get(scanner, &pconst.pjsip_DIGIT_SPEC, &port);<br>-- *p_port = pj_strtoul(&port);<br>-+ strtoi_validate(&port, PJSIP_MIN_PORT, PJSIP_MAX_PORT, p_port);<br>- } else {<br>- *p_port = 0;<br>- }<br>-@@ -1458,8 +1533,8 @@ static void* int_parse_sip_url( pj_scanner *scanner,<br>- url->transport_param = pvalue;<br>- <br>- } else if (!parser_stricmp(pname, pconst.pjsip_TTL_STR) && pvalue.slen) {<br>-- url->ttl_param = pj_strtoul(&pvalue);<br>--<br>-+ strtoi_validate(&pvalue, PJSIP_MIN_TTL, PJSIP_MAX_TTL,<br>-+ &url->ttl_param);<br>- } else if (!parser_stricmp(pname, pconst.pjsip_MADDR_STR) && pvalue.slen) {<br>- url->maddr_param = pvalue;<br>- <br>-@@ -1595,7 +1670,8 @@ static void int_parse_status_line( pj_scanner *scanner,<br>- <br>- parse_sip_version(scanner);<br>- pj_scan_get( scanner, &pconst.pjsip_DIGIT_SPEC, &token);<br>-- status_line->code = pj_strtoul(&token);<br>-+ strtoi_validate(&token, PJSIP_MIN_STATUS_CODE, PJSIP_MAX_STATUS_CODE,<br>-+ &status_line->code);<br>- if (*scanner->curptr != '\r' && *scanner->curptr != '\n')<br>- pj_scan_get( scanner, &pconst.pjsip_NOT_NEWLINE, &status_line->reason);<br>- else<br>-@@ -1780,20 +1856,34 @@ static void int_parse_contact_param( pjsip_contact_hdr *hdr,<br>- if (!parser_stricmp(pname, pconst.pjsip_Q_STR) && pvalue.slen) {<br>- char *dot_pos = (char*) pj_memchr(pvalue.ptr, '.', pvalue.slen);<br>- if (!dot_pos) {<br>-- hdr->q1000 = pj_strtoul(&pvalue) * 1000;<br>-+ strtoi_validate(&pvalue, PJSIP_MIN_Q1000, PJSIP_MAX_Q1000,<br>-+ &hdr->q1000);<br>-+ hdr->q1000 *= 1000;<br>- } else {<br>- pj_str_t tmp = pvalue;<br>-+ unsigned long qval_frac;<br>- <br>- tmp.slen = dot_pos - pvalue.ptr;<br>-- hdr->q1000 = pj_strtoul(&tmp) * 1000;<br>-+ strtoi_validate(&tmp, PJSIP_MIN_Q1000, PJSIP_MAX_Q1000,<br>-+ &hdr->q1000);<br>-+ hdr->q1000 *= 1000;<br>- <br>- pvalue.slen = (pvalue.ptr+pvalue.slen) - (dot_pos+1);<br>- pvalue.ptr = dot_pos + 1;<br>-- hdr->q1000 += pj_strtoul_mindigit(&pvalue, 3);<br>-+ if (pvalue.slen > 3) {<br>-+ pvalue.slen = 3;<br>-+ }<br>-+ qval_frac = pj_strtoul_mindigit(&pvalue, 3);<br>-+ if ((unsigned)hdr->q1000 > (PJ_MAXINT32 - qval_frac)) {<br>-+ PJ_THROW(PJSIP_SYN_ERR_EXCEPTION);<br>-+ }<br>-+ hdr->q1000 += qval_frac;<br>- } <br>-- } else if (!parser_stricmp(pname, pconst.pjsip_EXPIRES_STR) && pvalue.slen) {<br>-- hdr->expires = pj_strtoul(&pvalue);<br>--<br>-+ } else if (!parser_stricmp(pname, pconst.pjsip_EXPIRES_STR) && <br>-+ pvalue.slen) <br>-+ {<br>-+ strtoi_validate(&pvalue, PJSIP_MIN_EXPIRES, PJSIP_MAX_EXPIRES,<br>-+ &hdr->expires);<br>- } else {<br>- pjsip_param *p = PJ_POOL_ALLOC_T(pool, pjsip_param);<br>- p->name = pname;<br>-@@ -1890,19 +1980,22 @@ static pjsip_hdr* parse_hdr_content_type( pjsip_parse_ctx *ctx )<br>- static pjsip_hdr* parse_hdr_cseq( pjsip_parse_ctx *ctx )<br>- {<br>- pj_str_t cseq, method;<br>-- pjsip_cseq_hdr *hdr;<br>-+ pjsip_cseq_hdr *hdr = NULL;<br>-+ int cseq_val = 0;<br>- <br>-- hdr = pjsip_cseq_hdr_create(ctx->pool);<br>- pj_scan_get( ctx->scanner, &pconst.pjsip_DIGIT_SPEC, &cseq);<br>-- hdr->cseq = pj_strtoul(&cseq);<br>-+ strtoi_validate(&cseq, PJSIP_MIN_CSEQ, PJSIP_MAX_CSEQ, &cseq_val);<br>- <br>-- pj_scan_get( ctx->scanner, &pconst.pjsip_TOKEN_SPEC, &method);<br>-- pjsip_method_init_np(&hdr->method, &method);<br>-+ hdr = pjsip_cseq_hdr_create(ctx->pool);<br>-+ hdr->cseq = cseq_val;<br>- <br>-+ pj_scan_get( ctx->scanner, &pconst.pjsip_TOKEN_SPEC, &method);<br>- parse_hdr_end( ctx->scanner );<br>- <br>-- if (ctx->rdata)<br>-+ pjsip_method_init_np(&hdr->method, &method);<br>-+ if (ctx->rdata) {<br>- ctx->rdata->msg_info.cseq = hdr;<br>-+ }<br>- <br>- return (pjsip_hdr*)hdr;<br>- }<br>-@@ -1984,7 +2077,8 @@ static pjsip_hdr* parse_hdr_retry_after(pjsip_parse_ctx *ctx)<br>- hdr = pjsip_retry_after_hdr_create(ctx->pool, 0);<br>- <br>- pj_scan_get(scanner, &pconst.pjsip_DIGIT_SPEC, &tmp);<br>-- hdr->ivalue = pj_strtoul(&tmp);<br>-+ strtoi_validate(&tmp, PJSIP_MIN_RETRY_AFTER, PJSIP_MAX_RETRY_AFTER,<br>-+ &hdr->ivalue);<br>- <br>- while (!pj_scan_is_eof(scanner) && *scanner->curptr!='\r' &&<br>- *scanner->curptr!='\n')<br>-@@ -2073,7 +2167,8 @@ static void int_parse_via_param( pjsip_via_hdr *hdr, pj_scanner *scanner,<br>- hdr->branch_param = pvalue;<br>- <br>- } else if (!parser_stricmp(pname, pconst.pjsip_TTL_STR) && pvalue.slen) {<br>-- hdr->ttl_param = pj_strtoul(&pvalue);<br>-+ strtoi_validate(&pvalue, PJSIP_MIN_TTL, PJSIP_MAX_TTL,<br>-+ &hdr->ttl_param);<br>- <br>- } else if (!parser_stricmp(pname, pconst.pjsip_MADDR_STR) && pvalue.slen) {<br>- hdr->maddr_param = pvalue;<br>-@@ -2082,9 +2177,10 @@ static void int_parse_via_param( pjsip_via_hdr *hdr, pj_scanner *scanner,<br>- hdr->recvd_param = pvalue;<br>- <br>- } else if (!parser_stricmp(pname, pconst.pjsip_RPORT_STR)) {<br>-- if (pvalue.slen)<br>-- hdr->rport_param = pj_strtoul(&pvalue);<br>-- else<br>-+ if (pvalue.slen) {<br>-+ strtoi_validate(&pvalue, PJSIP_MIN_PORT, PJSIP_MAX_PORT,<br>-+ &hdr->rport_param);<br>-+ } else<br>- hdr->rport_param = 0;<br>- } else {<br>- pjsip_param *p = PJ_POOL_ALLOC_T(pool, pjsip_param);<br>-@@ -2213,7 +2309,8 @@ static pjsip_hdr* parse_hdr_via( pjsip_parse_ctx *ctx )<br>- pj_str_t digit;<br>- pj_scan_get_char(scanner);<br>- pj_scan_get(scanner, &pconst.pjsip_DIGIT_SPEC, &digit);<br>-- hdr->sent_by.port = pj_strtoul(&digit);<br>-+ strtoi_validate(&digit, PJSIP_MIN_PORT, PJSIP_MAX_PORT,<br>-+ &hdr->sent_by.port);<br>- }<br>- <br>- int_parse_via_param(hdr, scanner, ctx->pool);<br>-@@ -2298,9 +2395,10 @@ PJ_DEF(pj_status_t) pjsip_parse_headers( pj_pool_t *pool, char *input,<br>- unsigned options)<br>- {<br>- enum { STOP_ON_ERROR = 1 };<br>-+ pj_str_t hname;<br>- pj_scanner scanner;<br>- pjsip_parse_ctx ctx;<br>-- pj_str_t hname;<br>-+<br>- PJ_USE_EXCEPTION;<br>- <br>- pj_scan_init(&scanner, input, size, PJ_SCAN_AUTOSKIP_WS_HEADER,<br>-@@ -2323,7 +2421,7 @@ retry_parse:<br>- */<br>- hname.slen = 0;<br>- <br>-- /* Get hname. */<br>-+ /* Get hname. */ <br>- pj_scan_get( &scanner, &pconst.pjsip_TOKEN_SPEC, &hname);<br>- if (pj_scan_get_char( &scanner ) != ':') {<br>- PJ_THROW(PJSIP_SYN_ERR_EXCEPTION);<br>-diff --git a/pjsip/src/pjsip/sip_transaction.c b/pjsip/src/pjsip/sip_transaction.c<br>-index f3be93beb..7ac3d1b76 100644<br>---- a/pjsip/src/pjsip/sip_transaction.c<br>-+++ b/pjsip/src/pjsip/sip_transaction.c<br>-@@ -289,11 +289,11 @@ static pj_status_t create_tsx_key_2543( pj_pool_t *pool,<br>- <br>- /* Calculate length required. */<br>- len_required = method->name.slen + /* Method */<br>-- 9 + /* CSeq number */<br>-+ 11 + /* CSeq number */<br>- rdata->msg_info.from->tag.slen + /* From tag. */<br>- rdata->msg_info.cid->id.slen + /* Call-ID */<br>- host->slen + /* Via host. */<br>-- 9 + /* Via port. */<br>-+ 11 + /* Via port. */<br>- 16; /* Separator+Allowance. */<br>- key = p = (char*) pj_pool_alloc(pool, len_required);<br>- <br>-diff --git a/pjsip/src/pjsip/sip_transport.c b/pjsip/src/pjsip/sip_transport.c<br>-index 0c338e8fd..b24cbb411 100644<br>---- a/pjsip/src/pjsip/sip_transport.c<br>-+++ b/pjsip/src/pjsip/sip_transport.c<br>-@@ -1848,7 +1848,7 @@ PJ_DEF(pj_ssize_t) pjsip_tpmgr_receive_packet( pjsip_tpmgr *mgr,<br>- /* Check for parsing syntax error */<br>- if (msg==NULL || !pj_list_empty(&rdata->msg_info.parse_err)) {<br>- pjsip_parser_err_report *err;<br>-- char buf[128];<br>-+ char buf[256];<br>- pj_str_t tmp;<br>- <br>- /* Gather syntax error information */<br>-@@ -1862,7 +1862,10 @@ PJ_DEF(pj_ssize_t) pjsip_tpmgr_receive_packet( pjsip_tpmgr *mgr,<br>- pj_exception_id_name(err->except_code),<br>- (int)err->hname.slen, err->hname.ptr,<br>- err->line, err->col);<br>-- if (len > 0 && len < (int) (sizeof(buf)-tmp.slen)) {<br>-+ if (len >= (int)sizeof(buf)-tmp.slen) {<br>-+ len = (int)sizeof(buf)-tmp.slen;<br>-+ }<br>-+ if (len > 0) {<br>- tmp.slen += len;<br>- }<br>- err = err->next;<br>--- <br>-2.13.6<br>-<br>diff --git a/third-party/versions.mak b/third-party/versions.mak<br>index 4f645f1..a90a523 100644<br>--- a/third-party/versions.mak<br>+++ b/third-party/versions.mak<br>@@ -1,2 +1,2 @@<br> <br>-PJPROJECT_VERSION = 2.7<br>+PJPROJECT_VERSION = 2.7.1<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7207">change 7207</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/7207"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I55cea8e734b318c2df9daf86aa0802c559ec8357 </div>
<div style="display:none"> Gerrit-Change-Number: 7207 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>