Commit db05e8b0 authored by Olivier Crête's avatar Olivier Crête

Make clang-analyzer happy

Various little things, none of which should make a functional difference.
parent 0672758b
...@@ -1552,7 +1552,6 @@ pseudo_tcp_socket_recv_messages (PseudoTcpSocket *self, ...@@ -1552,7 +1552,6 @@ pseudo_tcp_socket_recv_messages (PseudoTcpSocket *self,
if (len == 0) { if (len == 0) {
/* Reached EOS. */ /* Reached EOS. */
len = 0;
goto done; goto done;
} else if (len < 0 && } else if (len < 0 &&
pseudo_tcp_socket_get_error (self) == EWOULDBLOCK) { pseudo_tcp_socket_get_error (self) == EWOULDBLOCK) {
......
...@@ -606,7 +606,7 @@ static gboolean priv_conn_check_tick_unlocked (NiceAgent *agent) ...@@ -606,7 +606,7 @@ static gboolean priv_conn_check_tick_unlocked (NiceAgent *agent)
/* step: when there's no pair in the Waiting state, /* step: when there's no pair in the Waiting state,
* unfreeze a new pair and check it * unfreeze a new pair and check it
*/ */
res = priv_conn_check_unfreeze_next (agent); priv_conn_check_unfreeze_next (agent);
for (i = agent->streams; i ; i = i->next) { for (i = agent->streams; i ; i = i->next) {
NiceStream *stream = i->data; NiceStream *stream = i->data;
......
...@@ -406,9 +406,8 @@ socket_recv_messages (NiceSocket *sock, ...@@ -406,9 +406,8 @@ socket_recv_messages (NiceSocket *sock,
/* Split up the monolithic buffer again into the caller-provided buffers. */ /* Split up the monolithic buffer again into the caller-provided buffers. */
if (parsed_buffer_length > 0 && allocated_buffer) { if (parsed_buffer_length > 0 && allocated_buffer) {
parsed_buffer_length = memcpy_buffer_to_input_message (message, buffer,
memcpy_buffer_to_input_message (message, buffer, parsed_buffer_length);
parsed_buffer_length);
} }
if (allocated_buffer) if (allocated_buffer)
......
...@@ -479,7 +479,7 @@ StunUsageBindReturn stun_usage_bind_run (const struct sockaddr *srv, ...@@ -479,7 +479,7 @@ StunUsageBindReturn stun_usage_bind_run (const struct sockaddr *srv,
size_t len; size_t len;
StunUsageTransReturn ret; StunUsageTransReturn ret;
int val; int val;
struct sockaddr_storage alternate_server; struct sockaddr_storage alternate_server = { AF_UNSPEC } ;
socklen_t alternate_server_len = sizeof (alternate_server); socklen_t alternate_server_len = sizeof (alternate_server);
StunUsageBindReturn bind_ret; StunUsageBindReturn bind_ret;
...@@ -548,6 +548,8 @@ StunUsageBindReturn stun_usage_bind_run (const struct sockaddr *srv, ...@@ -548,6 +548,8 @@ StunUsageBindReturn stun_usage_bind_run (const struct sockaddr *srv,
if (bind_ret == STUN_USAGE_BIND_RETURN_ALTERNATE_SERVER) { if (bind_ret == STUN_USAGE_BIND_RETURN_ALTERNATE_SERVER) {
stun_trans_deinit (&trans); stun_trans_deinit (&trans);
assert (alternate_server.ss_family != AF_UNSPEC);
ret = stun_trans_create (&trans, SOCK_DGRAM, 0, ret = stun_trans_create (&trans, SOCK_DGRAM, 0,
(struct sockaddr *) &alternate_server, alternate_server_len); (struct sockaddr *) &alternate_server, alternate_server_len);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment