diff options
author | Jakub Adam <jakub.adam@collabora.com> | 2019-01-03 21:26:41 +0100 |
---|---|---|
committer | Olivier CrĂȘte <olivier.crete@ocrete.ca> | 2019-01-04 23:40:18 +0000 |
commit | 6b1eec0630516698ac9cd3343ef7eb8515fee231 (patch) | |
tree | afaf9c7e82c3f01b4e6b63a91710dc1bd2cc5e82 /socket | |
parent | 603ab1d45d0103fc51031152a10ddbdaf7dadf86 (diff) | |
download | libnice-6b1eec0630516698ac9cd3343ef7eb8515fee231.tar.gz |
udp-turn: Avoid potential integer overflow
Diffstat (limited to 'socket')
-rw-r--r-- | socket/udp-turn.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/socket/udp-turn.c b/socket/udp-turn.c index 1bc5e03..c3b152d 100644 --- a/socket/udp-turn.c +++ b/socket/udp-turn.c @@ -362,7 +362,7 @@ socket_recv_messages (NiceSocket *sock, guint f_buffer_len = priv->fragment_buffer->len; for (i = 0; i < n_recv_messages && f_buffer_len >= sizeof (guint16); ++i) { - guint16 msg_len = ((f_buffer[0] << 8) | f_buffer[1]) + sizeof (guint16); + guint32 msg_len = ((f_buffer[0] << 8) | f_buffer[1]) + sizeof (guint16); if (msg_len > f_buffer_len) { /* The next message in the buffer isn't complete yet. Wait for more @@ -450,7 +450,7 @@ socket_recv_messages (NiceSocket *sock, if (nice_socket_is_reliable (sock) && parsed_buffer_length > 0) { /* Determine the portion of the current NiceInputMessage we can already * return. */ - guint16 msg_len = 0; + gint32 msg_len = 0; if (!priv->fragment_buffer) { msg_len = ((buffer[0] << 8) | buffer[1]) + sizeof (guint16); if (msg_len > parsed_buffer_length) { |